为什么 If 语句在此循环中被视为函数？

Question

我一直在构建一个经常使用迭代的 React 应用程序。我正在使用 JSLint 并收到烦人的警告：

不要在循环中创建函数

在以下循环中：

if(currentSearch[i].users.length > 0) {
    var matched = false;

    //I hate how JSLint just can't handle setting a variable to true within a function
    //this is loop where I get the warning 
    currentSearch[i].users.some(childUser => {
        if(childUser.id === user.id) {
            return matched = true;
        }
    })
    //^^^^^ Warning

    if(!matched) {
        var alert = new AlertObject(user,currentSearch[i],true,false);
        alerts.push(alert);
    }
}

我不认为我在循环中设置了一个函数？我正在使用 array.some 函数，如果我返回 true，它将打破循环，这就是我所做的。我返回一个在循环外声明的变量为真。这让我脱离了循环，并允许我在下面做逻辑。

我还应该指出，这也完全在一个循环中，因为我们正在迭代当前的搜索用户。我没有运行时或编译错误，这段代码运行良好，但也许我正在为将来的灾难做好准备。

任何想法为什么我会收到此错误？如果我遗漏了一些最佳实践？

Answer 1

由于在第一行中您引用了currentSearch[i]，因为[i] 我假设您在此处粘贴的整个代码块都在某种循环中，可能是for。

然后，您正在为Array.some 回调创建一个函数，它会触发错误。

一种解决方案是将回调声明移到父循环之外，但由于您使用的是作用域中的变量，因此需要进行一些重构。

---

可能的解决方案

您可以在父循环之外（您在此处提供的代码之外的那个）声明一个函数来检查子用户。

//Please provide a better name for the function according to the context.
const checkChildUser = function (childUser) {
    return this.id === childUser.id;
};

然后将其传递给您正在使用的Array.some 函数：

currentSearch[i].users.some(checkChildUser, user);

Answer 2

我不熟悉 React，但这看起来像一个 ES6 箭头函数：

childUser => { ... }

相当于

function (childUser) { ... }

Answer 3

这里的问题是您创建了一个修改matched 变量的函数。这个变量是用var 声明的，所以它的作用域是整个函数，而不是单个循环迭代。这可能会导致令人惊讶的结果，因为在每次迭代中创建的函数实际上都会引用 same 变量。

只需使用some() 返回的值而不是在回调中更改matched（如Yury Tarabanko 所建议的那样）即可消除警告。

Answer 4

在您的 sn-p 中，Don't make functions within a loop 警告不是由 if statement 引起的，而是由以下匿名函数引起的：

childUser => {
 if(childUser.id === user.id) {
  return matched = true;
 }
}

由于您已经说过整个代码都在一个循环中，因此每次迭代都会创建该匿名函数的一个新实例。这会影响性能。

Answer 5

好吧，您在 .some() 中提供了一个函数作为参数，这就是触发警告的原因。

ESLint 警告它的原因

Writing functions within loops tends to result in errors due to the way the function creates a closure around the loop - source

你可以这样做

function compareId(childUser) {
  if (childUser.id === user.id) {
    return true;
  }
}

if (currentSearch[i].users.length > 0) {
  var matched = currentSearch[i].users.some(compareId);

  if (!matched) {
    var alert = new AlertObject(user, currentSearch[i], true, false);
    alerts.push(alert);
  }
}