JS removeFromArray多参数移除元素时传参顺序导致结果异常

你写的双层循环逻辑存在索引错位bug：调用splice删除数组元素时会直接修改原数组，删除位置之后的所有元素都会往前移一位，但外层循环的索引i还是照常自增，会直接跳过被移到当前i位置的元素，漏做匹配检查。你遇到的参数顺序不同结果不同的情况，本质上第一次传参返回正确只是巧合。

执行流程拆解 #

先看报错的调用removeFromArray([1, 2, 3, 4], 3, 2)：

• 初始状态：数组为[1,2,3,4]，待移除参数args = [3,2]，索引i从0开始遍历

1. i=0，元素1和两个待移除值都不匹配，i自增到1
2. i=1，元素是2，内层循环j走到1时匹配到待移除值2，执行splice(1,1)删除元素2，数组直接变成[1,3,4]：原本在索引2的3往前移到了索引1的位置
3. 内层循环结束后i照常自增到2，直接跳过了索引1位置刚移过来的3，没有做匹配检查
4. i=2时元素是4，不匹配任何待移除值，i自增到3后不满足循环条件，遍历结束，最终返回错误结果[1,3,4]

再看你以为正常的调用removeFromArray([1, 2, 3, 4], 2, 3)：

• 初始状态：数组为[1,2,3,4]，待移除参数args = [2,3]

1. i=0，元素1不匹配，i自增到1
2. i=1，元素是2，j=0时匹配到2，执行splice(1,1)删除2，数组变成[1,3,4]，3前移到索引1位置
3. 此时内层j循环还没结束，j自增到1，正好拿当前i=1位置的新元素3和args[1]的3做匹配，刚好命中，又执行一次splice删掉3，数组变成[1,4]
4. 后续遍历没有匹配项，最终返回正确结果——这完全是参数顺序和元素移位的巧合，不是逻辑本身正确。

修复方法 #

如果要保留你原本的循环+splice写法，只需要在删除元素后补两行逻辑即可：

const removeFromArray = function(array, ...args) {
  for (let i = 0; i < array.length; i++) {
    for (let j = 0; j < args.length; j++){
      if (array[i] === args[j]) {
        array.splice(i, 1);
        i--; // 索引回退一位，下次循环重新检查当前i位置（删除后后续元素已经前移）
        break; // 已经匹配到待移除值，不需要再循环匹配其他参数
      }
    }
  }
  return array;
};

更简洁、不会出现索引问题的写法是用数组filter方法，不需要修改原数组：

const removeFromArray = function(array, ...args) {
  return array.filter(item => !args.includes(item));
};

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