数组迭代要优先使用for循环而不是for...in循环

时间:2022-12-30 12:34:47

下面这段代码中的mean的输出值是多少?

var scores = [98,74,85,77,93,100,89];
var total = 0;
for(var score in scores){
total+=score;
}
var mean = total/scores/scores.length;
mean;//?
你有没有发现Bug?如果你认为答案是88,那么你理解了这段程序的意图,但这并不是实际的结果。程序混淆了数字数组key和value这是极易犯下的错误。for...in循环始终枚举所有key。下一个看似合理的猜测是(0+1+...6)/7=21,但是这也不正确。请记住即使是数组的索引属性,对象属性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;
mean;//88
该方法确保当你需要证书索引和数组元素值就能获取它们,并且绝不会混淆它们或引发意想不到的字符串强制转换。此外,它还确保以正确的顺序迭代数组,并且不会意外地包括存储在数组对象或其原型链中的非整数属性。
请注意上面for循环中数组长度变量n的使用。如果循环体不修改该数组,那么在每次迭代中,循环都会简单的重新计算数组的长度。

for(var i=0;i<scores.length;i++){
...
}
另外,在循环的一开始就计算出数组的长度还有几个小的好处。首先,即使是优化的JavaScript编译器可能有时也很难保证避免重新计算scores.length是安全的。不过更重要的是,它给阅读该代码的程序员传递了一个信息:循环的终止条件是假单且确定的。

总结:迭代数组的索引属性应当总是使用for循环而不是for..in循环。
考虑在循环之前将数组的长度村春在一个局部变量中以避免重新计算数组长度。