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

51CTO推荐专题：JavaScript学习笔记 有问有答

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

举例：

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

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

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

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

2. 数组维度 Array dimensions

举例

var myArray = new Array(10); 

这里有两个不同的问题。首先，开发者尝试创建一个包含10项的数组，这将创建10个空槽的阵列。然而，如果你试图得到一数组项，你将得到”未定义“的结果。换句话说，效果就像你没有保存内存空间。没有真正的好原因来预定义数组长度。

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

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

var myArray = []; 

3. 未定义属性 Undefined properties

举例：

var myObject = {  
    someProperty: “value”,  
    someOtherProperty: undefined  
} 

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

typeof myObject['someOtherProperty'] // undefined

typeof myObject['unknownProperty'] // undefined

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

var myObject = {  
    someProperty: “value”,  
    someOtherProperty: null 
} 

4. 闭包的滥用 Misuse of Closures

举例：

function(a, b, c) {  
    var d = 10;  
    var element = document.getElementById(‘myID’);  
    element.onclick = (function(a, b, c, d) {  
        return function() {  
            alert (a + b + c + d);  
        }  
    })(a, b, c, d);  
} 

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

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

解决办法：使用闭环来简化你的代码。

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

5. 循环中的闭包 Closures in loops

举例：

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

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

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

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

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

6. DOM对象的内测泄漏 Memory leaks with DOM objects

举例：

function attachEvents() {  
    var element = document.getElementById(‘myID’);  
    element.onclick = function() {  
        alert(“Element clicked”);  
    }  
};  
attachEvents(); 

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

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

function attachEvents() {  
    var element = document.getElementById(‘myID’);  
    element.onclick = function() {  
        //Remove element, so function can be collected by GC  
        delete element;  
        alert(“Element clicked”);  
    }  
};  
attachEvents();