JavaScript：这里哪种方法更好…… forEach 还是 reduce？

Question

对于一个接受数字数组并在包含至少一个偶数时返回 true（否则为 false）的函数，我有以下 2 个解决方案。

解决方案 1（通过 forEach）：

var hasEven = function (collection) {
    var isEven = false;
    collection.forEach(function (entry) {
        if (entry % 2 === 0) {
            isEven = true;
            return;
        }
    });
    return isEven;
}

解决方案 2（通过 reduce）：

var hasEven = function (collection) {
    return collection.reduce(function (result, entry) {
        return (entry % 2 === 0) ? true : result; 
    }, false);
}

第一个解决方案允许我在找到偶数后立即返回 true。另一方面，第二种解决方案提供了最短的代码。哪个更好？

ps - 是的，有更好的解决方案...比如使用 array.some(...)。但我对比较这里提供的解决方案特别感兴趣。是的，现在解决方案正在正常工作。 :)

Answer 1

在您编辑的问题中，您说过：

我对比较这里提供的解决方案特别感兴趣

用凿子或螺丝刀敲钉子更好吗？唯一正确的答案是：不要，用锤子。

比较两种使用错误工具完成工作的解决方案最终是一个见仁见智的问题。你喜欢简单吗？使用forEach。简洁？使用reduce。我会在代码审查中将其中任何一个标记为次优选择，并且不会认为两者都比另一个次优。

---

我原来的答案：

在这种情况下，两者都不是。 Array#some (MDN | spec) 是确定数组是否至少有一个符合条件的条目的合适选择：

var hasEven = function(collection) {
    return collection.some(function(entry) {
        return entry % 2 === 0;
    });
};

Array#some 在回调返回真值后立即停止循环。 some 的返回值是 true 如果回调返回一个真实值，或者 false 如果它没有（所有元素都经过测试并且没有匹配）。

或者使用 ES2015：

// ES2015 (aka ES6) only
let hasEven = function(collection) {
    return collection.some(entry => entry % 2 === 0);
};

或者我们可以变得非常简洁（可能以牺牲可读性为代价，但当我们都真正习惯了箭头函数时可能不会）：

// ES2015 (aka ES6) only
let hasEven = collection => collection.some(entry => entry % 2 === 0);

---

（本节引用the code in your original question。）

我应该注意，您的示例实现都不起作用，forEach 和 reduce 都不起作用。 forEach 总是返回false，因为回调中的return true 只是从回调中返回，根本没有任何作用。 reduce 将 A) 无法测试数组中的第一个条目； B）只告诉你数组中的 last 条目是否是偶数，而不是前面的任何一个是偶数； C) 如果在只有一个条目的数组上调用，则返回第一个条目的 值，而不是 true 或 false。

正确的forEach 看起来像这样（但会不必要地循环）：

var hasEven = function(collection) {
    var result = false;
    collection.forEach(function(entry) {
        result = result || entry % 2 === 0;
    });
    return result;
};

正确的reduce 版本如下所示（但会不必要地循环）：

var hasEven = function(collection) {
    return collection.reduce(function(result, current) {
        return result || current % 2 === 0;
    }, false);
};

---

请注意，在上面，我使用了

return entry % 2 === 0;

而不是

 return (entry % 2 === 0) ? true : false;

=== 的结果已经是一个布尔值。