如何在条件为真时重复do-while循环？二进制输入校验代码求助

嘿，我帮你找出代码里导致异常的问题了，咱们一步步来修正：

1. checknumber() 函数的核心问题 #

• 参数传递错误：你把flag按值传给函数，函数内部修改它不会影响主函数里的变量，这个参数等于白传了！函数应该直接返回校验结果（1表示合法，0表示非法），完全不需要额外的flag参数。
• 循环逻辑漏洞：你在while循环的第一次迭代里就直接return flag了，根本没检查完所有数位！比如输入120，函数只检查最后一位0就返回合法，但中间的2完全没被检测到，这是大问题。
• 未处理输入0的情况：如果输入0，number != 0的循环不会执行，函数没有返回值，会导致程序出现未定义行为。

2. main() 函数的逻辑缺陷 #

• 你的do-while循环先读取数字再校验，但如果用户输入负数（本该直接退出的情况），程序还是会进入校验流程，不符合需求。
• 外层while(num > -1)的循环条件，会导致输入负数后还要执行一次校验才退出，逻辑不通。

修正后的完整代码 #

先修正checknumber()函数 #

让它直接返回校验结果，处理所有数位和边界情况：

int checknumber(int number) {
    // 负数用于退出，无需校验直接返回0
    if (number < 0) {
        return 0;
    }
    // 0是合法的二进制数，单独处理
    if (number == 0) {
        return 1;
    }
    while (number != 0) {
        int val = number % 10;
        if (val != 0 && val != 1) {
            // 发现非法数位，直接返回0
            return 0;
        }
        number /= 10; // 去掉最后一位，继续检查前面的数位
    }
    // 所有数位都合法，返回1
    return 1;
}

再修正main()函数 #

调整流程，先判断是否退出，再校验输入直到合法：

#include <iostream>
using namespace std;

// 先声明函数
int checknumber(int number);
int binaryToDecimal(int number);

int main() { 
    int num; 
    while (true) { 
        cout << "Enter Number: "; 
        cin >> num; 
        
        // 输入负数，直接退出程序
        if (num < 0) {
            cout << "正在退出程序..." << endl;
            break;
        }
        
        // 校验数位，不合法则提示重新输入
        while (!checknumber(num)) {
            cout << "输入无效！请输入仅包含0和1的数字：";
            cin >> num;
            // 重新输入时也要检查是否为负数
            if (num < 0) {
                cout << "正在退出程序..." << endl;
                return 0;
            }
        }
        
        // 转换并输出十进制结果
        cout << "Result in Decimal = " << binaryToDecimal(num) << endl; 
    } 
    return 0;
}

// 标准二进制转十进制实现（如果你的实现有问题，可以用这个）
int binaryToDecimal(int number) {
    int decimal = 0;
    int base = 1;
    while (number != 0) {
        int lastDigit = number % 10;
        decimal += lastDigit * base;
        base *= 2;
        number /= 10;
    }
    return decimal;
}

关键修正点说明 #

1. 简化checknumber()逻辑：去掉没用的flag参数，直接返回校验状态，代码更清晰易懂。
2. 完整检查所有数位：循环中每次去掉最后一位，确保每一位都被检查，不会提前返回结果。
3. 优化退出逻辑：输入后第一时间判断是否为负数，直接退出，避免不必要的校验操作。
4. 处理边界情况：明确处理输入0的合法情况，同时在重新输入时也检查负数，保证退出逻辑的一致性。

现在程序就能正常工作了：输入非法数字会提示重新输入，输入仅含0和1的数字会输出十进制结果，输入负数直接退出程序。

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