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

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

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 风格或以其他方式更改我的代码？

这里有两个问题，一个是 JSHint 警告您的问题，另一个是更根本的问题。

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

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

我说“理论上”是因为尽管规范要求新功能object每次都会创建，这并不意味着引擎不能重用底层执行函数的属性，这并不意味着如果您没有为其分配任何其他属性或保留对它的引用，引擎就无法重用同一函数对象。我向 V8 的人询问了此事 https://stackoverflow.com/a/10160351/157247（V8 是 Chrome 中的 JavaScript 引擎），他们表示 Chrome 将“......在大多数情况下......”重用源代码中同一点创建的不同函数对象的底层函数实现，并且他们会“期望”大多数其他引擎也会这样做。

因此，在这种特殊情况下，JSHint 可能有点过头了。但这通常是一个有用的警告，特别是当您在循环内创建的函数引用其内容在循环期间发生变化的变量时，这是人们犯的经典闭包错误。

但更根本的是，person is a String（这是一个属性的名称people), and String没有forEach。你自找的：

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

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