56

考虑同一循环迭代的两个版本：

for (var i = 0; i < nodes.length; i++) {
    ...
}

和

var len = nodes.length;
for (var i = 0; i < len; i++) {
    ...
}

后一个版本比前一个版本快吗？

4

6 回答 6

37

公认的答案是不正确的，因为任何体面的引擎都应该能够使用如此简单的循环体将属性负载提升到循环之外。

看到这个 jsperf - 至少在 V8中，有趣的是看到实际将它存储在变量中如何改变寄存器分配 - 在使用sum变量的代码中，变量存储在堆栈中，而使用array.length-in-a-loop-code它存储在寄存器中。我认为 SpiderMonkey 和 Opera 也发生了类似的事情。

根据作者的说法，JSPerf 有70% 的时间被错误地使用。此处所有答案中给出的这些损坏的 jsperfs 给出了误导性的结果，人们从中得出错误的结论。

一些危险信号是将代码而不是函数放入测试用例中，不测试结果的正确性或使用某种消除死代码的机制，在设置或测试用例中定义函数而不是全局。为了保持一致性，您将需要温暖-也可以在任何基准测试之前启动测试功能，这样编译就不会发生在定时部分。

于 2013-08-01T13:10:32.897 回答
35

更新：2015 年 16 月 12 日

由于这个答案似乎仍然有很多观点，我想随着浏览器和 JS 引擎的不断发展重新审视这个问题。

我没有使用 JSPerf，而是将一些代码放在一起，使用原始问题中提到的两种方法循环遍历数组。我已将代码放入函数中以分解功能，就像希望在现实世界的应用程序中那样：

function getTestArray(numEntries) {
  var testArray = [];
  for (var i = 0; i < numEntries; i++) {
    testArray.push(Math.random());
  }
  return testArray;
}

function testInVariable(testArray) {
  for (var i = 0; i < testArray.length; i++) {
    doSomethingAwesome(testArray[i]);
  }
}

function testInLoop(testArray) {
  var len = testArray.length;
  for (var i = 0; i < len; i++) {
    doSomethingAwesome(testArray[i]);
  }
}

function doSomethingAwesome(i) {
  return i + 2;
}

function runAndAverageTest(testToRun, testArray, numTimesToRun) {
  var totalTime = 0;
  for (var i = 0; i < numTimesToRun; i++) {
    var start = new Date();
    testToRun(testArray);
    var end = new Date();
    totalTime += (end - start);
  }
  return totalTime / numTimesToRun;
}

function runTests() {
  var smallTestArray = getTestArray(10000);
  var largeTestArray = getTestArray(10000000);

  var smallTestInLoop = runAndAverageTest(testInLoop, smallTestArray, 5);
  var largeTestInLoop = runAndAverageTest(testInLoop, largeTestArray, 5);
  var smallTestVariable = runAndAverageTest(testInVariable, smallTestArray, 5);
  var largeTestVariable = runAndAverageTest(testInVariable, largeTestArray, 5);

  console.log("Length in for statement (small array): " + smallTestInLoop + "ms");
  console.log("Length in for statement (large array): " + largeTestInLoop + "ms");
  console.log("Length in variable (small array): " + smallTestVariable + "ms");
  console.log("Length in variable (large array): " + largeTestVariable + "ms");
}

console.log("Iteration 1");
runTests();
console.log("Iteration 2");
runTests();
console.log("Iteration 3");
runTests();

为了获得尽可能公平的测试，每个测试运行 5 次并取平均值。我还运行了整个测试，包括生成数组 3 次。在我的机器上对 Chrome 进行的测试表明，使用每种方法所花费的时间几乎相同。

重要的是要记住，这个示例有点像玩具示例，事实上，大多数从应用程序上下文中提取的示例都可能产生不可靠的信息，因为您的代码正在执行的其他操作可能会直接或间接地影响性能。

底线

确定最适合您的应用程序的最佳方法是自己测试它！JS 引擎、浏览器技术和 CPU 技术在不断发展，因此您必须始终在应用程序的上下文中自行测试性能。同样值得问问自己是否有性能问题，如果没有，那么花时间进行用户无法察觉的微优化可能会更好地用于修复错误和添加功能，从而让用户更快乐:)。

原答案：

后者会稍微快一些。该length属性不会遍历数组来检查元素的数量，但每次在数组上调用它时，都必须取消引用该数组。通过将长度存储在变量中，每次循环迭代都不需要数组取消引用。

如果您对在 javascript 中循环遍历数组的不同方式的性能感兴趣，请查看此jsperf

于 2013-08-01T08:59:19.190 回答
4

根据w3schools “减少循环中的活动”，以下被认为是错误代码：

for (i = 0; i < arr.length; i++) {

以下被认为是好的代码：

var arrLength = arr.length;
for (i = 0; i < arrLength; i++) {

由于访问 DOM 很慢，因此编写了以下代码来测试该理论：

<!doctype html>

<html lang="en">
	<head>
		<meta charset="utf-8">
		<title>my test scripts</title>
	</head>
	
	<body>
		<button onclick="initArray()">Init Large Array</button>
		<button onclick="iterateArraySlowly()">Iterate Large Array Slowly</button>
		<button onclick="iterateArrayQuickly()">Iterate Large Array Quickly</button>
		
		<p id="slow">Slow Time: </p>
		<p id="fast">Fast Time: </p>
		<p id="access"></p>


	
	<script>
	var myArray = [];
			
		function initArray(){
			var length = 1e6;
			var i;
			for(i = 0; i < length; i++) {
				myArray[i] = i;
			}
			console.log("array size: " + myArray.length);
		}
		
		function iterateArraySlowly() {
			var t0 = new Date().getTime();
			var slowText = "Slow Time: "
			var i, t;
			var elm = document.getElementById("slow");
			for (i = 0; i < myArray.length; i++) {
				document.getElementById("access").innerHTML = "Value: " + i;
			}
			t = new Date().getTime() - t0;
			elm.innerHTML = slowText + t + "ms";
		}
		
		function iterateArrayQuickly() {
			var t0 = new Date().getTime();
			var fastText = "Fast Time: "
			var i, t;
			var elm = document.getElementById("fast");
			var length = myArray.length;
			for (i = 0; i < length; i++) {
				document.getElementById("access").innerHTML = "Value: " + i;
			}
			t = new Date().getTime() - t0;
			elm.innerHTML = fastText + t + "ms";
		
		}
	</script>
	</body>
</html>

有趣的是，首先执行的迭代似乎总是胜过另一个。但是在每个被执行几次之后，被认为是“坏代码”的大部分时间似乎都赢了。也许比我更聪明的人可以解释原因。但就目前而言，语法方面我坚持使用对我来说更清晰的内容：

for (i = 0; i < arr.length; i++) {
于 2015-06-24T22:27:40.383 回答
3

如果nodes是，DOM nodeList那么第二个循环会快得多，因为在第一个循环中，您在每次迭代时查找 DOM（非常昂贵）。jsperf

于 2013-08-01T09:17:32.270 回答
1

在我使用过的任何基准测试中，这一直是性能最高的。

for (i = 0, val; val = nodes[i]; i++) {
    doSomethingAwesome(val);
}
于 2019-01-01T18:41:11.247 回答
0

我相信nodes.length已经定义并且不会在每次使用时重新计算。所以第一个例子会更快，因为它定义了一个更少的变量。尽管差异将不明显。

于 2013-08-01T08:51:39.380 回答