JS质数判断代码为何同时输出质数与非质数两种结果

问题根源 #

你的代码同时输出两类判断结果，是几个典型语法和逻辑错误共同导致的：

• 标志位作用域失效：你在循环内部写了let isPrimeX = false，相当于在循环块里重新声明了一个同名局部变量，根本修改不到外层初始值为true的isPrimeX。哪怕触发非质数分支执行了break，外层的标志位始终保持true，循环结束后必然会追加「是质数」的文本，和分支里提前拼接的「不是质数」文本同时显示。
• 判断条件完全错位：你把x<=1、x>2这类边界判断塞进了循环的整除校验条件里，只要输入值大于2，第一次循环i=2时就会因为x>2条件成立直接进入非质数分支，根本没做正确的整除判断；而小于等于1的非质数、等于2的质数这类场景，本来就不需要进循环计算，放在循环内判断逻辑完全混乱。
• 输入值未做类型转换：通过DOM的value属性拿到的输入内容默认是字符串类型，虽然JS做算术运算时会隐式转换，但遇到空输入、非数字输入时会出现不符合预期的计算结果。
• 额外逻辑隐患：你在非质数分支里提前拼接结果文本，只要分支触发就会写入非质数内容，一旦标志位判断出错，必然会出现双文本拼接的问题。

修正后代码 #

function myFunction(){
  // 输入值统一转为数字类型
  let x = Number(document.getElementById("int").value);
  const rootx = Math.sqrt(x);
  let textx = "";
  let isPrimeX = true;

  // 优先处理边界场景，不需要进入循环
  if (x <= 1) {
    isPrimeX = false;
  } else if (x === 2) {
    isPrimeX = true;
  } else {
    // 仅对大于2的数值执行循环整除校验
    for (let i = 2; i <= rootx; i++){
      if(x % i === 0){
        isPrimeX = false;
        break;
      }
    }
  }

  // 所有判断完成后，统一根据标志位拼接结果
  if(isPrimeX){
    textx = `${x} is a Prime Number`;
  } else {
    textx = `${x} is not a prime number`;
  }
  
  document.getElementById('demo').innerHTML = textx;
}

优化说明：删除循环内重复的变量声明，把边界判断移到循环外前置处理，不在判断分支内提前拼接文本，等所有校验逻辑走完后再根据最终标志位输出对应结果，彻底避免重复输出的问题。

内容的提问来源于stack exchange，提问作者Mohammad Mustak Absar Khan