遍历JavaScript Map并修改其值的正确方式是什么？

先看你给出的两个选项的问题：

• 选项1的问题：回调参数顺序完全搞反了！Map.prototype.forEach的回调参数是(value, key, 当前Map)，你写的(key, value)把键值弄反了，导致实际执行时是把原Map的值当成键去设置，完全达不到更新原Map的目的。就算参数顺序改对，也要注意遍历原Map时修改值的逻辑是否符合需求。

• 选项2的问题：这是新建了一个Mapmap2，原有的map根本没被修改，不符合你要“值已更新的同一份Map”的要求，相当于复制了个新的出来用，原Map还是原样。

正确的做法（选项3） #

要更新原Map，有两种靠谱写法：

1. 修正参数顺序的forEach写法：

map.forEach((value, key) => {
  map.set(key, 'newValue');
});

这里拿到正确的key后，直接调用map.set修改原Map对应键的值，Map在遍历过程中修改现有键的值是安全的——因为遍历是按插入顺序进行的，修改值不会改变Map的键集合，所以遍历会完整执行，不会出现遗漏或异常。

2. 更直观的for...of遍历写法：

for (const [key, value] of map) {
  map.set(key, 'newValue');
}

通过解构Map的entries（每个entry是[key, value]数组），直接拿到键和值，再更新原Map，逻辑更清晰。

为什么要这么做？ #

如果你需要的是同一份Map（不是新的副本），就必须直接在原Map上修改键对应的值。Map的遍历机制允许这种操作：遍历过程中修改现有键的值，不会影响遍历的顺序和次数，因为没有新增或删除键，只是更新了值而已。

内容的提问来源于stack exchange，提问作者Platon Efimov