JS中for循环内使用pop()移除负数元素失效问题求助

核心问题有两个： #

• pop()方法的误用：pop()根本不接受参数，它的唯一作用是删除数组的最后一个元素，和你传入的s完全无关。所以你每次执行betterScores.pop(s)，都是在删掉数组最后一项，而不是当前遍历到的负数。
• 遍历过程中修改数组长度导致的错乱：用for...of遍历数组时，迭代器是基于初始数组长度生成的。当你执行pop()缩短数组后，遍历会提前结束——比如你的例子里，原数组长度是6，遍历到第三个元素-8时执行pop()，数组长度变成5，此时迭代器认为已经遍历完所有元素，直接终止循环，导致-8这个负数根本没被处理，反而最后一个元素3被删掉了。

正确的解决方案 #

方案1：用filter()（推荐，最简洁高效）

数组的filter()方法天生就是用来筛选元素的，一行代码就能解决：

const scores = [128, 0, -8, 50, 2, 3];
// 如果要去掉0，把条件改成s > 0即可
const betterScores = scores.filter(s => s >= 0);
console.log(betterScores); // 输出 [128, 0, 50, 2, 3]

方案2：手动遍历删除（从后往前遍历）

如果一定要手动操作数组，必须从后往前遍历，这样删除元素不会影响前面还没遍历到的索引：

const scores = [128, 0, -8, 50, 2, 3];
const betterScores = Array.from(scores);

// 从最后一个元素开始往前遍历
for (let i = betterScores.length - 1; i >= 0; i--) {
  if (betterScores[i] < 0) {
    betterScores.splice(i, 1); // splice(index, 1) 删除指定位置的元素
  }
}
console.log(betterScores); // 输出 [128, 0, 50, 2, 3]

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