为何我的C++代码未正确执行if语句？问题排查

问题1：变量重复定义导致未初始化行为 #

你在main函数开头定义了int integer;，但在while循环里又重新定义了int integer;。这导致循环内的integer是局部变量，和外层的完全无关。外层的integer从未被赋值，第一次进入while循环时integer != -1是未定义行为，程序可能随机进入或不进入循环；而且循环内的输入值根本没传递到外层变量，等于while条件里的判断完全失效。

问题2：总和超2000的判断时机错误 #

你在输入整数后、累加之前判断sum >= 2000，这时候sum还是上一次的累加结果。比如最后一次输入500时，sum当时是1650，判断不成立，累加后变成2150，但这时候已经跳过了祝贺判断，直接进入下一次循环。而下一次循环的while条件sum < 2000不成立，循环直接退出，所以永远不会触发祝贺信息。

问题3：阶段提示的逻辑顺序错误 #

你写的阶段判断是：

if (sum > 499){...}
if (sum > 999){...}
else if (sum > 1499){...}

这里第二个if是独立的，当sum>1499时，第一个if会触发（1499>499），第二个if也会触发（1499>999），而else if永远不会执行——因为sum>1499已经满足sum>999，else分支不会被走到。这就是重复触发低阶段提示的原因。

修复后的代码 #

#include<iostream>
using namespace std;

int main (){
    int sum = 0;
    int integer; // 只定义一次
    while (true) { // 改用无限循环，内部判断退出条件更清晰
        cout<<"Enter an integer: ";
        cin>>integer;

        // 先处理终止输入
        if (integer == -1){
            cout<<"Program was terminated.\nYour total sales are: "<<sum<<endl;
            break;
        }

        // 累加后再判断总和是否达标
        sum += integer;

        // 判断是否超过2000，达标则退出
        if (sum >= 2000) {
            cout<<"Congratulations! Your total sales are: "<<sum<<endl;
            break;
        }

        // 阶段提示用正确的if-else if链式判断
        if (sum > 1499){
            cout<<"You're almost there!"<<endl;
        }
        else if (sum > 999){
            cout<<"You're halfway through!"<<endl;
        }
        else if (sum > 499){
            cout<<"You're off to a good start!"<<endl;
        }
    }
    return 0;
}

修复后效果 #

当依次输入350、500、800、500时：

1. 输入350，sum=350 → 无阶段提示
2. 输入500，sum=850 → 触发"You're off to a good start!"
3. 输入800，sum=1650 → 触发"You're almost there!"
4. 输入500，sum=2150 → 触发"Congratulations! Your total sales are: 2150"，程序退出

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