Бесконечный цикл при вводе недопустимого ввода

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); 
    } 
}