Question

我有一个以数组作为值的对象。

people = {
    'steve':['foo','bar'],
    'joe':['baz','boo']
}

对于每个键，我想循环相应数组中的值。很简单：

for ( var person in people ) {
    person.forEach( function(item) {
      console.log(item)
    })
}

但JSHint抱怨道：

Don't make functions within a loop.

这真的是我的代码问题吗？我非常喜欢简短的ES5 for循环语法。我是否需要使用ES3样式或以其他方式更改我的代码？

Answer 1

有两个问题，一个是JSHint警告你的问题，另一个是更基本的问题。

JSHint警告你的是理论上，每次该循环运行时，会创建一个新函数。这会更好：

for ( var person in people ) {
    person.forEach(handlePerson);
}
function handlePerson(item) {
  console.log(item)
}

我说“理论上”因为虽然规范要求每次都创建一个新的函数 object ，但这并不意味着引擎无法重用底层的实现，并不意味着如果您没有为其分配任何其他属性或保留对它的引用，则引擎不能重用相同的函数对象。我asked the V8 guys about it（V8是Chrome中的JavaScript引擎），他们说Chrome将“...在大多数情况下......”重复使用在源中同一点创建的不同功能对象的底层函数实现代码，他们会“期望”大多数其他引擎会做同样的事情。

因此，在这种特殊情况下，JSHint可能会有点过分。但它经常是一个有用的警告，特别是如果你在循环中创建的函数引用了循环期间内容发生变化的变量，这是人们经典的闭包错误。

但更重要的是，person是String（它是people中属性的名称），String没有forEach。你想要：

for ( var person in people ) { people[person].forEach(handlePerson); } function handlePerson(item) { console.log(item) }

...例如，people[person]获取该密钥的数组。

Answer 2

除了其他评论者之外，如果您知道自己在做什么，可以使用JSHint选项loopfunc禁用此警告：

/*jshint loopfunc:true */

for ( var person in people ) {
  person.forEach( function(item) {
    console.log(item)
  })
}

您可以全局设置JSHint选项（如果您使用的是NPM模块），每个文件或每个功能。

Answer 3

你被允许在循环中使用forEach，但你不允许在循环中声明一个函数。

function looper (item) {
  console.log(item)
}

for ( var person in people ) {
    person.forEach(looper)
}

...否则你将为每次迭代重建相同的函数。

Answer 4

这不是forEach，而是它抱怨的匿名函数。

Answer 5

这是一个问题的原因是它每次执行此操作时都会在堆中的foreach调用内创建对匿名函数的新对象引用。如果你将函数分配给for循环之外的变量会更好，这样你就不会不必要地占用内存

JSHint不允许我在'for'循环中使用'forEach'

5 个答案: