Code ignoriert IF-Befehl in C, Probleme mit FOR

Question

Ich hatte hier zuvor gepostet, aber ich war nie in der Lage, die Hilfe zu bekommen, die ich brauchte. Ich arbeite an einem Schulprojekt und ich kann mein Programm nicht richtig funktionieren lassen.

Das Programm sollte den Benutzer auffordern, die Anzahl der verwendeten Gallonen einzugeben, und die Anzahl der Meilen für jeden der 3 Tanks von Gas gefahren. Das Programm sollte dann berechnen und die Meilen pro Gallone anzeigen, die für jeden Tank erhalten werden. Sobald die Verarbeitung für die 3 Tanks abgeschlossen ist, berechnet das Programm die Gesamtmeilenzahl von (Gesamtgallonen/Gesamtmeilen) und zeigt eine freundliche "Auf Wiedersehen" -Nachricht an.

Das Problem, das ich habe, ist, dass ich es nicht zu OVERALL Millage anzeigen kann. Es endet nach dreimaligem Looping. Ich weiß, dass verschiedene Schleifenanweisungen Bedingungen erfüllen müssen, aber ich kann die FOR-Schleife nicht richtig funktionieren lassen. Ich werde wirklich frustriert, weil ich weiß, dass das nicht so schwer sein sollte.

-Code

#include <stdio.h> 
int main(void) 
{ 
    int miles,i=3; 
    float gallons, mg, overall = 0, avg = 0; 
    while(i>0) 
    { 
     printf("Enter the gallons used: "); 
     scanf("%f", &gallons); 
     printf("Enter miles driven: "); 
     scanf("%d", &miles); 
     mg = miles/gallons; 
     printf("The miles/gallon for this tank was : %f\n", mg); 
     overall += miles; 
     avg += gallons;i--; 
    } 

    if(gallons == 0) 
    { 
     printf("\n\n The overall miles/gallon was: %f\n", overall/avg); 
     exit(0); 
    } 
    return 0; 
} 

Answer 1

Wenn ich Ihren Code richtig gelesen, was dann von der Gesamtlaufleistung verhindert gedruckt wird, ist die folgende endgültige if Aussage:

if (gallons == 0) 

Wenn Sie es entfernen, dann die Gesamtkilometer sollte gedruckt werden. Verwenden Sie dieses Muster:

while (i > 0) 
{ 
    // your while loop here 
} 

printf("\n\n The overall miles/gallon was: %f\n", overall/avg); 
exit(0); 

Answer 2

Hinzufügen zu Tim Biegeleisen' s Antwort:

mg = miles/gallons; 

Was passiert, wenn gallons gleich zu 0? z.B. 0 Meilen für 0 Gallonen
Dies wird zu Gleitkommaausnahme führen.

Ein einfaches if-else kann dieses Problem lösen!

if(!gallons) 
    mg = 0; 
else 
    mg = miles/gallons; 

Answer 3

Dieser If (if (gallons == 0) {}) Block ist außerhalb der While-Schleife. Zuerst müssen Sie die if-Schleife innerhalb der while-Schleife verschieben. und dies sollte Bedingung für Variable i wie folgt sein und nicht für Gallonen.

if (i == 0) 
{ 
    printf("\n\n The overall miles/gallon was: %f\n", overall/avg); 
} 

In diesem Fall nach 3 Iteration Wert von i wird 0, so wird es in den Block, wenn eingeben und berechnet und die Gesamt Meilen/Gallone drucken.

Answer 4

sollten Sie die if (gallons == 0) { /* print something that says gallons can't be 0 and skip the mg = miles/gallons to avoid a divide by zero ! */ }

habe ich das für nicht-Schleife sehen konnte, auf dass jede Beratung zu geben.

wo ist die Berechnung für avg = Gesamt/avg

if (gallons == 0) { /* print something that says gallons can't be 0 
and skip the mg = miles/gallons to avoid a divide by zero ! */ 
} 
if (gallons == 0) { /* print something that says gallons can't be 0 
and skip the mg = miles/gallons to avoid a divide by zero ! */ 
} 
enter code here 

and then printing the avg ? `printf();` 

if (gallons == 0) { /* print something that says gallons can't be 0 
and skip the mg = miles/gallons to avoid a divide by zero ! */ 
} 

`