Wie validiere ich richtig wie viele Eingaben in scanf gelesen wurden?

Question

Wie kann ich richtig sehen, wie viele Eingänge in meiner scanf Funktion gelesen wurden? wenn ich den folgenden Code ausführen es das Ergebnis nicht angezeigt werden, ist dies, weil ich nicht haben 2 Eingänge von scanf oder aus einem anderen Grund lesen hier ist mein Code:

#include <stdio.h> 

int main() 
{ 
float numberOne; 
float numberTwo; 

scanf("%f %f", &numberOne, &numberTwo); 
float result = numberOne + numberTwo; 

int howManyRead = scanf("%f %f", &numberOne, &numberTwo); 

if (howManyRead == 2) 
{ 
    printf("%f", &result); 
} 
else 
{ 
    printf("invalid input"); 
} 
sleep(10); 

} 

Answer 1

Sie haben zwei Anrufe von scanf in Ihrem Code . Das Ergebnis des ersten Aufrufs wird ignoriert, während das Ergebnis des zweiten Aufrufs überprüft wird.

Wenn Sie zwei Zahlen eingeben, gibt die erste scanf 2 zurück, die Ihr Code ignoriert. Danach versucht der Anruf an die zweite scanf zwei weitere Nummern zu lesen.

Sie können dieses Problem beheben, indem Sie den ersten Aufruf scanf Entfernung:

float numberOne, numberTwo; 
if (scanf("%f %f", &numberOne, &numberTwo) == 2) { 
    float result = numberOne + numberTwo; 
    printf("%f", result); 
} else { 
    printf("invalid input"); 
} 

Answer 2

Sie nicht den Rückgabewert von scanf() in der erwarteten Art und Weise verwenden. Es ist da, um den Erfolg/Misserfolg beim Scannen zu identifizieren und eine Entscheidung darüber zu treffen.

Es gibt drei Dinge, die Sie tun müssen.

• Entfernen Sie die beiden Linien

  scanf("%f %f", &numberOne, &numberTwo); 
  float result = numberOne + numberTwo; 
  

  Dies liegt daran, ohne zu überprüfen, ob Sie die Zielvariablen zu verwenden versuchen, kann es Werte im Fall unbestimmt scanf() ausfällt. Auch gibt es wiederholt scanf() s, die einfach falsch sind und nicht benötigt werden.

• Fügen Sie die Zeile float result = numberOne + numberTwo; in den Bedingungsblock if (howManyRead == 2) ein.

• die & vom printf Aufruf entfernen: printf("%f", result);