javascript - 我应该避免在 reduce 中使用 object spread 吗？

Question

给定以下示例:

// option 1
items.reduce((values, item) => ({
     ...values,
     [item.id]: item.name
}), {})


// option 2
items.reduce((values, item) => {
    values[item.id] = item.name;
    return values;
}, {});

在这种情况下，是否有赞成或反对使用对象传播语法的最佳实践？

Answer 1

出于性能原因，选项 2 显然更可取:

• 选项 1 在 O(n²) 时间内运行，因为扩展语法在每次迭代时复制 O(n) 个属性。选项 2 在 O(n) 时间内运行。
• 选项 1 创建 O(n²) 垃圾，因为它在每次迭代时创建大小为 O(n) 的垃圾对象。选项 2 不会产生垃圾。

就是说，在大多数情况下，您应该只用普通的旧式 for 循环来编写:

let result = {};
for(let item of items) {
    result[item.id] = item.name;
}

使用for 循环还不错，而且代码比问题中的两个选项更具可读性。选项 2 可能看起来更像是函数式编程风格，但如果您使用变异来实现您想要的结果，那么您并不是真正在进行函数式编程。

参见 this article更深入地讨论为什么选项 1 是反模式。