C++ for循环数值范围布尔校验代码偶发判断错误求助

代码不存在偶发的玄学问题，判断错误是两个硬逻辑缺陷导致的，只是部分输入场景刚好撞上了正确结果，才看起来「多数时候正常」：

• 第一，变量语义和判断逻辑完全写反。你定义allAllowed是用来标记「是否所有值都合法」，但实际代码里碰到非法值设为1、碰到合法值设为0，最后输出时又把allAllowed=1判定为「全合法」，完全颠倒了标记含义。
• 第二，标记值会被遍历过程反复覆盖，且首个值非法时直接跳过剩余值的读取。进入循环后你每读一个值就重写一次allAllowed，根本不保留之前的检测结果，最终标记只代表最后一个读取到的值是否非法，不是全局校验结果；如果第一个待校验值本身非法，你会直接跳过整个for循环，根本不读取后续的输入值。

拿你给的两个测试用例举例：

1. 第一个输入"7 1000 1002 896 897 1004 987 960"：第一个待校验值1000合法，进入循环读后续6个值，最后两个值987、960都是合法值，最终allAllowed被覆盖为0，刚好触发你写的Unallowed value(s)输出分支，属于结果碰对了，逻辑本身是错的。
2. 第二个输入"7 896 1003 1004 899 897 898 906"：第一个待校验值896非法，直接跳过循环把allAllowed设为1，最终触发Only allowed values输出分支，和预期结果完全相反。

修正逻辑要遵循两个原则：

• 初始把allAllowed设为true（默认全合法），只要读到任意一个非法值，就把它设为false，之后再也不修改这个值（因为只要有一个非法，结果就固定了）
• 不管第一个值是否合法，都要把所有输入值全部读完校验，不要中途跳过大段逻辑

修复后的完整代码：

#include <iostream>
using namespace std;

int main() {
    int inputCount;
    bool allAllowed = true; // 初始默认全合法
    int val;

    cin >> inputCount;
    for (int i = 0; i < inputCount; ++i) {
        cin >> val;
        // 只要出现一个非法值，就把标记设为false，后续不再修改
        if (val < 900 || val > 1000) {
            allAllowed = false;
        }
    }

    if (allAllowed) {
        cout << "Only allowed values" << endl;
    } else {
        cout << "Unallowed value(s)" << endl;
    }

    return 0;
}

注：如果需要在读到非法值时立刻终止读取节省性能，可以在读到非法值时直接用break跳出循环，不需要保留多余的分支判断。

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