Код игнорирует команду IF в C, проблемы с FOR

Question

Я уже писал здесь, но я никогда не мог получить помощь, в которой я нуждался. Я работаю над школьным проектом, и я не могу заставить свою программу работать правильно.

Программа должна предлагать пользователю ввести количество используемых галлонов и количество миль, рассчитанных на каждый из 3-х танков газа. Программа должна , а затем вычислить и отобразить мили за галлон, полученные для каждого резервуара. После завершения обработки для 3-х танков программа рассчитает общий пробег (общий галлон/всего миль) и отобразит дружеское сообщение «До свидания».

Проблема, с которой я сталкиваюсь, заключается в том, что я не могу ее отобразить для ОБЩЕЙ МИГРАЦИИ. он заканчивается после цикла 3 раза. Я знаю, что разные операторы цикла требуют условий для выполнения, но я не могу заставить цикл FOR работать правильно. Я очень расстраиваюсь, потому что знаю, что этого не должно быть так сложно.

Код

#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

Если я прочитал ваш код правильно, то, что мешает общий пробег от печатаемого следующего окончательного if заявления:

if (gallons == 0) 

Если вы удалите его, то общий пробег необходимо принт. Используйте этот шаблон:

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

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

Answer 2

Добавление Tim Biegeleisen' сек ответ:

mg = miles/gallons; 

Что делать, если gallons равно 0? например 0 миль за 0 галлонов
Это приведет к исключению с плавающей запятой.

Простой if-else может решить эту проблему!

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

Answer 3

Это если (if (gallons == 0) {}) блок вышел из цикла. Во-первых, вам нужно переместить цикл if внутри цикла while. , и это условие должно быть для переменной i следующим, а не для галлонов.

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

В этом случае после 3 итерации, значение I будет 0, так что будет входить в блок, если и рассчитать и распечатать общие мили/галлон.

Answer 4

вы должны поставить 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 ! */ 
} 
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 ! */ 
} 

`