C++循环逻辑异常：判断输入整数是否含0时结果不符预期

问题描述 #

需求为：声明布尔变量noCopies，读取输入值inputCount，再读取inputCount个整数，若其中无0则输出“No copies of 0”，否则输出“0 present”。但原代码在输入5 0 2 4 -5 1时，本该输出“0 present”却输出“No copies of 0”，需排查for循环中的逻辑缺失。

原代码存在的核心问题 #

1. 循环边界错误 #

原循环条件for (i = 0; i <= numVals; ++i)会执行numVals + 1次，而需求仅需读取numVals个整数。这会导致多读取一个输入，打乱后续判断的逻辑基础。正确的循环条件应为i < numVals。

2. 布尔变量的赋值逻辑完全错误 #

原代码中每次循环都会覆盖allPositive的值，最终结果仅由最后一个输入的数字决定。比如输入5 0 2 4 -5 1时，最后一个数字是1，allPositive被设为true，因此错误输出“No copies of 0”。

需求的核心是判断是否存在0，正确逻辑应该是：

• 初始将noCopies设为true（假设没有0）
• 只要遇到0，就将noCopies设为false，且后续无需再修改该值（因为只要有一个0就满足输出“0 present”的条件）

3. 变量名与需求不符 #

原代码使用allPositive变量，但需求是判断是否存在0，变量名应改为noCopies以贴合语义，避免混淆。

修复后的代码 #

#include <iostream>
using namespace std;

int main() {
    int inputCount;
    int num;
    bool noCopies = true; // 初始假设没有0存在

    cin >> inputCount;

    for (int i = 0; i < inputCount; ++i) {
        cin >> num;
        if (num == 0) {
            noCopies = false;
            // 可选优化：找到0后直接退出循环，无需处理剩余输入
            // break;
        }
    }

    if (noCopies) {
        cout << "No copies of 0" << endl;
    } else {
        cout << "0 present" << endl;
    }

    return 0;
}

修复说明 #

• 修正循环条件为i < inputCount，确保仅读取指定数量的整数。
• 调整布尔变量的初始值与赋值逻辑：初始假设无0，遇到0则标记为存在0，且该标记一旦设为false就不会再被改变。
• 可选添加break语句优化性能，一旦找到0就提前终止循环，减少不必要的输入读取。

内容的提问来源于stack exchange，提问作者Flin-Paolo Roxas Cabagnot