本文详解 JavaScript 数组遍历中因动态修改长度导致末尾元素被跳过的问题，以质数筛选为例，指出 splice() 与 forEach 混用的根本缺陷，并提供安全、函数式、高性能的替代方案。
本文详解 javascript 数组遍历中因动态修改长度导致末尾元素被跳过的问题，以质数筛选为例，指出 `splice()` 与 `foreach` 混用的根本缺陷，并提供安全、函数式、高性能的替代方案。
在 JavaScript 中，试图通过原地修改数组（如 splice()）配合 forEach() 或普通 for 循环来筛选质数，极易引发逻辑错误——最典型的表现就是最后一个被插入或位于末尾的合数未被检查。问题根源在于：forEach() 的迭代索引是基于原始数组长度预先确定的，而 splice() 会实时缩短数组，导致后续元素前移、索引错位，最终使某些元素“逃逸”检测。
以原始代码为例：
function ModuloPrimes(variables) {
const variables = [2,3,25,5,7,9,15,14,4]; // 注意：此处重新声明变量，覆盖入参
variables.forEach(val => {
for(let i = 0; i < variables.length; i++) {
if(variables[i] % val === 0 && variables[i] !== val) {
variables.splice(i, 1);
i--; // ✅ 关键修正：删除后需回退索引，否则跳过下一个元素
}
}
});
return variables;
}即使修复为 i--，该方法仍存在严重缺陷：
-
forEach不响应数组长度变化，循环次数固定为初始长度（9次），但实际需多次扫描才能收敛； - 原地修改破坏函数纯度，难以调试且易引发并发问题；
- 时间复杂度高达 O(n³)，效率低下。
✅ 推荐方案：使用不可变函数式方法
利用 filter() + some() 组合，语义清晰、安全高效：
const variables = [2, 3, 25, 5, 7, 9, 15, 14, 4]; const primes = variables.filter(n => !variables.some(val => n % val === 0 && n !== val) ); console.log(primes); // [2, 3, 5, 7]
原理说明：对每个 n，检查是否存在另一个 val（非自身）能整除 n；若不存在，则 n 是质数（此处隐含前提：数组仅含 ≥2 的正整数）。该逻辑天然规避了索引偏移问题，且一次遍历即可完成筛选。
⚠️ 注意事项：
- 此方法假设输入数组不含
1或负数（1会被误判为质数，因无其他正整数能整除它且不等于自身）； - 若需严格质数判断（如处理
1,0, 负数），应单独封装isPrime(n)函数； -
&&运算符优先级高于&（按位与），原代码中误用&可能引发隐式类型转换错误，务必使用&&。
总结：永远避免在 forEach 或固定上限 for 循环中修改正在遍历的数组长度。优先选择声明式方法（filter/map/reduce），既提升可读性与健壮性，也符合现代 JavaScript 最佳实践。
