如何安全地从数组中批量删除满足条件的元素

本文详解为何在遍历数组时直接使用 `foreach` 配合 `splice` 会导致漏删，并提供多种可靠替代方案（如 `while` 循环、`filter`、反向 `for` 循环等），帮助开发者正确实现“删除所有已完成任务”等常见逻辑。

你在 ToDo 应用中遇到的问题非常典型：试图用 Array.prototype.forEach() 遍历数组并同时调用 splice() 删除元素，结果只删掉了第一个匹配项——这是因为 forEach 的内部索引是预先确定的，不会因数组长度变化而回退或重校准。当删除第 i 个元素后，原 i+1 位置的元素会前移至 i，但 forEach 已经“走过”了该索引，导致跳过下一个待处理项。

✅ 正确方案一：使用 while 循环（原答案推荐，可控性强）

if (confirmDelete) {
  let index = 0;
  while (index < tasks.length) {
    if (tasks[index].completion === true) {
      tasks.splice(index, 1); // 删除当前项
      // 不递增 index，下一轮继续检查新位于 index 的元素
    } else {
      index++; // 仅当未删除时才前进
    }
  }
}

✅ 优势：原地修改、内存友好、逻辑清晰；
⚠️ 注意：需手动管理索引，避免无限循环（务必确保 index 在不删除时递增）。

✅ 正确方案二：使用 filter()（推荐 —— 函数式、安全、可读性高）

if (confirmDelete) {
  tasks = tasks.filter(task => !task.completion);
}

✅ 优势：语义明确（“保留未完成的任务”）、无副作用、不可变思维更利于调试与测试；
? 提示：若需同步更新 DOM（如移除 .completed 元素），建议先收集待删 ID 或索引，再统一操作，避免 DOM 与数据状态不同步。

✅ 正确方案三：反向 for 循环（高效且无需额外变量）

if (confirmDelete) {
  for (let i = tasks.length - 1; i >= 0; i--) {
    if (tasks[i].completion === true) {
      tasks.splice(i, 1);
    }
  }
}

✅ 优势：从末尾向前遍历，删除元素不影响尚

未访问的索引，无需调整计数器；
⚡ 性能：比正向遍历 + indexOf 更快（避免每次 indexOf 的 O(n) 查找）。

❌ 为什么 forEach + splice 会失败？（关键原理）

• forEach(callback) 内部按初始数组长度和索引 0, 1, 2... 顺序调用 callback；
• 当 tasks.splice(1, 1) 执行后，原索引 2 的元素变为新索引 1，但 forEach 下一步已执行 callback(tasks[2])（此时已是原索引 3 的元素），原 2 处的元素被跳过。

✅ 最佳实践建议

• 优先使用 filter() 实现“逻辑删除”，保持数据纯净；
• 若必须原地修改（如受限于外部引用），选用反向 for 或 while；
• 删除 DOM 元素时，建议先通过 tasks.filter(...) 获取待删任务 ID 列表，再用 querySelectorAll 精准定位并移除，避免因 DOM 节点动态变化导致 allCompletedTasksOnPage 与 tasks 状态错位。

至此，你不仅能修复当前 Bug，更能建立起对数组遍历与修改安全边界的清晰认知——这是写出健壮前端逻辑的重要一步。