Целочисленный результат проверки

Question

Я сам изучал C и перерабатывал программу из книги C Primer. Я надеялся, что свежий набор глаз может определить тот самый вопрос, который у меня есть. Как вы можете видеть по моему результату и ожидаемому результату, я бы хотел избавиться от строки «0 - это число». Я считаю, что проблема с циклом while является проблемой, но я не могу избавиться от нее, несмотря на те вариации, которые я пробовал.

Выход:

Enter some integers. Enter 0 to end. 
    1 two 3 0 4 
    1 is a number. 
    two is not an integer 
    3 is a number. 
    0 is a number. 

Ожидаемый результат:

Enter some integers. Enter 0 to end. 
    1 two 3 0 4 
    1 is a number. 
    two is not an integer 
    3 is a number. 

#include <stdio.h> 
#include <ctype.h> 

int get_int(void); //validate that input is an integer 

int main(void) 
{ 
    int integers; 

    printf("Enter some integers. Enter 0 to end.\n"); 
    while (integers != 0) 
    { 
     integers = get_int(); 
     printf("%d is a number\n", integers); 
    } 
    return(0); 

} // end main 

int get_int(void) 
{ 
    int input; 
    char ch; 

    while (scanf("%d", &input) != 1) 
    { 
     while (!isspace(ch = getchar())) 
      putchar(ch); //dispose of bad input 
     printf(" is not an integer\n"); 
    }  
    return input; 
}// end get_int 

Answer 1

Что я хотел бы сделать, это переместить вызов get_int в состояние время цикла:

int main(void) 
{ 
    int integers; 

    printf("Enter some integers. Enter 0 to end.\n"); 
    while ((integers = get_int()) != 0) 
    { 
     printf("%d is a number\n", integers); 
    } 
    return(0); 

} // end main 

Проблема с вашим существующим кодом заключается в том, что между вызовом get_int() и печатью значения вы не проверяете, вернул ли ваш дозор 0.

Другой вариант заключается в том, чтобы добавить if (integers == 0) { break; } состояние между , но, на мой взгляд, назначение в состоянии является более чистым.

Answer 2

Вы правы, подозреваете, что вам нужно переделать петлю while. Вы попробовали что-то вроде этого?

for (;;) 
{ 
    integers = get_int(); 
    if (integers == 0) break; 
    printf("%d is a number\n", integers); 
} 

Кроме того, ваш get_int будет лучше написана с fgets (или getline если таковые имеются) и strtol. scanf соблазнительно удобен, но почти всегда больше проблем, чем того стоит.

Answer 3

Рассмотрим ядро ​​вашего цикла:

integers = get_int(); 
printf("%d is a number\n", integers); 

Независимо от того, что get_int() возвращается, то printf линия будет выполнена. Эта линия должна отдельный if:

integers = get_int(); 
if (integers != 0) printf("%d is a number\n", integers); 

Answer 4

Самый простой способ это поставить ваше состояние и назначение в то время как петли. Прямо сейчас ваш код полагается на integer, который устанавливается в цикле, а затем петли снова, чтобы проверить, равен ли он нулю.

while((integer = get_int()) != 0) 

Позволяет вам проверять одновременно с назначением целого числа. Не забудьте скобки, или ваше целочисленное значение будет результатом integer = (get_int != 0), потому что != имеет более высокий приоритет, чем = в C и C++.