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

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

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

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

但更根本的是，person 是一个String(它是people 中属性的名称)，而String code> 没有 forEach。你想要:

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

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