Korrekte Verwendung von scanf() in C, falsche Ausgabe

Question

Ich versuche, ein Programm zu entwerfen, das den Durchschnitt einer beliebigen Anzahl von Gleitkommazahlen bis EOF berechnet. Das Programm sollte auch prüfen, ob die Eingabe korrekt war und "Falsche Eingabe" zurückgeben, wenn z.B. Eingabe einer Zeichenfolge Der Code, den ich geschrieben habe, funktioniert, aber es gibt falsche Ausgaben für den Durchschnitt. Kann mir jemand sagen warum?

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

Mit freundlichen Grüßen.

Answer 1

Sie überprüfen scanvalue nicht nach scanf() und verwenden immer noch den Wert scan, der den Durchschnitt durcheinander bringt. Beachten Sie, dass scanf()EOFzurückgibt und es wird daher seinen letzten Wert weiterhin haben, so dass Sie den letzten Wert zweimal hinzufügen.

Aber wenn man am Anfang ungültige Eingabe und geben Sie dann das Verhalten nicht definiert ist, ändern Sie ihn auf

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

Auch ich geändert absichtlich die Namen Ihrer Variablen zu benennen sie einen besseren Weg zu illustrieren.

Answer 2

Sie müssen scanvalue nach dem scanf überprüfen, sonst werden die Zeiten inkrementiert, auch wenn Sie EOF eingeben und die Summe eine falsche Eingabe erhält, was dazu führt, dass der Durchschnitt falsch/durcheinander ist.

So sollten Sie

ändern

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

Um

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