时间戳

首页 > 解决方案 > 警告 ESLint 规则下的意外未命名函数 func-names

javascript - 警告 ESLint 规则下的意外未命名函数 func-names

问题描述

我的eslint版本是 4.18.2，它会给出这样的警告：

意外的未命名函数
需要赋值或函数调用，但看到的是表达式

以这种方式定义函数时：

 const farmerIds = a.reduce((function (hash) {
  return function (prev, curr) {
    !hash[curr.farmerId] && (hash[curr.farmerId] = prev.push(curr));
    return prev;
  };
}(Object.create(null))), []);

标签： javascripteslintrules

解决方案

这是两个不同的问题……</p>

意外的未命名函数

正如康斯坦丁所指出的，这就是 ESLint 规则func-names。

如果您不想禁用此规则，您可以为您的函数使用名称，如下所示：

const farmerIds = a.reduce((function reducer(hash) {
  return function fn(prev, curr) {
    !hash[curr.farmerId] && (hash[curr.farmerId] = prev.push(curr));
    return prev;
  };
}(Object.create(null))), []);

或者，我个人建议使用箭头函数：

const farmerIds = a.reduce(
  (hash => {
    return (prev, curr) => {
      !hash[curr.farmerId] && (hash[curr.farmerId] = prev.push(curr));
      return prev;
    };
  })(Object.create(null)),
  []
);

期望一个赋值或函数调用，而是看到一个表达式

ESLint 正在抱怨这一行，这确实是一个表达式，而不是赋值或函数调用：

!hash[curr.farmerId] && (hash[curr.farmerId] = prev.push(curr));

您可以将其重写为if语句：

if (!hash[curr.farmerId]) {
  hash[curr.farmerId] = prev.push(curr);
}

修复两者

将上面的代码示例放在一起，这段代码应该在没有 ESLint 抱怨的情况下运行：

const farmerIds = a.reduce(
  (hash => (prev, curr) => {
    if (!hash[curr.farmerId]) {
      hash[curr.farmerId] = prev.push(curr);
    }
    return prev;
  })(Object.create(null)),
  []
);

请注意，我还删除了第一个箭头函数主体周围的花括号，这是箭头的一个很好的附加功能，可以使代码更简洁。

推荐阅读