ES6：在 Set/Map 迭代期间从 Set/Map 中删除元素是否危险？

Question

new Set() 的安全代码可能如下所示：

let items = [];
for (let item of set)
  if (isBad(item))
    items.push(item);
for (let item of items)
  set.delete(item)

我可以将代码简化为：

for (let item of set)
  if (isBad(item))
    set.delete(item);

new Map() 的安全代码可能如下所示：

let keys = [];
for (let [key, val] of map)
  if (isBadKey(key) || isBadValue(val))
    keys.push(key);
for (let key of keys)
  map.delete(key)

我可以将代码简化为：

for (let [key, val] of map)
  if (isBadKey(key) || isBadValue(val))
    map.delete(key)

Answer 1

是的，您可以简化为，这是完全安全的。

• Set 和 Map 始终按插入顺序迭代
• 删除项目不会影响任何迭代器的位置 - 您可以看到集合的形状没有改变，只是被清空。
• 所以：已删除且尚未迭代的元素将不会被迭代
• 已经迭代并被删除的元素（如您的情况）不会影响其他迭代/查找。
• 在迭代期间添加的元素（并且还不是集合的一部分）将始终被迭代

从最后一点可以看出，唯一危险的事情就是

const s = new Set([1]);
for (let x of s) {
    s.delete(x);
    s.add(1);
}

但不是因为未定义的行为或内存积累，而是因为无限循环。

Answer 2

我会说是的，它是安全的。当您在后台使用for ... of 遍历Set/Map 时，循环将通过@@iterator。而Iterator 仅与.next() 一起使用：所以没有索引，也不管当前位置之前是什么。只有下一个元素很重要。

因此，除非您删除当前迭代器位置“前面”的元素 - 这样做是安全的。