在条件表达式中不必要地使用布尔文字答案

【问题标题】：Unnecessary use of Boolean literals in conditional expression在条件表达式中不必要地使用布尔文字
【发布时间】：2018-10-02 22:09:05
【问题描述】：

我有一个函数可以检查是否在数组中找到了一个值。我想返回一个真或假。当前代码有效，但抛出 js-standerd/es-lint 错误“在条件表达式中不必要地使用布尔文字” 我在这里搜索了大量这些错误消息，但似乎无法理解它。对我来说，这就是说'如果找到该值，则返回 true，否则返回 false'

let found = value.find(val => {
  return val === item
})
return found ? true : false

我试过了

  return value.find(val => {
     return val === item
   }) || false

有效但不返回布尔值，如果找到，则返回项目。

我知道我可以通过多种方式完成这项工作，但我只是想弄清楚我的代码是坏还是不正确，或者 es-lint 是否有错误地标记它。

【问题讨论】：

【解决方案1】：

linter 抱怨这个：

return found ? true : false

应该读作“如果found 是真实的，则返回true，否则返回false”。这种结构被称为“三元”运算符，并且从 C 的早期开始就一直在使用，如果不是更早的话。 ? 运算符计算左侧的条件，如果条件计算结果为 true，则返回第一个参数，否则返回第二个参数，其中参数用冒号分隔。

您的代码的问题在于返回条件本身相当于返回布尔文字 true 或 false。因此，检查和文字是不必要的，可以删除。但是，因为这是 javascript，您可能希望在返回条件之前双重否定条件，以强制它成为布尔值。因此，结果如下所示：

return !!found

这更容易阅读，并且将来被错误实施或误解的可能性更小。

当然，这可以更进一步：

return !!value.find(val => val === item)

这样，您甚至根本不需要在代码中引入符号found。此外，使用some() 会更好，但我认为您的问题更多是关于三元运算符而不是如何搜索列表。

【讨论】：

【解决方案2】：

return value.some(val => { return val === item; });

【讨论】：