无法弄清楚 C 中的 while/if 循环的逻辑

Question

我有一些代码，我遇到问题的功能是：

unsigned int getInputData() {

    printf("Please input a positive integer number terminated with a carriage return.\n");

    do{ 
        scanf("%c", &input);

        if(isdigit(input))  {
            temp = charToInt(input);
            rValue = mergeInt(rValue, temp);
        }
        if(rValue >= imax)  {
            rValue = 0;
            printf("ERROR: That is too large of an integer. Please try again. \n");
        }
        else if(isalpha(input)){
            rValue = 0;
            printf("This is not a integer. Please try again. \n");
        }
        else{
            printf("OK. This is a good number. \n");
        }
    } while(1);
}

我正在单独扫描每个char，并将其合并到int。这正是我想要做的，但我只希望它在用户输入时打印一次"OK. This is a good number."。例如：如果有人要输入：12345 我希望它返回："OK. This is a good number." 一次5 个字符而不是每个字符一次。希望这是有道理的，已经有一段时间了，所以任何事情都会有所帮助。

Answer 1

您的代码背后存在巨大的逻辑问题：

• 您无限循环而不检查输入结束：

您说您想在用户输入几个数字时判断这是否是一个好数字，但是您一次只读取一个字符，并且您没有定义数字的结尾方式。 尽管您确实指定以回车结束，但您并没有这样设计您的算法，您从不检查 \n 字符。

• 您为getInputData() 函数定义了一个返回值，但您永远不会从该函数返回。
• 您测试input 是否是一个数字来更新值，但对于错误，只有当它是一个字母字符时才会显示错误。

基本上，为了保持您编写算法的方式，这里有另一种看法：

unsigned int getInputData() {
    char input;
    long value=0;

    do {
        scanf("%c", &input);

        if (isdigit(input))
            value = value*10+input+'0';
        else if (input == '\n')
            return 1;
        else
            return 0;

    } while(1);
}

int main() {
    printf("Please input a positive integer number terminated with a carriage return.\n");

    if (getInputData() == 1)
        printf("OK. This is a good number.\n");
    else
        printf("This is not a integer. Please try again. \n");

    return 0;
}

但我确实退出了无限循环以便能够检查结果。

注意：出于示例的目的，我没有检查溢出。 NB1：我一直使用scanf() 来接近您的代码，但如果您一次只想读取一个字符，最好使用getchar()，这样更简单、更快捷. N.B.2：您还可以通过使用scanf() 的更多功能来简化您的代码：

unsigned int getInputData() {
    unsigned input;
    long value=0;
    int n;

    do {
        n = scanf("%u", &input);
        if (n == 0)
            return 0;
        else
            return 1;
    } while(1);
}

您甚至可以尝试使用scanf("%a[0-9]")，它是一个 GNU 扩展。详情请见man scanf。