开发者最容易犯的13个JavaScript错误

开发者最容易犯的JavaScript错误，总结出13个。这些当中可能少不了你犯的错误。我们描述了这些陋习，并列出来解决办法，希望对开发者有帮助。

1.for..in数组迭代的用法 Usage of for..in to iterate Arrays

举例：

1. var myArray = [ “a”, “b”, “c” ];  
2. var totalElements = myArray.length;  
3. for (var i = 0; i < totalElements; i++) {  
4. console.log(myArray[i]);  
5. }  

这里主要的问题是语句中的“for..in”不能保证顺序，这意味着你将获得不同的执行结果。此外，如果有人增加一些其他自定义功能的函数Array.prototype，你的循环将重复遍历这些函数，就像原数组项。

解决办法：一直使用规则的for..in循环来遍历数组。

1. var myArray = [ “a”, “b”, “c” ];  
2. for (var i=0; i<myArray.length; i++) {  
3. console.log(myArray[i]);  
4. }  

2.Array dimensions

举例

1. var myArray = new Array(10); 

第二个问题是开发者使用数组构成器来创建数组，技术上是正确的，然而会比文字符号（literal notation）。

解决办法：使用文字符号来初始化数组，不要预定义数组长度。

var myArray = [];

3.Undefined properties

举例：

1. var myObject = {  
2.     someProperty: “value”,  
3.     someOtherProperty: undefined  
4. } 

未定义属性，将在对象中创建元素（键‘someOtherProperty’和值‘undefined’.）。如果你遍历数组，检测已存在的元素，那么下面的语句将都返回“未定义/undefined”

typeof myObject['someOtherProperty'] // undefined  

typeof myObject['unknownProperty'] // undefined  

解决办法：如果你想明确声明对象中的未初始化的属性，标记它们为Null。

1. var myObject = {  
2. someProperty: “value”,  
3. someOtherProperty: null  
4. } 

4.Misuse of Closures

举例：

1. function(a, b, c) {  
2. var d = 10;  
3. var element = document.getElementById(‘myID’);  
4. element.onclick = (function(a, b, c, d) {  
5. return function() {  
6. alert (a + b + c + d);  
7. }  
8. })(a, b, c, d);  
9. }  

这里开发者使用两个函数来传递参数a、b、c到onclick handler。双函数根本不需要，徒增代码的复杂性。

变量abc已经在局部函数中被定义，因为他们已经在主函数中作为参数被声明。局部函数中的任何函数都可创建主函数中定义的所有变量的闭包。因此不需要再次传递它们。

看看这里JavaScript Closures FAQ 了解更多。

解决办法：使用闭合来简化代码。

1. function (a, b, c) {  
2. var d = 10;  
3. var element = document.getElementById(‘myID’);  
4. element.onclick = function() {  
5. //a, b, and c come from the outer function arguments.  
6. //d come from the outer function variable declarations.  
7. //and all of them are in my closure  
8. alert (a + b + c + d);  
9. };  
10. }  

5.Closures in loops

举例：

1. var elements = document.getElementByTagName(‘div’);  
2. for (var i = 0; i<elements.length; i++) {  
3. elements[i].onclick = function() {  
4. alert(“Div number “ + i);  
5. }  
6. }  

在这里例子里面，当用户点击不同的divs时，我们想触发一个动作(显示“Div number 1”, “Div number 2”… 等) 。然而，如果你在页面有10个divs，他们全部都会显示“Div number 10”。

问题是当我们使用局部函数创建一个闭包时，函数中的代码可以访问变量i。关键是函数内部i和函数外部i涉及同样的变量。当我们的循环结束，i指向了值10，所以局部函数中的i的值将是10。

解决办法：使用第二函数来传递正确的值。

1. var elements = document.getElementsByTagName(‘div’);  
2. for (var i = 0; i<elements.length; i++) {  
3. elements[i].onclick = (function(idx) { //Outer function  
4. return function() { //Inner function  
5. alert(“Div number “ + idx);  
6. }  
7. })(i);  
8. }  

6.Memory leaks with DOM objects

举例：

1. function attachEvents() {  
2. var element = document.getElementById(‘myID’);  
3. element.onclick = function() {  
4. alert(“Element clicked”);  
5. }  
6. };  
7. attachEvents();  

该代码创建了一个引用循环。变量元素包含函数的引用（归于onclick属性）。同时，函数保持一个DOM元素的引用（提示函数内部可以访问元素，因为闭包。）。所以JavaScript垃圾收集器不能清除元素或是函数，因为他们被相互引用。大部分的JavaScript引擎对于清除循环应用都不够聪明。

解决办法：避免那些闭包，或者不去做函数内的循环引用。

1. function attachEvents() {  
2. var element = document.getElementById(‘myID’);  
3. element.onclick = function() {  
4. //Remove element, so function can be collected by GC  
5. delete element;  
6. alert(“Element clicked”);  
7. }  
8. };  
9. attachEvents();  

7.区别整数数字和浮点数字Differentiate float numbers from integer numbers

举例：

