Правильное использование scanf() в C, неправильный вывод

Question

Я пытаюсь создать программу, которая вычисляет среднее значение любого желаемого числа поплавков до EOF. Программа также должна проверить правильность ввода и вернуть «Неверный ввод», когда, например, ввод строки. Код, который я написал, работает, но он дает неправильные выходы для среднего. Может ли кто-нибудь сказать мне, почему?

#include <stdio.h> 

int main(void) { 
    int times = 0; 
    float sum = 0; 
    float scan; 
    float avrg; 
    int scanvalue = 1;  
    while (scanvalue == 1) {    
     scanvalue = scanf("%f", &scan); 
     sum = sum + scan; 
     times++;     
    }  
    if (scanvalue == EOF) { 
     avrg = sum/times; 
     printf("The average is %f\n", avrg); 
    } else { 
     printf("Wrong input"); 
    } 
    return 0; 
} 

С уважением.

Answer 1

Вы не проверяете scanvalue после scanf() и по-прежнему используете значение scan, которое встряхивает средний показатель. Обратите внимание, что когда scanf() возвращает EOF, он не будет изменять scan, и поэтому он все равно будет иметь последнее значение, поэтому вы добавляете последнее значение дважды.

Но если вы ввели неверный ввод в начале, то поведение не определено, изменить его

while ((result = scanf("%f", &value)) == 1) { 
} 

Кроме того, я сознательно изменил имена ваших переменных, чтобы показать лучший способ назвать их.

Answer 2

Вам необходимо проверить значение scanvalue после scanf, в противном случае время будет увеличиваться, даже если вы введете EOF, и сумма получит неправильный ввод, что приведет к тому, что среднее значение будет неправильным/беспорядочным.

Таким образом, вы должны изменить

while (scanvalue == 1) {    
    scanvalue = scanf("%f", &scan); 
    sum = sum + scan; 
    times++;     
} 

Для

while (scanvalue == 1) { 
    scanvalue = scanf("%f", &scan); 
    if (scanvalue != 1) 
     break; 
    sum = sum + scan; 
    times++; 
}