無限循環時輸入無效輸入

Question

我是新來C編程，我想檢查我的所有數組元素是除第一個元素以外的整數。

我寫了下面的代碼，但是一旦插入錯誤的輸入，循環就不會停止。

bool validate_int(char input[]){ 
    fgets(input,10, stdin); 
    for(int i = 1; i < strlen(input); ++i) { 
     if(!isdigit(input[i])){ 
      i = 1; 
      fgets(input,10, stdin); 
     } 
     else{ 
     } 
    } 
    return true; 
} 

Answer 1

您的代碼有一些小問題。這裏是一個更好的辦法來做到這一點（未經測試）：

bool validate_int(char input[]) /* Bad function name; See @Filipe's comment */ 
{ 
    for(;;) /* Infinite loop */ 
    { 
     if(fgets(input, 10, stdin) == NULL) /* If fgets failed */ 
     { 
      puts("fgets failed"); 
      return false; 
     } 

     int i, len = strlen(input); 

     if(len > 0 && input[len - 1] == '\n') /* If there is a newline character at the end of input */ 
      input[--len] = '\0'; /* Replace the '\n' with '\0' and decrement len */ 

     if(!isalpha(input[0])) /* If the first character of input is not an alphabet */ 
      continue; /* Loop again */ 

     if(len == 1) /* There is no number */ 
      continue; 

     for(i = 1; i < len; ++i) 
     { 
      if(!isdigit(input[i])) /* If not a digit */ 
       continue; /* Loop again */ 
     } 

     break; /* Get out of the loop */ 
    } 

    return true; 
} 

更更好的方式是獨立的輸入和驗證分爲兩個獨立的功能（未經測試）：

bool getInput(char input[]) 
{ 
    if(fgets(input, 10, stdin) == NULL) /* If fgets failed */ 
    { 
     puts("fgets failed"); 
     return false; 
    } 

    int len = strlen(input); 

    if(len > 0 && input[len - 1] == '\n') /* If there is a newline character at the end of input */ 
     input[--len] = '\0'; /* Replace the '\n' with '\0' and decrement len */ 

    return true; 
} 

bool validate(char input[]) 
{ 
    if(!isalpha(input[0])) /* If the first character of input is not an alphabet */ 
     return false; 

    int i, len = strlen(input); 

    if(len == 1) /* There is no number after the character */ 
     return false; 

    for(i = 1; i < len; ++i) 
    { 
     if(!isdigit(input[i])) /* If not a digit */ 
      return false; 
    } 

    return true; 
} 

和在調用功能（再次，未經測試），

char input[10]; 
if(getInput(input)) 
{ 
    if(validate(input)) 
    { 
     puts("Input is in correct format"); 
    } 
    else 
    { 
     puts("Input is in wrong format"); 
    } 
} 
else 
{ 
    puts("Failed to get input"); 
} 

Answer 2

試試這個

bool validate_int(char input[]){ 
    bool valid; 

    do{ 
     valid = false; 
     fgets(input,10, stdin); 
     for(int i = 1; input[i] && input[i] != '\n'; ++i) { 
      if(!isdigit(input[i])){ 
       valid = false; 
       break; 
      } else { 
       valid = true; 
      } 
     } 
    }while(!valid); 
    return true; 
} 

Answer 3

這裏是另一種方法，我會採取，這是必須清潔IMO：

這是你的驗證功能：

bool customValidation(char *string) 
{ 
    int len = strlen(string); 
    if (!isalpha(string[0]) || (len > 1 && !isdigit(string[1]))) 
     return false; 

    for (int i = 1; i < len && string[i] != '\n'; ++i) 
     if (!isdigit(string[i])) 
      return false; 

    return true; 
} 

這是你將如何使用它：

char input[10]; 
do 
{ 
    fgets(input, 10, stdin); 
} while (!customValidation(input)); 

顯然你應該重命名customValidation（）爲更重要的東西。

Answer 4

檢查：

int a_Length = 10; 
char input[a_Length]; 
fgets(input,a_Length, stdin); 
for(int i = 1; i < strlen(input); ++i) { 
    if(!isdigit(input[i])&& input[i]!='\n'){ 
     i = 0; 
     printf("Again try: "); 
     if (input[a_Length - 1 ]!='\n') 
      getchar(); 
     fgets(input,10, stdin); 
    } 
}