Проверка пользовательского ввода в C

Question

Так мне нужен ввод данные пользователя для междунар быть больше 2.

printf("Number of triangles (must be greater than 2) = "); 
    fflush(stdin); 
    scanf("%d", &num_of_triangles); 
    while (num_of_triangles < 3) // ?how to check using scanf? 
    { 
     printf("Number of triangles (must be greater than 2) = "); 
     fflush(stdin); 
     scanf("%d", &num_of_triangles); 
    } 

Есть ли возможность оптимизировать этот код в отношении повторяющихся строк?

Answer 1

Вместо while, do ... while. Таким образом, цикл будет выполняться хотя бы один раз.

num_of_triangles = 0; 
do { 
    printf("Number of triangles (must be greater than 2) = "); 
    scanf("%d", &num_of_triangles); 
    while (getchar() != '\n'); // this flushes the input buffer 
} while (num_of_triangles < 3); 

Кроме того, не fflush(stdin), так как это неопределенное поведение.

EDIT:

кажется, что Visual Studio позволяет fflush(stdin). От MSDN:

Функция fflush смывает поток. Если файл, связанный с потоком , открыт для вывода, fflush записывает в этот файл содержимое буфера, связанного с потоком. Если поток открыт для ввода , fflush очищает содержимое буфера. fflush отрицает эффект любого предшествующего вызова для отказа от потока. Кроме того, fflush (NULL) очищает все потоки, открытые для вывода. Поток остается открытым после вызова . fflush не влияет на небуферизованный поток.

В целом, однако, на это нельзя влиять. Предпочтительнее делать что-то более портативное, например, вышеуказанный код.

Answer 2

fflush(stdin) - это неопределенное поведение, поэтому вам нужно полностью удалить его и заменить что-то еще.

Вы также должны проверить выходы scanf, в случае его отказа.

Если вы хотите, чтобы улучшить читаемость, вы также можете переместить scanf в свое время цикл непосредственно, как этот

while (scanf("%d",&num_of_triangles)==1 && num_of_triangles < 3) 

Answer 3

Вы можете написать свой код, так как это -

printf("Number of triangles (must be greater than 2) = "); 

while (scanf("%d",&num_of_triangles)==1 && num_of_triangles < 3) //check return of scanf as well as number 
{ 
    printf("Number of triangles (must be greater than 2) = ");  
} 

Loop будет перебирать до scanf является успешным и num_of_triangles составляет менее 3.