C 上的布尔表达式

Question

创建一个简单的 C 函数来防止用户输入除 1 到 9 之间的数字以外的任何内容。不应接受任何其他输入，包括字母符号和任何小于 1 和大于 9 的数字。 到目前为止，它非常简单。但是，代码应该检查输入的字符不是符号或字母的部分没有按照我希望的方式工作。

int validateUserInput(){
    printf("%s\n", "Please enter a number from 1 to 9: ");

    char value = getchar();
    int numValue = value;
    char temp;

    int digitCounter = 0;

    while((temp = getchar()) != '\n'){
        digitCounter++;
    }
    //if there is more than 1 digit.
    if(digitCounter>0){
        printf("%s\n","Input too long!");
        return validateUserInput();
    }
    // if the char entered is not between 1 and 9
    // this part is giving me a hard time.
    else if(numValue < 49 || numValue > 57){
        printf("%s\n", "Imput is not within the valid parameters");
        return validateUserInput();
    }
    return numValue;
}

Answer 1

在使用输入例程进行输入时，还有几点需要注意。如果用户需要取消输入怎么办？ （例如，在 Windows 上按 ctrl+d 或 ctrl+z）正如您所写，无法取消循环。 （虽然这适用于您仅强制输入 1-9 的目的，但它无法取消）如果您捕获 EOF，则您提供了两种方式，一种取消方式，以及一种在调用函数（例如通过检查 EOF 的返回）

虽然递归有它的位置，但请注意，每个递归本身都是一个单独的函数调用，需要一个单独的堆栈，以及函数调用的所有其他陷阱。很多时候，通过简单的goto 语句可以避免这种情况。 glibc 经常使用goto（例如，检查qsort、getdelim 的来源等。）在您的情况下，单个goto 标签可以完全消除递归的需要。例如，您可以在满足所有条件的同时执行以下类似操作：

int validateuserinput()
{
    int c, extra, tmp;

    getinput:;  /* a simple goto can avoid recursion completely */
    extra = 0;
    printf ("Please enter a number from 1 to 9: ");
    /* prompt/loop while chars not between 1 and 9 */
    for (c = getchar(); (c < '1' || '9' < c ); c = getchar()) {
        if (c == '\n') {    /* no digits in input */
            fprintf (stderr, "error: invalid input.\n");
            goto getinput;
        }
        else if (c == EOF) {  /* trap EOF */
            fprintf (stderr, "\nerror: input canceled.\n");
            return c;
        }
    }
    /* empty input buffer -- increment extra count */
    for (tmp = getchar(); tmp != '\n' && tmp != EOF; tmp = getchar())
        extra++;

    if (extra) { /* if extra chars -- input too long */
        fprintf (stderr, "error: input too long.\n");
        goto getinput;
    }

    return c - '0';  /* return integer value instead of ASCII value */
}

（只是一种风格说明，C 通常避免使用 camelCase 变量名而支持小写，这只是一般性，完全取决于您）

您可以使用以下一小段代码检查功能（并响应取消）：

#include <stdio.h>

int validateuserinput();

int main (void) {

    int n;

    if ((n = validateuserinput()) != EOF)
        printf ("\n valid input : %d\n", n);

    return 0;
}

使用/输出示例

测试只接受 1-9 的输入：

$ ./bin/inputhandler
Please enter a number from 1 to 9: Hello World!
error: invalid input.
Please enter a number from 1 to 9: ?
error: invalid input.
Please enter a number from 1 to 9: 0
error: invalid input.
Please enter a number from 1 to 9: 23
error: input too long.
Please enter a number from 1 to 9: 6

 valid input : 6

测试输入取消（例如 ctrl+d 或 Windows 上的 ctrl+z）

$ ./bin/inputhandler
Please enter a number from 1 to 9:
error: input canceled.

虽然使用递归并没有错，但问“这是否需要从递归函数开始？” 有时答案是肯定的，但通常有简单的答案避免额外开销的方法。 （注意： 一些递归的开销很小，因此在您的情况下这不是一个重要的考虑因素，但如果您无意中调用了一个旋转一百万次的递归函数，它很快就会成为一个问题）

查看所有答案，如果您有任何问题，请告诉我们。

Answer 2

您只设置一次 numValue - 对于第一个 getchar() - 看起来您希望在每个 getchar() 之后设置它。

Answer 3

 while((temp = getchar()) != '\n'){
        digitCounter++;
    }

可能

 while((temp = getchar()) != '\n'){
        if (temp < 49 || temp > 57 || ++digitCounter>0)
            return validateUserInput();
    }

我认为这种方式很简单