递归scanf死循环

Question

我试图征求意见。如果它不是整数，那么它应该再次要求输入，这就是我使用递归的原因，如下所示。

#include <stdio.h>


void hi() {
    int a;
    printf("Input a: ");
    if(scanf("%d", &a) == 0)
        a = -1;
    switch(a) {
        case 1:
            printf("Hello!");
            break;
        case 0:
            return;
        default:
            hi();
            return;
            break;
    }
}

int main()
{
    hi();

    return 0;
}

输出是

Input a: *asd* Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a: Input a:

当我输入非整数值时，为什么 hi() 中的 scanf 不工作 （它不会停止 printf 并询问我的输入） 我该如何解决？

Answer 1

对于需要处理无效输入的交互式程序，您不想使用scanf，因为它无法从解析错误中恢复。请改用 fgets/sscanf 的组合。

另外，最好不要使用递归来解决可以使用迭代解决的问题。更容易维护和调试。你也会得到更好的成绩！

   int a = 0 ;
   while ( 1 ) {
       char buff[200] ;
       printf("Enter ...") ;
       if ( !fgets(buff, sizeof(buff), stdin) ) { perror("EOF") ; exit(1) ; }
       if ( sscanf(buff, " %d", &a) == 1 ) break ; // All Good
   } ;      

原始代码“scanf”将在第一次解析错误时反复失败，递归导致所有堆栈被使用，程序崩溃。

Answer 2

scanf 在您的示例中不起作用，因为它从缓冲的标准输入中读取。所以 scanf 认为你已经再次输入了一些东西，再次......再次......直到缓冲区被清除。清除缓冲区的时间取决于系统，并且可能因每个系统而异。

因此，最好的做法是像其他用户已经提到的那样使用 while 或 do-while 循环。

要让您的代码以某种方式工作，请在 if 之后添加 fflush(stdin)。 但请仅出于测试目的执行此操作，并尝试按照 as 循环中提到的方式实现它。

#include <stdio.h>    
void hi() {
    int a;
    printf("Input a: ");
    if(scanf("%d", &a) == 0){
        a = -1;            
    }
    fflush(stdin)
    switch(a) {
        case 1:
            printf("Hello!");
            break;
        case 0:
            return;
        default:
            hi();
            return;
            break;
    }
}

int main()
{
    hi();

    return 0;
}