ecmascript-6 - ESLint 警告 ES6 一致返回规则

Question

我收到 ESLint 警告:

预期在箭头函数结束时返回一个值艺术(一致返回)

errors.details.forEach((error) => {
  const errorExists = find(errObj, (item) => {  // <== ESLint warning
    if (item && item.field === error.path && item.location === location) {
      item.messages.push(error.message);
      item.types.push(error.type);
      return item;
    }
  });
  if (!errorExists) {
    errObj.push({
      field: error.path,
      location: error.location,
      messages: [error.message],
      types: [error.type]
    });
  }
});

但是，如果我插入返回

  const errorExists = find(errObj, (item) => {    // <== ESLint warning
    if (item && item.field === error.path && item.location === location) {
      item.messages.push(error.message);
      item.types.push(error.type);
      return item;
    }
    return;   // <== inserted return
  });

然后在这一行上不再有警告，但随后我在插入的返回中收到 2 个警告......

箭头函数期望返回值(一致返回)
不必要的返回语句(no-useless-return)
我不知道如何正确解决这个问题..
欢迎任何反馈

Answer 1

http://eslint.org/docs/rules/consistent-return说:

This rule requires return statements to either always or never specify values.

当您的 if 条件不满足时，箭头函数将终止，而不会遇到 return 语句，这违反了此规则。你的第二个版本违反了这个规则，因为你的第二个返回没有指定一个值，与第一个返回相反。第二个警告告诉您额外的 return 语句是多余的。
为了让 linter 满意，您可能应该考虑在不满足条件时从箭头函数正确返回什么。我不知道你的 find 函数到底做了什么，但如果它的行为类似于 Array.prototype.find 你可能希望在箭头函数的末尾返回 false。如果在这种情况下您需要返回 undefined，则同一页面中的此段落适用:

When Not To Use It

If you want to allow functions to have different return behavior depending on code branching, then it is safe to disable this rule.

编辑:我以前写过看看选项 treatUndefinedAsUnspecified ，但如果您只需要在其中一个分支中返回 undefined ，那么这两种设置似乎都无济于事。