如何检查函数返回值并重复执行循环？附C++二进制数校验代码问题

首先，咱们先梳理下你现有代码里的几个核心问题，再一步步修复：

现有代码的问题点 #

1. checknumber()函数逻辑错误：
   • 循环里的return flag写在了循环内部，导致只检查数字的最后一位就直接返回，根本没遍历所有位
   • flag是值传递，函数内部修改它不会影响主函数里的变量
   • 当输入数字为0时，函数没有返回值，会触发未定义行为
2. 主循环逻辑不严谨：
   • while(num>-1)的条件不符合你的需求（你要的是输入小于-1才退出）
   • flag未初始化就使用，判断条件完全不可靠

修复后的完整代码 #

#include <iostream>
#include <cstdlib> // 用于abs()处理负数

using namespace std;

// 检查数字每一位是否只有0或1，返回1表示合法，0表示非法
int checknumber(int number);
// 将二进制数字转换为十进制
int binaryToDecimal(int num);

int main() {
    int num = 0; // 初始值设为0，确保第一次进入主循环
    // 只有输入小于-1时才退出程序
    while (num >= -1) {
        do {
            cout << "Enter Number: ";
            cin >> num;
            
            // 如果输入已经小于-1，直接跳出输入循环
            if (num < -1) {
                break;
            }
            
            // 非法输入时给出提示
            if (checknumber(num) == 0) {
                cout << "Invalid input! Please enter a number with only 0s and 1s." << endl;
            }
        // 合法输入 或 输入小于-1时，退出输入循环
        } while (num >= -1 && checknumber(num) == 0);
        
        // 确认输入小于-1，直接退出主循环
        if (num < -1) {
            break;
        }
        
        // 输出转换结果
        cout << "Result in Decimal = " << binaryToDecimal(num) << endl;
    }
    return 0;
}

int checknumber(int number) {
    // 0是合法的二进制数
    if (number == 0) {
        return 1;
    }
    // 取绝对值，只检查数字部分（比如-110的数字位是1、1、0，属于合法）
    int num = abs(number);
    while (num != 0) {
        int val = num % 10;
        // 遇到非0非1的数字，直接返回非法
        if (val != 0 && val != 1) {
            return 0;
        }
        // 去掉最后一位，继续检查下一位
        num /= 10;
    }
    // 所有位都合法，返回1
    return 1;
}

int binaryToDecimal(int num) {
    int decimal = 0;
    int base = 1;
    int absNum = abs(num);
    
    while (absNum > 0) {
        int lastDigit = absNum % 10;
        decimal += lastDigit * base;
        base *= 2;
        absNum /= 10;
    }
    // 保留原数的正负号
    return (num < 0) ? -decimal : decimal;
}

关键修复说明 #

1. 重构checknumber()：
   • 去掉了冗余的flag参数，直接返回检查结果，避免值传递的坑
   • 处理了数字为0的特殊情况
   • 使用abs()处理负数，只检查数字部分的每一位，同时保留原数的正负逻辑
   • 完整遍历所有数字位，确保每一位都被检查
2. 优化主循环逻辑：
   • 把主循环条件改为while(num >= -1)，严格符合你“输入小于-1才退出”的需求
   • 在输入循环中加入提前退出判断，避免对小于-1的输入做无效检查
   • 增加非法输入的提示，提升用户体验
3. 完善binaryToDecimal()：
   • 处理了负数的二进制转换，返回对应的负十进制值

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