JavaScript质数检测函数仅对数字9输出错误，求代码排查建议

问题定位 #

你的代码在输入9时出错，核心问题出在大于2的数字的循环判断逻辑：
当输入数字大于2时，循环只执行一次就直接break。比如判断9时，第一次检查9%2=1，不等于0，就直接进入else分支输出“是质数”并终止循环，完全没检查后续的除数（比如3，9%3=0，这才是正确的判定依据）。

另外还有两个小问题：

• 函数调用时没有传入参数（primeDetector();），测试时需要传入目标数字，比如primeDetector(9);
• 多个独立的if判断会做冗余检查，比如输入2时，会依次判断inputNumber<1、inputNumber===1，虽然不影响结果，但可以优化。

修复后的代码 #

const primeDetector = function (inputNumber) {
  // 处理小于1的输入
  if (inputNumber < 1) {
    console.log("Your input is not a prime");
    return; // 提前终止，避免后续判断
  }
  // 处理1的情况
  if (inputNumber === 1) {
    console.log("Your input is Neither prime nor composite");
    return;
  }
  // 处理2的情况（唯一的偶质数）
  if (inputNumber === 2) {
    console.log(`${inputNumber} is a prime number`);
    return;
  }
  // 处理大于2的偶数（直接排除，因为除了2所有偶数都不是质数）
  if (inputNumber % 2 === 0) {
    console.log(`${inputNumber} is NOT a prime number`);
    return;
  }
  // 处理大于2的奇数，只需要检查奇数除数，从3开始到平方根（优化性能）
  let isPrime = true;
  // 只需要检查到输入数字的平方根即可，因为如果n有大于平方根的因数，必然有一个对应的小于平方根的因数
  for (let i = 3; i <= Math.sqrt(inputNumber); i += 2) {
    if (inputNumber % i === 0) {
      isPrime = false;
      break;
    }
  }
  // 根据标志位输出结果
  if (isPrime) {
    console.log(`${inputNumber} is a PRIME number`);
  } else {
    console.log(`${inputNumber} is NOT a prime number`);
  }
};

// 测试示例
primeDetector(9); // 输出：9 is NOT a prime number
primeDetector(7); // 输出：7 is a PRIME number
primeDetector(2); // 输出：2 is a prime number
primeDetector(1); // 输出：Your input is Neither prime nor composite
primeDetector(-5); // 输出：Your input is not a prime

关键优化点 #

1. 修复循环逻辑：使用isPrime标志位，遍历所有可能的除数后再判定结果，不会提前终止循环。
2. 性能优化：
   • 先排除大于2的偶数，不需要进入循环
   • 循环只检查到输入数字的平方根，减少循环次数
   • 除数只取奇数，跳过偶数，进一步减少循环次数
3. 提前终止函数：每个条件判断后加return，避免不必要的后续判断，提升代码效率。

内容的提问来源于stack exchange，提问作者Abir Tasrif Anto