JavaScript的9个陷阱及评点

for...in...会遍历所有的东西

有一段这样的代码：

var arr = [5,10,15]

var total = 1;

for ( var x in arr) {

    total = total * arr[x];

}

运行得好好的，不是吗？但是有一天它不干了，给我返回的值变成了NaN, 晕。我只不过引入了一个库而已啊。原来是这个库改写了Array的prototype，这样，我们的arr平白无过多出了一个属性（方法），而for...in...会把它给遍历出来。所以这样做才是比较安全的：

for ( var x = 0; x < arr.length; x  ) {

    total = total * arr[x];

}

其实，这也是污染基本类的prototype会带来危害的一个例证。

事件处理器的陷阱

这其实只会存在使用作为对象属性的事件处理器才会存在的问题。比如window.onclick = MyOnClickMethod这样的代码，这会复写掉之前的window.onclick事件，还可能导致IE的内容泄露（sucks again）。在IE还没有支持DOM 2的事件注册之前，作者建议使用库来解决问题，比如使用YUI:

YAHOO.util.Event.addListener(window, "click", MyOnClickMethod);

这应该也属于常识问题，但新手可能轻易犯错。

Focus Pocus

新建一个input文本元素，然后把焦点挪到它上面，按理说，这样的代码应该很自然：

var newInput = document.createElement("input");

document.body.appendChild(newInput);

newInput.focus();

newInput.select();

但是IE会报错（sucks again and again）。理由可能是当你执行fouce()的时候，元素尚未可用。因此，我们可以延迟执行：

var newInput = document.createElement("input");

newInput.id = "TheNewInput";

document.body.appendChild(newInput);

setTimeout(function(){ //这里我使用闭包改写过，若有爱好可以对比原文

	document.getElementById('TheNewInput').focus();

	document.getElementById('TheNewInput').select();}, 10);