給我一個無限循環的代碼

Question

#include <stdio.h> 
void load_menu(void); 

int main(void) 
{ 
    load_menu(); 
    return 0; 
} 

void load_menu(void) 
{ 
int choice; 
int loopagain; 

do 
{ 
    printf("Menu \n\n"); 
    printf("Please enter your choice: \n"); 
    printf("1. \n"); 
    printf("2.\n"); 
    printf("3.\n"); 
    printf("4. Exit\n"); 
    if (scanf("%d",&choice)==1) 
    { 

     switch(choice) 
     { 
      case 1: 
        break; 
      case 2: 
        break; 
      case 3: 
        break; 
      case 4: printf("Quitting program!\n"); 
        break; 
      default: printf("Invalid choice! Please try again\n"); 
        printf("\n"); 
       break; 
     } 
    } 

    else 
    { 
     printf("Characters are invalid, please enter a number: \n "); 
     if (scanf("%d",&loopagain)==1) 
      load_menu(); 
    } 

}while((choice !=4)); 
} 

爲什麼當我輸入一個字符時，這仍然給我一個無限循環？這是一個菜單（案例陳述仍需要填寫），但我正在照顧由if語句輸入的字符，但它似乎仍然不起作用。謝謝

Answer 1

如果字符輸入無效，則新調用的load_menu()中的loopagain將與其調用者中的loopagain不同。當您輸入的東西是不是一個數字，因此不被接受scanf("%d",&choice)輸入緩衝器不刷新

else 
{ 
    printf("Characters are invalid, please enter a number: \n "); 
    choice = 0; // Unused, so continue the loop 
} 

Answer 2

：不循環的。我相信在處理不可接受的輸入時，您應該能夠通過致電fflush(stdin)解決此問題。更好的是，每次調用scanf之後，您可能會更好地刷新輸入緩衝區。

在我看來，處理不正確的輸入是沒有意義的。它應該像你的default:案件一樣處理，我想。正如其他人所說的，遞歸調用沒有意義，當您要返回顯示菜單並再次獲取用戶輸入時，也不會再次調用scanf以進行輸入。

Answer 3

我相信，除了迄今爲止發現的問題之外，有問題的「字母」會卡在輸入緩衝區中。當用scanf讀取一個數字時，只要它碰到任何不是白色而不是數字的東西就會停下來。因此，如果緩衝區包含「a \ n」，並且我們調用scanf("%d", ...)，那麼scanf將返回immediatelty，並將繼續這樣做，直到從緩衝區中刪除「off」爲止。

我們需要的是一個小小的循環來從輸入緩衝區中刪除有問題的「垃圾」。

這裏是一個問題（雖然沖洗是一個稍微不同的原因，解決的辦法是一樣的）前問： Question about flushing buffer

Answer 4

我覺得你的問題是loopagain變量。通過這個名字，你正在考慮這個變量，就像一個標誌一樣循環或者不再循環，並且管理你的第二個循環的方式。既然你是從stdin（scanf）讀取它，你將失去對它的控制。 由於您的實現中已經有一個scanf，並且由於它是一個循環，所以您不需要遞歸調用，並且可以始終使用相同的scanf，並以正確的方式使用loopagain變量/標誌。 甚至更​​好的是，除了EOT（ascii-cntr-D）之外，沒有字符，它的整數值是4（它絕對不會通過scanf測試，值爲1，但仍然...） - 一般的，你可以把它看作是打破你的程序的另一種方式。

一個soluiton這是（我想，我的解釋）：

#include <stdio.h> 
    void load_menu(void); 

    int main(void) 
    { 
     load_menu(); 
     return 0; 
    } 

    void load_menu(void) 
    { 
    int choice; 
    int loopagain = 1; 

    do 
    { 
     if(loopagain != 0){ /*You'll set it to different from 0 if the user entered a 'bad' number so the menu is only printed once*/ 
      printf("Menu \n\n"); 
      printf("Please enter your choice: \n"); 
      printf("1. \n"); 
      printf("2.\n"); 
      printf("3.\n"); 
      printf("4. Exit\n"); 
     } 
     if (scanf("%d",&choice)==1) 
     { 

      switch(choice) 
      { 
       case 1: 
         break; 
       case 2: 
         break; 
       case 3: 
         break; 
       case 4: printf("Quitting program!\n"); 
         break; 
       default:printf("Invalid choice! Please try again\n"); 
         loopagain = 0; 
         printf("\n"); 
        break; 
      } 
     }else{ 
      printf("Characters are invalid, please enter a number: \n "); 
      loopagain = 0; 
     } 

    }while(choice !=4); 
} 

希望它幫助。