JsHint（W083）：不要在循环中创建函数。-使用[]. foreach（）；

Question

我得到这个错误：

JsHint（W083）：不要在循环中生成函数。

使用以下代码时：

for (var prop in cmd.properties) {
  Object.keys(meta[prop].data).forEach(function (rule) {
    rules.data[rule] = meta[prop].data[rule] ? true : false;
  }.bind(this));
}

基本上，我是在一个对象的属性meta[prop].data中循环，对于每个属性，我使用三元操作符设置其他对象属性的真/假。

阅读一些文档，我看到：

JSHint和ESLint在for、while或do语句体中遇到函数表达式。

• 这个错误合法吗？
• 如果是，如何更好地重写这些行？
• 如果否，如何使用JsHint禁用此特定错误验证

Answer 1

这个错误合法吗？

是的，您正在循环中声明一个函数。最重要的是，bind可能会非常昂贵，因为它每次都必须创建一个新的词法作用域，并返回附加到该作用域的“新”函数。

如果是，如何更好地重写这些台词？

如果可以的话，在循环之前声明函数，或者绑定一次，或者使用简单的闭包来避免显式绑定调用：

var scope = this;
var ruleFunc = function (rule) {
  rules.data[rule] = meta[prop].data[rule] ? true : false;
}

for (var prop in cmd.properties) {
  Object.keys(meta[prop].data).forEach(ruleFunc);
}

不过，我没有看到您在函数中使用this的地方，因此您可以完全删除它：

var ruleFunc = function (rule) {
  rules.data[rule] = meta[prop].data[rule] ? true : false;
}

for (var prop in cmd.properties) {
  Object.keys(meta[prop].data).forEach(ruleFunc);
}

这两者都需要重构代码，因为您使用闭包从循环中获取prop变量。您可以使用bind来解决这个问题，并使用性能命中：

var ruleFunc = function (prop, rule) {
  rules.data[rule] = meta[prop].data[rule] ? true : false;
}

for (var prop in cmd.properties) {
  Object.keys(meta[prop].data).forEach(ruleFunc.bind(this, prop));
}

您还使用条件返回true/false，这是一种常见的气味。您通常希望将其转换为布尔值，使用 是惯用的JS方式：

rules.data[rule] = !!(meta[prop].data[rule]);

如果可以，请避免使用。。。在中，循环通常会让您的生活变得更好，因此您可能还需要重构它：

Object.keys(cmd.properties).forEach(function (prop) {
  Object.keys(meta[prop].data).forEach(function (rule) {
    rules.data[rule] = !!(meta[prop].data);
  });
});

你也许可以进一步改进。