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
  }
}

您应该只在属性列表上使用for（var x in y），而不是在对象上（如上所述）。

原因是一个构造：

变量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.食品*/

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上的循环中：”）；

我认为我没有什么要补充的，比如Triptych的答案或CMS关于为什么使用for。。。在某些情况下应避免。

然而，我想补充一点，在现代浏览器中，有一种替代。。。在这种情况下。。。无法使用中的。另一种选择是。。。第页，共页：

for (var item of items) {
    console.log(item);
}

注：

遗憾的是，没有版本的Internet Explorer支持。。。（Edge 12+确实如此），因此您必须等待一段时间，直到可以在客户端生产代码中使用它。然而，在服务器端JS代码中使用它应该是安全的（如果您使用Node.JS）。

这不一定是坏的（根据您正在做的事情），但在数组的情况下，如果在Array.prototype中添加了一些东西，那么您将得到奇怪的结果。您希望此循环运行三次：

var arr = ['a','b','c'];
for (var key in arr) { ... }

如果一个名为helpfulUtilityMethod的函数被添加到Array的原型中，那么循环将运行四次：键将是0、1、2和helpfulUtilityMethod。如果你只需要整数，噢。