在JavaScript中用forEach+push创建输出数组是否为反模式？用Map更优吗？

这种forEach结合push创建新数组的写法不算严格的反模式，但对比map方法，确实有明显的不足，后者在这类场景下更为合适，原因如下：

• 语义更明确：map方法的设计初衷就是对数组的每个元素做转换，生成新数组，看到map就能立刻明白代码的核心意图是“数组转换”；而forEach是通用遍历工具，配合外部数组的push操作，需要额外理解代码是在构建新数组，语义传递不够直接。
• 代码更简洁：用map实现示例需求只需一行代码，无需提前声明空数组：

  const result = arr.map(element => element.name)
  

  对比原写法，减少了冗余的变量初始化和push调用，代码更紧凑易读。
• 符合纯函数风格：map是纯函数，不会修改外部变量，输入固定时输出也固定，更利于测试和调试；而forEach配合push会修改外部的result数组，属于带有副作用的写法，在复杂场景下可能增加维护成本。

当然，如果遍历过程中需要执行额外的复杂逻辑（比如条件判断后才添加元素，或同时完成其他非转换类操作），forEach的灵活性会更有优势，但如果只是单纯从原数组提取值生成新数组，map无疑是更优的选择。

内容的提问来源于stack exchange，提问作者Yassin Mo