1. var myNumber = 3.5;  
2. var myResult = 3.5 + 1.0; //We use .0 to keep the result as float  

在JavaScript中，浮点与整数间没有区别。事实上，JavaScript中的每个数字都表示使用双精度64位格式IEEE 754。简单理解，所有数字都是浮点。

解决办法：不要使用小数（decimals），转换数字（numbers）到浮点（floats）。

1. var myNumber = 3.5;  
2. var myResult = 3.5 + 1; //Result is 4.5, as expected  

8.Usage of with() as a shortcut

举例：

1. team.attackers.myWarrior = { attack: 1, speed: 3, magic: 5};  
2. with (team.attackers.myWarrior){  
3. console.log ( “Your warrior power is ” + (attack * speed));  
4. }  

讨论with()之前，要明白JavaScript contexts如何工作的。每个函数都有一个执行context（语句），简单来说，包括函数可以访问的所有的变量。因此context包含arguments和定义变量。

with()真正是做什么？是插入对象到context链，它在当前context和父级context间植入。就像你看到的with()的快捷方式会非常慢。

解决办法：不要使用with() for shortcuts，仅for context injection，如果你确实需要时。

1. team.attackers.myWarrior = { attack: 1, speed: 3, magic: 5};  
2. var sc = team.attackers.myWarrior;  
3. console.log(“Your warrior power is ” + (sc.attack * sc.speed));  

9.setTimeout/setInterval 字符串的用法 （Usage of strings with setTimeout/setInterval）

举例：

1. function log1() { console.log(document.location); }  
2. function log2(arg) { console.log(arg); }  
3. var myValue = “test”;  
4. setTimeout(“log1()”, 100);  
5. setTimeout(“log2(” + myValue + “)”, 200);  

setTimeout()和setInterval()可被或一个函数或一个字符串作为首个参数。如果你传递一个字符串，引擎将创建一个新函数（使用函数构造器），这在一些浏览器中会非常慢。相反，传递函数本身作为首个参数，更快、更强大、更干净。

解决办法：一定不要使用strings for setTimeout()或setInterval()。

1. function log1() { console.log(document.location); }  
2. function log2(arg) { console.log(arg); }  
3. var myValue = “test”;  
4. setTimeout(log1, 100); //Reference to a function  
5. setTimeout(function(){ //Get arg value using closures  
6. log2(arg);  
7. }, 200);  

10.setInterval()的用法（Usage of setInterval() for heavy functions）

举例：

1. function domOperations() {  
2. //Heavy DOM operations, takes about 300ms  
3. }  
4. setInterval(domOperations, 200);  

setInterval()将一函数列入计划被执行，仅是在没有另外一个执行在主执行队列中等待。如果在队列中没有另外一个执行，JavaScript引擎将在下一个队列中执行。这可能导致跳过执行或者运行2个不同的执行，没有在它们之间等待200ms的情况下。

一定要搞清，setInterval()没有考虑进多长时间domOperations()来完成任务。

解决办法：避免 setInterval()，使用 setTimeout()

1. function domOperations() {  
2. //Heavy DOM operations, takes about 300ms  
3. //After all the job is done, set another timeout for 200 ms  
4. setTimeout(domOperations, 200);  
5. }  
6. setTimeout(domOperations, 200);  

11.Misuse of ‘this’

这个常用错误，没有例子，因为非常难创建来演示。this的值在JavaScript中与其他语言有很大的不同。

函数中的this值被定义是在当函数被调用时，而非声明的时间，这一点非常重要。下面的案例中，函数内this有不同的含义。

* Regular function: myFunction(‘arg1’);

this points to the global object, wich is window for all browers.

* Method: someObject.myFunction(‘arg1’);

this points to object before the dot, someObject in this case.

* Constructor: var something = new myFunction(‘arg1’);

this points to an empty Object.

* Using call()/apply(): myFunction.call(someObject, ‘arg1’);

this points to the object passed as first argument.

12.eval()访问动态属性的用法Usage of eval() to access dynamic properties

举例：

1. var myObject = { p1: 1, p2: 2, p3: 3};  
2. var i = 2;  
3. var myResult = eval(‘myObject.p’+i);  

主要问题在于使用eval()开始一个新的执行语句，会非常的慢。

解决办法：使用方括号表示法（square bracket notation）代替 eval()。

1. var myObject = { p1: 1, p2: 2, p3: 3};  
2. var i = 2;  
3. var myResult = myObject[“p”+i];  

13.未定义(undefined)作为变量的用法Usage of undefined as a variable

举例：

1. if ( myVar === undefined ) {  
2. //Do something  
3. }  

在上面的例子中，未定义实际上是一变量。所有的JavaScript引擎会创建初始化的变量window.undefined给未定义作为值。然而注意的是变量不仅是可读，任何其他的代码可以更改它的值。很奇怪能找到window.undefined用来未定义的不同值的场景，但是为什么冒险呢？

解决办法：检查未定义时，使用typeof。

1. if ( typeof myVar === “undefined” ) {  
2. //Do something  
3. }  

英文链接：Top 13 JavaScript Mistakes

译文链接：开源中国社区