javascript - 即使 .forEach 在某些情况下看起来更符合逻辑，我也可以始终使用 .map 吗？

Question

我发现 .map() 和 .forEach() 在大多数情况下工作相似，尽管我知道 .map() 更有用适合根据转换输入返回一些结果，.forEach() 只是迭代项目并对每个项目应用一些操作。

我知道它非常适合这种情况:

var customersList = dataItems.map(function (item) {
    return item.isCustomer
})

但我发现将 .forEach() 从我的脑海中抛出并在所有代码中使用 .map() 更容易。在这种情况下使用 .map() 在概念上正确吗？

domElements.map(function (el) {
    if (el.enabled) {
        el.show()
    } else {
        el.hide()
    }
})

Answer 1

你可以。但你不应该这样做。

这对性能影响很小，但更重要的是，它也是维护负担。如果我必须来维护你的代码，我会假设出了什么问题，或者需要花更长的时间查看那段代码来判断其意图。为什么map在那里？它应该返回一些东西吗？是不是我理解的不对？

这是一个比性能更重要的问题。