看下面一段代码:
var scores=[98,74,85,77,93,100,89];
var total=0;
for(var score in scores){
total+=score;
}
var mean=total/scores.length;
alert(mean); //???
有没有发现Bug?如果你认为答案是88,那么你理解了这段程序的意图,但这并不是实际的结果。程序混淆了数字数组key和value这是极易犯下的错误。for...in循环始终枚举所有key。下一个看似合理的猜测是(0+1+2+3+4+5+6)/7=3,但是这也是不正确的。请记住即使是数组的索引属性,对象属性key始终是字符串。因此,“+=”操作符将执行字符串的连接操作,结果便得到一个意想不到的total值“00123456”。那最终的结果是什么?一个难以置信的mean值17636.571428571428。
迭代数组内容的正确方法是使用传统的for循环。
var scores=[98,74,85,77,93,100,89];
var total=0;
for(var i=0,n=scores.length;i<n;i++){
total+=scores[i];
}
var mean=total/scores.length;
alert(mean); //88
该方法能够确保当你需要整数索引和数组元素值时就能获取它们,并且绝不会混淆它们或引发意想不到的字符串强制转换。此外,它还确保以正确的顺序迭代数组,并且不会意外地包括存储在数组对象或其原型链中的非整数属性。
请注意上面for循环中数组长度变量n的使用。如果循环体不修改该数组,那么在每次迭代中,循环都会简单地重新计算数组的长度。
for(var i=0;i<scores.length;i++){ ... }
另外,在循环的一开始就计算出数组的长度还有几个小的好处。首先,即使是优化的JavaScript编译器可能有时也很难保证避免重新计算scores.length是安全的。不过更重要的是,它给阅读该代码的程序员传递了一个信息:循环的终止条件是简单且确定的。
总结:
1.迭代数组的索引属性应当总是使用for循环而不是for...in循环。
2.考虑在循环之前将数组的长度存储在一个局部变量中以避免重新计算数组长度。