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的属性，对于每个属性，我使用ternary运算符设置其他对象属性的true / false。

阅读我看到的一些文档：

JSHint和ESLint在for，while或中遇到函数表达式声明正文。

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

Answer 1

此错误是否合法？

是的，您在循环中声明了一个函数。最重要的是，bind可能相当昂贵，因为它必须每次创建一个新的词法范围并返回一个新的＆＃34;附加到该范围的功能。

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

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

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]);

如果可以的话，避免for ... in循环通常会让你的生活变得更好，所以你也可能想要重构它：

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

你或许可以进一步提高。

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

1 个答案: