Browse Source

refactoring to get the code through the jslint tool ;)

* combined various variable defintions
* moved the invocation into the parens that contain the function
* some minor things
master
T. Meissner 12 years ago
parent
commit
9053b6997e
1 changed files with 16 additions and 18 deletions
  1. +16
    -18
      scripts/haltestellen.js

+ 16
- 18
scripts/haltestellen.js View File

@ -31,8 +31,8 @@ function ajaxCall(dataUrl, outputElement, callback, responseType) {
if (DEBUG === 1) {console.log("ajax function");} if (DEBUG === 1) {console.log("ajax function");}
// variable definitions // variable definitions
var response;
var request = getHTTPObject(); // get the xmlhttp object which is supported
var response,
request = getHTTPObject(); // get the xmlhttp object which is supported
outputElement.innerHTML = "Lade Daten ..."; outputElement.innerHTML = "Lade Daten ...";
@ -67,8 +67,10 @@ function ajaxCall(dataUrl, outputElement, callback, responseType) {
//variable definitions //variable definitions
var serverUrl;
var target = document.getElementById("output");
var serverUrl, // this will hold the url we call with the ajaxCall() function
target = document.getElementById("output"), // get the html area where we print out the data
searchForm = document.getElementById("search-form"), // get the search form
haltestelle; // object with methods to get and process the data
// debug log // debug log
if (DEBUG === 1) {console.log("anonymous function");} if (DEBUG === 1) {console.log("anonymous function");}
@ -80,15 +82,11 @@ function ajaxCall(dataUrl, outputElement, callback, responseType) {
if (serverUrl.indexOf("https") === -1) { if (serverUrl.indexOf("https") === -1) {
serverUrl += "/cgi-bin/"; serverUrl += "/cgi-bin/";
} else { } else {
var path = "/" + window.location.pathname.split("/")[1];
serverUrl += path + "/cgi-bin/";
serverUrl += "/" + window.location.pathname.split("/")[1] + "/cgi-bin/";
} }
// get the search form
var searchForm = document.getElementById("search-form");
// haltestelle object // haltestelle object
var haltestelle = {
haltestelle = {
getAbfahrten : function(event) { getAbfahrten : function(event) {
@ -99,8 +97,8 @@ function ajaxCall(dataUrl, outputElement, callback, responseType) {
event.preventDefault(); event.preventDefault();
// construct ajax url // construct ajax url
var hstName = document.getElementById("q").value;
var hstUrl = encodeURI(serverUrl + "abfahrtsmonitor.py?ort=dresden&hst=" + hstName);
var hstName = document.getElementById("q").value,
hstUrl = encodeURI(serverUrl + "abfahrtsmonitor.py?ort=dresden&hst=" + hstName);
// get the data from the server with an ajax call // get the data from the server with an ajax call
ajaxCall(hstUrl, target, haltestelle.processAbfahrten, "text"); ajaxCall(hstUrl, target, haltestelle.processAbfahrten, "text");
@ -114,11 +112,11 @@ function ajaxCall(dataUrl, outputElement, callback, responseType) {
if (DEBUG === 1) {console.log("received data: " + data);} if (DEBUG === 1) {console.log("received data: " + data);}
//variable definitions //variable definitions
var i;
var y;
var htmlOutput;
var entry;
var dataLength;
var i,
y,
htmlOutput,
entry,
dataLength;
// process of response only if it's not empty // process of response only if it's not empty
if (data.indexOf("[]") === -1) { if (data.indexOf("[]") === -1) {
@ -171,4 +169,4 @@ function ajaxCall(dataUrl, outputElement, callback, responseType) {
//event listeners //event listeners
searchForm.addEventListener("submit", haltestelle.getAbfahrten, false); searchForm.addEventListener("submit", haltestelle.getAbfahrten, false);
})(); // end of anonymous function
}()); // end of anonymous function

Loading…
Cancel
Save