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…In”语法可能更慢，因为索引是字符串，而不是整数。

var a = ["a"]
for (var i in a)
    alert(typeof i)  // 'string'
for (var i = 0; i < a.length; i++)
    alert(typeof i)  // 'number'

原因是一个构造：

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

孤立地说，在数组中使用for没有错。For-in迭代对象的属性名，在“现成”数组的情况下，财产对应于数组索引。（像length、toString等内置属性不包含在迭代中。）

然而，如果您的代码（或您正在使用的框架）将自定义财产添加到数组或数组原型中，那么这些财产将包含在迭代中，这可能不是您想要的。

一些JS框架，如Prototype修改了Array原型。像JQuery这样的其他框架则不然，因此使用JQuery，您可以在中安全地使用。

如果你有疑问，你可能不应该使用。

迭代数组的另一种方法是使用for循环：

for (var ix=0;ix<arr.length;ix++) alert(ix);

然而，这有一个不同的问题。问题是JavaScript数组可能有“洞”。如果您将arr定义为：

var arr = ["hello"];
arr[100] = "goodbye";

然后数组有两个项目，但长度为101。在中使用for将产生两个索引，而for循环将产生101个索引，其中99的值为undefined。

截至2016年（ES6），我们可能会使用for…for数组迭代，正如John Slegers已经注意到的那样。

我只想添加一个简单的演示代码，以使事情更清楚：

Array.prototype.foo = 1;
var arr = [];
arr[5] = "xyz";

console.log("for...of:");
var count = 0;
for (var item of arr) {
    console.log(count + ":", item);
    count++;
    }

console.log("for...in:");
count = 0;
for (var item in arr) {
    console.log(count + ":", item);
    count++;
    }

控制台显示：

for...of:

0: undefined
1: undefined
2: undefined
3: undefined
4: undefined
5: xyz

for...in:

0: 5
1: foo

换句话说：

对于的计数从0到5，并且还忽略Array.prototype.foo。它显示数组值。对于中只列出了5个，忽略了未定义的数组索引，但添加了foo。它显示数组属性名称。

简短的回答：这不值得。

更长的答案：即使不需要顺序元素顺序和最佳性能，这也不值得。

长篇大论：这不值得。。。

使用for（数组中的var属性）将导致数组作为对象迭代，遍历对象原型链，最终执行速度比基于索引的for循环慢。for（…in…）不能保证按顺序返回对象财产，这可能是意料之中的。使用hasOwnProperty（）和！用于筛选对象财产的isNaN（）检查是一项额外的开销，导致它的执行速度更慢，并否定了首先使用它的主要原因，即因为它的格式更简洁。

出于这些原因，甚至不存在性能和便利性之间可接受的权衡。除非目的是将数组作为对象处理，并对数组的对象财产执行操作，否则实际上没有任何好处。