JavaScript checkPrime函数返回错误值问题及优化咨询

首先咱们拆解原代码里导致误判的核心问题：

• 循环逻辑错误：原函数在for循环的第一次迭代（i=2）时，只要num % i !== 0就直接返回num + ' is prime'，根本没检查后续的因数。比如27，27%2确实不等于0，但它能被3整除，这就直接导致了误判。
• 冗余的数组判断：你创建的numbers数组完全没必要，而且把范围判断放在循环内部也不合理，应该先做范围校验。
• 循环范围低效：判断质数不需要遍历到num-1，只需要遍历到Math.sqrt(num)就够了——如果一个数有大于其平方根的因数，那必然对应一个小于平方根的因数，这样能大幅减少循环次数。

接下来是修正后的代码，同时咱们把数字2的判断逻辑和其他质数逻辑合并：

function checkPrime(num) {
    // 先处理范围判断
    if (num < 0 || num > 10000) {
        return num + ' is not in range';
    }
    // 处理0、1的特殊情况
    if (num <= 1) {
        return 'It is a separate case';
    }
    // 从2遍历到num的平方根，合并2的判断逻辑
    for (let i = 2; i <= Math.sqrt(num); i++) {
        if (num % i === 0) {
            return num + ' is not prime';
        }
    }
    // 如果循环结束都没找到因数，就是质数（包括2）
    return num + ' is prime';
}

console.log(checkPrime(27)); // 输出：27 is not prime
console.log(checkPrime(145)); // 输出：145 is not prime
console.log(checkPrime(2)); // 输出：2 is prime

关键修改说明： #

1. 提前做范围校验：把num是否在0-10000的判断放在最前面，避免后续无效计算。
2. 合并2的判断逻辑：当num=2时，Math.sqrt(2)约等于1.414，循环条件i <= 1.414不成立，循环直接跳过，最后进入返回质数的分支，完美和其他质数逻辑合并，不需要单独判断2。
3. 修正循环逻辑：只有当遍历完所有可能的因数（到平方根）都没找到能整除num的数，才判定为质数，不会提前返回错误结果。
4. 使用let声明变量：替换原代码的var，避免变量提升带来的潜在问题，符合现代JS规范。

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