JS数组循环时最后一个元素未被检查的问题排查求助

你的代码存在以下几个关键错误： #

1. 参数被重复声明覆盖
   函数定义了参数variables，但内部又用const variables重新赋值，导致传入参数完全失效，且严格模式下会触发语法错误。

2. forEach遍历的是原数组快照
   forEach会在遍历开始前创建原数组的副本，后续修改数组不会影响遍历的元素序列。比如原数组中的4，当处理val=2的遍历阶段结束后，后续遍历的val（如3、25等）都无法整除4，而当遍历到val=4时，检查逻辑无法移除自身，最终导致4残留。

3. splice后错误递增索引
   当用splice(i,1)移除元素时，数组后续元素会向前移位，此时++i会跳过新移到当前索引的元素。比如原数组中移除索引7的14后，4移到索引7，但i被递增到8，直接超出数组长度，导致4未被检查。

4. 逻辑运算符错误
   代码中使用了&（按位与）而非&&（逻辑与），虽然此处结果巧合正确，但属于语法误用，会导致逻辑判断隐患。

修正方案 #

方案1：用filter+some实现（简洁高效）

利用数组方法直接筛选出无法被其他元素整除的数：

function ModuloPrimes(inputVariables) {
  // 先去重，避免重复元素干扰判断
  const uniqueVars = [...new Set(inputVariables)];
  return uniqueVars.filter(num => {
    // 检查当前数是否能被其他元素整除，不能则保留
    return !uniqueVars.some(other => other !== num && num % other === 0);
  });
}

// 测试
console.log(ModuloPrimes([2,3,25,5,7,9,15,14,4])); // 输出 [2,3,5,7]

方案2：修正原循环写法

如果要保留循环逻辑，需注意遍历副本、正确处理索引：

function ModuloPrimes(variables) {
  // 创建数组副本，避免修改原输入数组
  const arr = [...variables];
  // 遍历副本的快照，确保所有初始元素都被用来检查
  [...arr].forEach(val => {
    for(let i = 0; i < arr.length; ) {
      if(arr[i] % val === 0 && arr[i] !== val) {
        arr.splice(i, 1);
        // 移除元素后不递增i，因为后续元素已前移到当前位置
      } else {
        i++; // 未移除元素时再递增索引
      }
    }
  });
  return arr;
}

// 测试
console.log(ModuloPrimes([2,3,25,5,7,9,15,14,4])); // 输出 [2,3,5,7]

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