JavaScript含if-else的while无限循环终止问题求助

你的代码出现无限循环主要有三个核心问题：

1. 终止条件逻辑混乱 #

你用stopme和stopme1两个变量互相赋值控制循环，还在分支里重复对变量赋值（比如else分支多次写stopme=false），导致逻辑绕来绕去，很容易出现触发时机错误的情况。

2. 未处理循环求和的死循环场景 #

当n分解质因数的和等于自身时（比如n=4，分解为2+2=4），代码会不断重复分解求和→回到原数的过程，永远触发不了终止条件。

3. 质数判断逻辑有漏洞 #

原代码中遍历smallestvalue判断质数的逻辑不严谨，部分质数场景下无法正确触发stopme=false，导致循环无法终止。

修复后的代码 #

下面是重构后的代码，简化逻辑并解决上述问题：

var smallestValue = function(n) {
    // 单独实现质数判断函数，逻辑更清晰
    function isPrime(num) {
        if (num < 2) return false;
        for (let i = 2; i <= Math.sqrt(num); i++) {
            if (num % i === 0) return false;
        }
        return true;
    }

    while (true) {
        // 如果当前n是质数，直接返回
        if (isPrime(n)) {
            return n;
        }
        // 分解质因数并计算总和
        let sum = 0;
        let current = n;
        for (let i = 2; i <= current; i++) {
            while (current % i === 0) {
                sum += i;
                current = current / i;
            }
        }
        // 如果求和后和等于原数，说明进入无法跳出的循环，直接返回
        if (sum === n) {
            return n;
        }
        // 更新n为求和结果，继续循环
        n = sum;
    }
};

修复说明 #

• 用独立的isPrime函数处理质数判断，避免原代码中遍历判断的漏洞，逻辑更直观。
• 循环逻辑直接明确：只要n不是质数就分解求和，若求和后等于原数则终止（避免死循环），否则更新n继续循环。
• 去掉了混乱的stopme和stopme1变量，用while(true)结合return直接终止循环，从根源上避免了变量赋值导致的逻辑错误。

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