From db4410e5681837a782a6956614563e9fb9724ad7 Mon Sep 17 00:00:00 2001 From: Jacek Kowalski <Jacek@jacekk.info> Date: Sun, 14 Apr 2019 09:33:45 +0000 Subject: [PATCH] Reduce variable redefinitions --- map.js | 62 +++++++++++++++++-------------- 1 files changed, 34 insertions(+), 28 deletions(-) diff --git a/map.js b/map.js index e8550d4..11c7aa0 100644 --- a/map.js +++ b/map.js @@ -226,8 +226,7 @@ stop_selected_layer.setStyle(style); - var feature = null; - var prefix = null; + var feature, prefix; for(var i = 0; i < stops.length; i++) { feature = null; if(stops[i].getId) { @@ -360,7 +359,7 @@ ).done(function(data) { if(!data || !data.paths || !data.paths[0] || !data.paths[0].wayPoints) return; - var point = null; + var point; var points = []; for(var i = 0; i < data.paths[0].wayPoints.length; i++) { point = data.paths[0].wayPoints[i]; @@ -396,8 +395,10 @@ deleteChildren(table); - for(var i = 0, il = data.old.length; i < il; i++) { - var tr = document.createElement('tr'); + var i, il; + var tr; + for(i = 0, il = data.old.length; i < il; i++) { + tr = document.createElement('tr'); addCellWithText(tr, data.old[i].actualTime || data.old[i].plannedTime); addCellWithText(tr, data.old[i].stop_seq_num + '. ' + data.old[i].stop.name); @@ -407,8 +408,8 @@ var stopsToMark = []; - for(var i = 0, il = data.actual.length; i < il; i++) { - var tr = document.createElement('tr'); + for(i = 0, il = data.actual.length; i < il; i++) { + tr = document.createElement('tr'); addCellWithText(tr, data.actual[i].actualTime || data.actual[i].plannedTime); addCellWithText(tr, data.actual[i].stop_seq_num + '. ' + data.actual[i].stop.name); @@ -438,32 +439,36 @@ ).done(function(data) { deleteChildren(table); - for(var i = 0, il = data.old.length; i < il; i++) { - var tr = document.createElement('tr'); + var i, il; + var tr, dir_cell, vehicle, status, status_cell, delay, delay_cell; + for(i = 0, il = data.old.length; i < il; i++) { + tr = document.createElement('tr'); addCellWithText(tr, data.old[i].patternText); - var dir_cell = addCellWithText(tr, data.old[i].direction); - var vehicle = parseVehicle(data.old[i].vehicleId); + dir_cell = addCellWithText(tr, data.old[i].direction); + vehicle = parseVehicle(data.old[i].vehicleId); dir_cell.appendChild(displayVehicle(vehicle)); - var status = parseStatus(data.old[i]); - addCellWithText(tr, status); - addCellWithText(tr, ''); + status = parseStatus(data.old[i]); + status_cell = addCellWithText(tr, status); + delay_cell = addCellWithText(tr, ''); tr.className = 'active'; table.appendChild(tr); } - for(var i = 0, il = data.actual.length; i < il; i++) { - var tr = document.createElement('tr'); + for(i = 0, il = data.actual.length; i < il; i++) { + tr = document.createElement('tr'); addCellWithText(tr, data.actual[i].patternText); - var dir_cell = addCellWithText(tr, data.actual[i].direction); - var vehicle = parseVehicle(data.actual[i].vehicleId); + dir_cell = addCellWithText(tr, data.actual[i].direction); + vehicle = parseVehicle(data.actual[i].vehicleId); dir_cell.appendChild(displayVehicle(vehicle)); - var status = parseStatus(data.actual[i]); - var status_cell = addCellWithText(tr, status); - var delay = parseDelay(data.actual[i]); - var delay_cell = addCellWithText(tr, delay); + status = parseStatus(data.actual[i]); + status_cell = addCellWithText(tr, status); + delay = parseDelay(data.actual[i]); + delay_cell = addCellWithText(tr, delay); - if(status == lang.boarding_sign) { + if(data.actual[i].status === 'DEPARTED') { + tr.className = 'active'; + } else if(status == lang.boarding_sign) { tr.className = 'success'; status_cell.className = 'status-boarding'; } else if(parseInt(delay) > 9) { @@ -642,18 +647,19 @@ addParaWithText(div, lang.select_feature); + var feature, p, a, full_type, typeName; for(var i = 0; i < features.length; i++) { - var feature = features[i]; + feature = features[i]; - var p = document.createElement('p'); - var a = document.createElement('a'); + p = document.createElement('p'); + a = document.createElement('a'); p.appendChild(a); a.addEventListener('click', function(feature) { return function() { featureClicked(feature); }}(feature)); - var full_type = feature.getId().match(/^[a-z]+/)[0]; - var typeName = lang.types[full_type]; + full_type = feature.getId().match(/^[a-z]+/)[0]; + typeName = lang.types[full_type]; if(typeof typeName === 'undefined') { typeName = ''; } -- Gitblit v1.9.1