以下是这种做法（通常）不好的原因：

对于in循环迭代它们自己的所有可枚举财产及其原型的可枚举财产。通常在数组迭代中，我们只需要迭代数组本身。即使您自己可能没有向数组中添加任何内容，但您的库或框架可能会添加一些内容。

例子：

Array.prototype.hithere=“此处”；var数组=[1，2，3]；for（let el in array）{//hithere属性也将被迭代控制台日志（el）；}

对于in循环不能保证特定的迭代顺序。尽管现在大多数现代浏览器中通常都能看到isorder，但仍然没有100%的保证。对于in循环忽略未定义的数组元素，即尚未分配的数组元素。

例子：：

常量arr=[]；arr[3]=“foo”；//将阵列大小调整为4arr[4]=未定义；//添加另一个未定义值的元素//遍历数组，for循环确实显示了未定义的元素for（设i=0；i<arr.length；i++）{控制台日志（arr[i]）；}console.log（'\n'）；//for in忽略未定义的元素for（let el in arr）{控制台日志（arr[el]）；}

for in语句本身并不是一个“坏习惯”，但是它可能被错误地使用，例如，用于迭代数组或类似数组的对象。

for-in语句的目的是枚举对象财产。该语句将出现在原型链中，也会枚举继承的财产，这有时是不需要的。

此外，规范并不能保证迭代的顺序，这意味着如果要“迭代”数组对象，那么使用此语句无法确保财产（数组索引）将按数字顺序访问。

例如，在JScript（IE<=8）中，即使在Array对象上，枚举的顺序也定义为创建了财产：

var array = [];
array[2] = 'c';
array[1] = 'b';
array[0] = 'a';

for (var p in array) {
  //... p will be "2", "1" and "0" on IE
}

此外，谈到继承的财产，例如，如果您扩展Array.prototype对象（像MooTools这样的一些库），那么财产也会被枚举：

Array.prototype.last = function () { return this[this.length-1]; };

for (var p in []) { // an empty array
  // last will be enumerated
}

正如我之前所说的，迭代数组或类似数组的对象，最好的方法是使用顺序循环，例如普通的旧for/while循环。

如果只想枚举对象自己的财产（未继承的属性），可以使用hasOwnProperty方法：

for (var prop in obj) {
  if (obj.hasOwnProperty(prop)) {
    // prop is not inherited
  }
}

有些人甚至建议直接从Object.prototype调用该方法，以避免在有人向我们的对象添加名为hasOwnProperty的属性时出现问题：

for (var prop in obj) {
  if (Object.prototype.hasOwnProperty.call(obj, prop)) {
    // prop is not inherited
  }
}

因为它通过对象字段而不是索引枚举。你可以用索引“length”获得值，我怀疑你是否想要这个。

TL&DR：在数组中使用for in循环并不是坏事，事实上恰恰相反。

我认为如果在数组中正确使用，for in循环是JS的瑰宝。你应该完全控制你的软件，并知道你在做什么。让我们看看上面提到的缺点，并逐一反驳它们。

它还遍历继承的财产：首先，对Array.prototype的任何扩展都应该使用Object.defineProperty（）完成，并且它们的可枚举描述符应该设置为false。任何没有这样做的库都不应该被使用。添加到继承链中的财产稍后会被计数：当按Object.setPrototypeOf或按类扩展进行数组子分类时。您应该再次使用Object.defineProperty（），它默认将可写、可枚举和可配置的属性描述符设置为false。让我们在这里看到一个数组子类示例。。。

函数堆栈（…a）{var stack=新数组（…a）；Object.setPrototypeOf（堆栈，stack.prototype）；返回堆栈；}Stack.prototype=对象.create（Array.prototype）；//现在堆栈可以完全访问数组方法。Object.defineProperty（Stack.prototype，“constructor”，｛value:Stack｝）；//现在Stack是一个合适的构造函数Object.defineProperty（Stack.prototype，“peak”，｛value:function（）｛//将Stack“only”方法添加到Stack.prototype。返回this[this.length-1]；}});var s=新堆栈（1,2,3,4,1）；console.log（s.peak（））；s[s.length]=7；console.log（“长度：”，s.length）；s.push（42）；console.log（JSON.stringify）；console.log（“长度：”，s.length）；for（var i in s）console.log（s[i]）；

所以你看。。因为您关心代码，所以in循环现在是安全的。

for in循环很慢：见鬼，没有。如果你在不时需要的稀疏数组上循环，这是迄今为止最快的迭代方法。这是人们应该知道的最重要的表演技巧之一。让我们看一个例子。我们将在稀疏数组上循环。

变量a=[]；a[0]=“零”；a[10000000]=“1000万”；console.time（“用于数组a上的循环：”）；对于（var i=0；i<a.length；i++）a[i]&&console.log（a[i]）；console.timeEnd（“用于数组a上的循环：”）；console.time（“用于数组a上的循环：”）；对于（a中的var i）a[i]&&console.log（a[i]）；console.timeEnd（“用于数组a上的循环中：”）；

原因是一个构造：

变量a=[]；//创建新的空数组。a[5]=5；//调整数组大小的完全合法的JavaScript。对于（var i=0；i<a.length；i++）{//按照每个人的预期，在从0到5的数字索引上重复。控制台日志（a[i]）；}/*将显示：未定义未定义未定义未定义未定义5.*/

有时可能完全不同：

变量a=[]；a[5]=5；for（a中的var x）{//仅显示显式设置的索引“5”，并忽略0-4console.log（x）；}/*将显示：5.*/

还要考虑JavaScript库可能会这样做，这会影响您创建的任何数组：

//在JavaScript库的深处。。。Array.prototype.foo=1；//现在你不知道下面的代码会做什么。变量a=[1，2，3，4，5]；for（a中的var x）{//现在foo是每个数组的一部分//将在此处显示为“x”值。console.log（x）；}/*将显示：01.2.3.4.食品*/

虽然这个问题没有特别提到，但我想补充一点，有一个很好的理由永远不要使用。。。在中使用NodeList（就像从querySelectorAll调用中获得的那样，因为它根本看不到返回的元素，而只在NodeListneneneba属性上迭代。

在单个结果的情况下，我得到：

var nodes = document.querySelectorAll(selector);
nodes
▶ NodeList [a._19eb]
for (node in nodes) {console.log(node)};
VM505:1 0
VM505:1 length
VM505:1 item
VM505:1 entries
VM505:1 forEach
VM505:1 keys
VM505:1 values

这解释了为什么我的for（节点中的节点）node.href=newLink；正在失败。