为什么scanf会在无效输入时陷入无限循环？ [复制]

Question

在第 5 行中，我读取了一个整数，isint 如果读取的是整数，则为 1，如果不是整数，则为 0。如果isint 为 0，我有一个循环要求用户提供一个整数，我会一直阅读直到用户提供一个整数。我尝试这段代码给出一个字符而不是一个整数，但我有一个无限循环。该程序只是不等待提供新的输入。我的代码有什么问题？

#include <stdio.h>

int main(void) {

  int arg1;
  //int arg2;
  int attacknum = 1;
  int isint = 1;

  //printf("Insert argument attacks and press 0 when you have done this.\n");
  printf("Attack %d\n", attacknum);
  attacknum++;
  printf("Give attacking argument:");
  isint = scanf("%d", &arg1);  //line 5

  while(isint == 0){
    printf("You did not enter a number. Please enter an argument's number\n");
    isint = scanf("%d", &arg1);
    printf("is int is %d\n", isint);
  }
  return 0;
}

Answer 1

正如其他人所提到的，如果scanf 无法解析输入，它就会使其无法扫描。

由于这种行为，通常scanf 不适合交互式输入，并且与用户体验的一次一行的界面不匹配。

最好使用fgets 将一行读入缓冲区。然后使用sscanf 解析该行。如果您不喜欢输入，请将整行扔掉并阅读另一行。

类似这样的：

#include <stdio.h>

int main(void)
{
  char line[256];

  int arg1;
  int isint;

  while (1) {
    printf("Give attacking argument:");
    fgets(line, sizeof line, stdin);
    isint = sscanf(line, "%d",&arg1);
    if (isint) break;

    printf("You did not enter a number.Please enter an argument's number\n");
  }

  printf("Thanks for entering %d\n", arg1);

  return 0;
}

（对于生产代码，您需要处理长行，检查返回码，还要检查数字后面的尾随垃圾等）

实际上，如果您只想读取整数，不使用scanf，而使用strtol 是一种更好的方法。这为您提供了一个方便的指针，指向数字后面的字符，您可以检查它是空格还是 nul。

Answer 2

当scanf 遇到非数字时，它不会消耗任何输入并返回读取的零整数。非数字将保留在输入中，以便下次调用 scanf，其行为与第一次调用相同，等等。

在下面回答您的问题。您可以使用fgetc 解析至少一个字符，但这会为每个已键入的字符提供错误消息。通常我认为你想跳到换行符。为此，您可以按照 poolie 的建议使用fgets。或者您可以在scanf 之后添加以下内容。

int ch;
if (isint == 0)
while ((ch = fgetc(stdin)) != EOF && ch != '\n')
{
     /* Skip characters */
}

P.S：在您的情况下，最好将它放在循环中的第一个 printf 之前。