输入字符而不是 int 时的 C 无限循环

Question

我有一个 C 程序，它应该验证来自用户的输入是 1 到 8 之间的整数。如果输入一个整数，它就可以工作，但是当输入字符时，验证循环将永远重复。你能告诉我我做错了什么吗？

#include <stdio.h>

int main(void)
{
  int i;
  int input;
  domainEntry *myDomains = buildDomainDB();

  printf("You have the choice between the"
         " following top domains: 1-EDU, 2-COM"
         ", 3-ORG, 4-GOV, 5-MIL, 6-CN, 7-COM.CN, 8.CAN\n");
  printf("Which one do you want to pick?");
  scanf(" %d", &input);

  if(input < 1 || input > 9)
  {
    do  
    {   
      printf("Invalid input. Please try again.");
      printf("You have the choice between the"
             " following top domains: 1-EDU, 2-COM"
             ", 3-ORG, 4-GOV, 5-MIL, 6-CN, 7-COM.CN, 8.CAN\n");
      printf("Which one do you want to pick?");
      scanf(" %d", &input);

    }while((input < 1) || (input > 8));
  }
}

Answer 1

每当您接受用户输入时，你必须考虑保留在输入缓冲区中的每个字符(这里是 stdin)。使用 scanf 输入时尤其如此。 (或家人)因路scanf处理输入或匹配失败。当任一发生时，不再读取更多字符，并且任何有问题的字符都留在输入缓冲区 未读 -- 只是等着在下一次尝试读取时再次咬你(如果你在循环中输入，通常会导致无限循环)

此外，您必须了解每个转换说明符如何处理前导空格以及说明符是否使用前导空格(例如数字转换说明符和 "%s" )以及哪些不使用(所有其他说明符，特别是 "%c" 和 "%[...]" )。

这是诸如 fgets 之类的面向行的函数的主要原因之一。 (具有足够大小的缓冲区)或 POSIX getline建议用于获取用户输入。两者都读取并包含尾随 '\n'在填充的缓冲区中。这完全消耗了该行，消除了在输入缓冲区中未读取的额外字符的任何不确定性。然后您可以将缓冲区传递给 sscanf用于解析。这允许独立验证 (1) 读取(“我收到输入了吗？”)和(2)来自行的信息解析(“它是否包含我需要的信息？”)。
scanf如果使用得当，可以使用。这意味着 你负责查返回 的 scanf每次。你必须处理三个条件

(return == EOF)用户通过生成手册取消输入 EOF按 Ctrl+d (或在 Windows 上按 Ctrl+z，但请参阅 CTRL+Z does not generate EOF in Windows 10 (early versions) )；

(return < expected No. of conversions)发生匹配或输入失败。对于匹配失败，您必须考虑将留在输入缓冲区中的每个字符。 (在输入缓冲区中向前扫描读取和丢弃字符，直到找到 '\n' 或 EOF)；最后

(return == expected No. of conversions)指示成功读取——然后由您检查输入是否满足任何其他标准(例如正整数、正浮点数、在所需范围内等)。

使用您的代码，您可以通过以下方式获取输入 - 不断循环，直到收到有效输入或用户通过生成手动取消 EOF ，例如

#include <stdio.h>

void empty_stdin (void) /* simple helper-function to empty stdin */
{
    int c = getchar();

    while (c != '\n' && c != EOF)
        c = getchar();
}

int main(void)
{
    int input = 0,
        rtn = 0;    /* variable to save scanf return */
    // domainEntry *myDomains = buildDomainDB();

    for (;;) {  /* loop continually until valid input or EOF */
        printf ("\nSelect top level domain:\n"
                "  1-EDU\n"
                "  2-COM\n"
                "  3-ORG\n"
                "  4-GOV\n"
                "  5-MIL\n"
                "  6-CN\n"
                "  7-COM.CN\n"
                "  8.CAN\n\n"
                "choice: ");
        rtn = scanf (" %d", &input);    /* save return */

        if (rtn == EOF) {   /* user generates manual EOF */
            fputs ("(user canceled input.)\n", stderr);
            return 1;
        }
        else if (rtn == 0) {    /* matching failure */
            fputs (" error: invalid integer input.\n", stderr);
            empty_stdin();
        }
        else if (input < 1 || 8 < input) {  /* validate range */
            fputs (" error: integer out of range [1-8]\n", stderr);
            empty_stdin();
        }
        else {  /* good input */
            empty_stdin();
            break;
        }
    }

    printf ("\nvalid input: %d\n", input); 
}

( 注意 辅助函数的使用 empty_stdin() -- 即使在成功读取之后，您仍然应该 empty_stdin() 以确保输入缓冲区为空并为下一个用户输入做好准备-不管那是什么)

示例使用/输出

$ ./bin/getintmenu

Select top level domain:
  1-EDU
  2-COM
  3-ORG
  4-GOV
  5-MIL
  6-CN
  7-COM.CN
  8.CAN

choice: edu
 error: invalid integer input.

Select top level domain:
  1-EDU
  2-COM
  3-ORG
  4-GOV
  5-MIL
  6-CN
  7-COM.CN
  8.CAN

choice: 9
 error: integer out of range [1-8]

Select top level domain:
  1-EDU
  2-COM
  3-ORG
  4-GOV
  5-MIL
  6-CN
  7-COM.CN
  8.CAN

choice: 4

valid input: 4

如果你做你的工作，你可以成功使用 scanf如所须。

或者，您可以让您的生活更轻松并使用 fgets然后测试缓冲区中的第一个字符(通过简单地取消引用指针)是否是有效的菜单选择，例如

#include <stdio.h>

#define MAXC 1024   /* read buffer max characters */

int main (void) {

    int input = 0;
    char buf[MAXC];
    // domainEntry *myDomains = buildDomainDB();

    for (;;) {  /* loop continually until valid input or EOF */
        fputs  ("\nSelect top level domain:\n"
                "  1-EDU\n"
                "  2-COM\n"
                "  3-ORG\n"
                "  4-GOV\n"
                "  5-MIL\n"
                "  6-CN\n"
                "  7-COM.CN\n"
                "  8.CAN\n\n"
                "choice: ", stdout);
        if (!fgets (buf, MAXC, stdin)) {
            fputs ("(user canceled input.)\n", stderr);
            return 1;
        }

        if (*buf < '1' || '8' < *buf) { /* check 1st char, validate range */
            fputs (" error: invalid input\n", stderr);
            continue;
        }

        input = *buf - '0';     /* convert char to integer */
        break;
    }

    printf ("\nvalid input: %d\n", input); 
}

当然，如果某个键卡住并且用户输入的内容超过 1023字符——字符将保留在输入缓冲区中。但是最后一个字符是否为 '\n'的简单测试如果不是 MAXC - 1读取的字符会让您知道是否是这种情况。您的选择，但 fgets提供了一个更容易的实现。请记住——不要吝啬缓冲区大小。我宁愿有一个 10,000 字节太长的缓冲区，而 1 字节太短....