在 while < 0 条件下将字符串输入 scanf 会导致无限循环

Question

我有一个“分钟”变量，我希望用户输入一个正数。

int main(void)
{
    float minutes;
    minutes = -1;
    printf("Find out how many bottles worth of water your showers use!\n");
    printf("How many minutes do you spend in the shower? ");
    scanf("%f", &minutes);
    while(minutes < 0)
    {
        printf("Please enter a positive number: ");
        scanf("%f", &minutes);
    }
}

它按预期用于数字。如果 minutes >= 0，它接受它，如果 minutes

printf("Please enter a positive number: "); 

并且永远不会给我输入新值的机会。为什么会这样，我该如何解决？谢谢！

Answer 1

如果您不输入数值，您输入的任何内容都会保留在输入缓冲区中。您可以通过读取scanf 的返回值来检查这一点，它告诉您读取的项目数。如果为 0，则可以使用getchar 读取字符，直到下一个换行符刷新缓冲区。

int main(void)
{
    int rval, c;
    float minutes;
    minutes = -1;
    printf("Find out how many bottles worth of water your showers use!\n");
    printf("How many minutes do you spend in the shower? ");
    rval = scanf("%f", &minutes);
    if (rval == 0) {
        while (((c = getchar()) != '\n') && (c != EOF));
    }
    while(minutes < 0)
    {
        printf("Please enter a positive number: ");
        rval = scanf("%f", &minutes);
        if (rval == 0) {
            while (((c = getchar()) != '\n') && (c != EOF));
        }
    }
}

Answer 2

%f 转换说明符告诉scanf 在看到不属于合法浮点常量的字符（即，不是数字、小数点、或签名）。那个坏字符留在输入流中，所以下一次调用scanf 失败，下一个，下一个，等等。

您应该始终检查 scanf 的返回值 - 它会告诉您从输入流中成功读取和分配了多少项目。在这种情况下，您期望的是单个项目，因此您应该获得返回值 1。如果您获得返回值 0，则意味着输入不是正确的浮点值，并且错误的输入必须以某种方式被清除。这是一种可能的解决方案：

if ( scanf( "%f", &minutes ) == 1 )
{
  // process minutes as normal
}
else
{
  // clear everything up to the next whitespace character
  while ( !isspace( getchar() ) )
    ; // empty loop 
}

唯一的问题是scanf有点笨，如果你输入类似123fgh的东西，它会转换并分配123，同时将fgh留在输入流中；您可能希望完全拒绝整个输入。

一种解决方案是将输入读取为文本，然后使用strtod 进行转换：

char buffer[BUFSIZE]; // where BUFSIZE is large enough to handle expected input
...
if ( fgets( buffer, sizeof buffer, stdin ) )
{
  char *chk; // chk will point to the first character *not* converted; if
             // it's anything other than whitespace or the string terminator,
             // then the input was not a valid floating-point value.
  double tmp = strtod( buffer, &chk );
  if ( isspace( *chk ) || *chk == 0 )
  {
    minutes = tmp;
  }
  else
  {
    // input was not a proper floating point value
  }
}

这样做的好处是不会在输入流中留下废话。