Mercury13

'0' — это не число 0, а код символа «0». Во всех машинах, кроме совсем уж мамонтов, используются надмножества кодировки ASCII, и '0' = 48.

И в ASCII, и в EBCDIC, и во внутренних кодировках, скажем, ЖК-индикаторов коды цифр идут последовательно от 0 до 9 — так что, отняв от кода цифры код символа «0», получаем числовое значение цифры.
То есть ,

'0' - '0' = 48 - 48 = 0
'1' - '0' = 49 - 48 = 1
...
'9' - '0' = 57 - 48 = 9

(цифры справедливы для ASCII и его надмножеств, но в EBCDIC аналогично)

В компьютере вообще нет понятия «символ», и что такое 48 — сорок восемь попугаев, X-координата на экране или код символа '0' — программист или компилятор должен думать сам. Так что Си (который, по сути, кроссплатформенный ассемблер), а ним и Java ставят очень эфемерную грань между одним и другим. В отличие от Паскаля, где надо писать Ord(c) - Ord('0').

Это дело сложное. И я бы разделил его на три перекрывающихся этапа.
1. Увидеть в строке уравнение от одной переменной. Например, в таком виде.

2. Увидеть в том, что получилось, линейное уравнение.
3. Запустить модуль «Линейное уравнение», который приведёт подобные слагаемые и высчитает ответ.

Есть способы, но они периодически меняются.
Это политика Microsoft: программы не должны засирать эти экраны.
https://blogs.msdn.microsoft.com/oldnewthing/20030...

Несмотря на то, что массив часто отождествляют с указателем, массив — это НЕ указатель. У него, например, другие операции приводят к неопределённому поведению.

И самое противное в Си — это то, что в коде
void sort(int x[5]);
x — это не массив, а именно что указатель. Чтобы был массив, надо Си++
void sort(int (&x)[5]);
И компилятор даже подсвечивает, что параметр функции, отмеченный как массив — всегда указатель.

typedef int Arr[SIZE];
int sum(Arr arr, int n)
// warning: 'sizeof' on array function parameter 'arr' will return size of 'int *' [-Wsizeof-array-argument]|

Попробуйте Brush.Color := $FF000000 or clBlue;. Велика вероятность, что механизмы работы с 32-битными изображениями тут слабоваты.

https://ru.wikipedia.org/wiki/Условия_Йоды
Аж в двух местах.

Причём в Си не нужно ставить в сложных условиях скобки, как в Паскале, вполне покатит такое…

if ((a >= 11 && a <= 14) || m == 0 || (m >= 5 && m <= 9))

Но нет, вы наставили лишних скобок и заглушили ошибку.

Также, что мне не нравится.
Вместо 0,1,2,3 лучше использовать константы MANY/FEW/ONE/UNKNOWN.

В case 1 символ подстановки плохо содран с доски.

Если условие на MANY самое сложное и выполняется по остаточному принципу — почему его делаем самым первым? Лучше что-то типа…
• 10…20 → MANY
• 1 → ONE
• 2…4 → FEW
• Иначе MANY

Единственно правильный способ — tg(π/2 − x).
Дело в том, что 1/tg будет глючить в районе 90°.

Причина в другом. Если человек введёт −1 пройденного пути, надо сразу break, а не проходить огрызок цикла. Пока я проверял, ns 5d выписал действующий код.

Ну и, конечно, ошибка в логике: среднее потребление (в mpg) = сумма миль / сумму галлонов.

UPD. Раз уж не хотите break, то…

while(petrol != -1)
    {
        printf("Enter the gallons used (-1 to end): ");
        scanf("%f", &petrol);
        if(petrol != -1)  {
           // Тут делай ВСЕ вычисления по MPG
        }
   }

У меня всё сработало. Может, зависит от библиотеки Си? Какой компилятор?

UPD. Сработало на MinGW.

Заводим структуру-сумматор. Изначально она 0/1. Что-то типа

typedef struct {
  int num, den;
} Fraction;

…

Fraction sum = { 0, 1 };

Дальше пишу псевдокодом, НОД и НОК разберись, как реализовать самому.

для каждой очередной дроби
  ввести(дробь)
  новыйЗнам = НОК(сумматор.знам, дробь.знам)
  множСумм = новыйЗнам / сумматор.знам
  множДробь = новыйЗнам / дробь.знам
  новыйЧисл = сумматор.числ × множСумм + дробь.числ × множДробь
  сокрНод = НОД(новыйЧисл, новыйЗнам)
  сумматор.числ = новыйЧисл/сокрНод
  сумматор.знам = новыйЗнам/сокрНод

Можно и оптимизировать, но тогда нужна хитрая разновидность алгоритма Евклида; в учебных целях не стоит.

И ещё. Одна из известных ошибок в написании НОК — произведение аргументов не влезает в свой тип, а НОК влезает. Поймёшь, как в такой ситуёвине избавиться от переполнения?

Очевидно, расширение GNU.
MinGW, даже с ключом -std=c99, OK.

Причём работает только в режиме Си. MinGW/Cи++
C:\TestApps\NestedFunction3\main.cpp|7|error: a function-definition is not allowed here before '{' token|

Embarcaredo x86 (Borland):
[bcc32 Error] File1.c(7): E2141 Declaration syntax error

Embarcadero x64 (clang):
[bcc64 Error] File1.c(6): expected ';' at end of declaration

Будет подставлено ИМЯ ПЕРЕМЕННОЙ b, а чему она равна — это уже другой вопрос. Никаких проблем не должно быть.

#include <iostream>

#define CHECK(a,b) \
   if ((a) == (b))     \
      std::cout << "Equal" << std::endl;  \
      else std::cout << "Inequal" << std::endl;

int main()
{
    for (int i = 0; i < 5; ++i) {
        CHECK (i, 2)
    }
    return 0;
}

После препроцессирования программа превратится вот во что…

int main()
{
    for (int i = 0; i < 5; ++i) {
        if ((i) == (2))
           std::cout << "Equal" << std::endl;
           else std::cout << "Inequal" << std::endl;
    }
    return 0;
}

Вывод в консоли

Inequal
Inequal
Equal
Inequal
Inequal

Разумеется, нехороши ситуации, когда в макрос подставляется код с побочными эффектами, но это другой вопрос.

Переполняется longitude, говорите? Ну убедитесь, что он размера SIZE_J. Не этой функции, правда, дело.
Кроме того, вы массивы latitude[] и longitude[] заполняете случайным мусором, появившимся в результате malloc.