如何从JavaScript数组中移除多个指定元素？

你的代码没达到预期效果，核心是三个关键错误：

1. 判断逻辑完全颠倒
   你写的if (myArray[num] == checkNums[num2])搞混了元素和目标值的对应关系：num是checkNums的索引，num2是myArray的索引，等于拿checkNums索引对应的数组元素去和myArray索引对应的移除数字对比，逻辑完全错误。正确判断应该是检查当前数组元素myArray[num2]是否等于要移除的数字checkNums[num]。

2. 用for...in遍历数组不合适
   for...in是为遍历对象属性设计的，用来遍历数组时，可能会拿到原型链上的额外属性，且遍历顺序不保证稳定，更适合用数组专用的遍历方式。

3. splice修改数组导致索引错位
   当你用splice删除元素后，数组长度变短，后面的元素会往前移，但for...in的索引不会自动调整，容易导致漏删或误删。

修复后的代码（直接修改原数组） #

换成倒序的普通for循环，避免索引错位，同时修正判断逻辑：

function test(myArray, ...checkNums) {
  // 遍历每个要移除的数字
  for (let i = 0; i < checkNums.length; i++) {
    const targetNum = checkNums[i];
    // 倒序遍历数组，删除元素时不会影响前面的索引
    for (let j = myArray.length - 1; j >= 0; j--) {
      if (myArray[j] === targetNum) {
        myArray.splice(j, 1);
      }
    }
  }
  return myArray;
}

const arr = test([1, 2, 3, 4], 3, 2);
console.log(arr); // 输出 [1,4]

更简洁的方案（返回新数组，无副作用） #

如果不想修改原数组（避免意外影响其他使用该数组的代码），可以用filter结合Set实现，代码更简洁高效：

function test(myArray, ...checkNums) {
  // 把要移除的数字转成Set，查找速度更快
  const numsToRemove = new Set(checkNums);
  // 过滤出不在移除列表里的元素
  return myArray.filter(item => !numsToRemove.has(item));
}

const arr = test([1, 2, 3, 4], 3, 2);
console.log(arr); // 输出 [1,4]

这种方式不会改动原数组，返回全新的结果数组，更安全也符合现代JS的写法。

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