let
or const
instead of var
JS-02391417 },
1418 ],
1419 };
1420 var check = intersectionCheck(geoJson);1421 //join array elements with whitespace and comma
1422 req.body["yirmibesbin"] = check.yirmibes.join(", ");
1423 req.body["yuzbin"] = check.yuz.join(", ");
1400 req.body["datum"]
1401 );
1402 coordinates.push([parseFloat(close.lng), parseFloat(close.lat)]);
1403 var geoJson = {1404 type: "FeatureCollection",1405 features: [1406 {1407 type: "Feature",1408 properties: {1409 mytag: "datdat",1410 name: "datdat",1411 tessellate: true,1412 },1413 geometry: {1414 type: "Polygon",1415 coordinates: [coordinates],1416 },1417 },1418 ],1419 };1420 var check = intersectionCheck(geoJson);
1421 //join array elements with whitespace and comma
1422 req.body["yirmibesbin"] = check.yirmibes.join(", ");
1393 coordinates.push([cornerPoint.lng, cornerPoint.lat]);
1394 }
1395
1396 var close = converter(1397 parseFloat(corners[0].split(",")[0]),1398 parseFloat(corners[0].split(",")[1]),1399 req.body["zone"].length > 1 ? req.body["zone"][0] : req.body["zone"][0],1400 req.body["datum"]1401 );1402 coordinates.push([parseFloat(close.lng), parseFloat(close.lat)]);
1403 var geoJson = {
1404 type: "FeatureCollection",
1384 var x = parseFloat(cornerCoordinates[0]);
1385 var y = parseFloat(cornerCoordinates[1]);
1386
1387 var cornerPoint = converter(1388 x,1389 y,1390 req.body["zone"].length > 1 ? req.body["zone"][i] : req.body["zone"][0],1391 req.body["datum"]1392 );1393 coordinates.push([cornerPoint.lng, cornerPoint.lat]);
1394 }
1395
1382 var corner = corners[i];
1383 var cornerCoordinates = corner.split(",");
1384 var x = parseFloat(cornerCoordinates[0]);
1385 var y = parseFloat(cornerCoordinates[1]);1386
1387 var cornerPoint = converter(
1388 x,
It is recommended to use let
or const
over var
.
This will help prevent re-declaration of variables that are in the global scope when using var
.
ES6 allows programmers to create variables with block scope instead of function scope using the let
and const
keywords.
Block scope is common in many other programming languages and helps programmers avoid mistakes such as this one:
var count = people.length;
var enoughFood = count > sandwiches.length;
if (enoughFood) {
var count = sandwiches.length; // accidentally overriding the count variable
console.log("We have " + count + " sandwiches for everyone. Plenty for all!");
}
// our count variable is no longer accurate
console.log("We have " + count + " people and " + sandwiches.length + " sandwiches!");
Block scoped variables shadow outer declarations instead of writing to them.
NOTE: There are certain edge cases where users might want to consider var. Consider this example:
var lib = lib || { run: () => {} }
Here, lib
might be a library that is exposed to an HTML file using a <script>
tag.
The var
keyword helps avoid re-writing lib
if it has already been declared via an injected script that was executed before this one.
Ideally, you should let bundlers worry about cases like this.
But if you want to use var
anyway, consider using a skipcq comment, or disabling the issue altogether.
var x = "y";
var CONFIG = {};
let x = "y";
const CONFIG = {};