Почему компилятор пишет, что типы не совпадают?

Question

[ваня бро]

Я написал такой калькулятор

Но при попытке компиляции он выдаёт данную ошибку:

main.c:24:5: error: conflicting types for 'calculate'
int calculate(int n1, int n2, char oper)

Более того, имеется ещё одна проблема: когда я пробовал скомпилировать программу без функции вычисления результата примера, то после ввода значений первых двух переменных (чисел), программа не давала ввести мне действие (типа char), сразу же обрываясь.

#include <stdio.h>

int num1, num2;
char op;

int calculate();

int main()
{
    printf("First number >> ");
    scanf("%d", &num1);

    printf("Second number >> ");
    scanf("%d", &num2);

    printf("Action >> ");
    op = getchar();

    calculate(num1, num2, op);

    return 0;
}

int calculate(int n1, int n2, char oper)
{
    switch(oper) {
        case '+':
            return n1 + n2;
            break;
        case '-':
            return n1 - n2;
            break;
        case '*':
            return n1 * n2;
            break;
        case ':':
            return n1 / n2;
            break;
        default:
            return 0;
            break;
    }
}

Comments

[Дмитрий]

Калькулятор не точный получится, и проверки деления на 0 нет

Answer 1

[Hemul GM]

Потому что ты определил функцию вверху без параметров int calculate();

Comments

[ваня бро]

Спасибо. Но я всё равно не могу ввести действие

[jcmvbkbc]

я всё равно не могу ввести действие

ваня бро, пробел добавь во вторую форматную строчку: scanf("%d ", &num2);
Без пробела в action попадает символ конца строки который ты вводишь чтобы закончить num2.

[jcmvbkbc]

Потому что ты определил функцию вверху без параметров int calculate();

Hemul GM, объявил, а не определил.

[ваня бро]

jcmvbkbc,
Возникла другая проблема: scanf срабатывает раньше, чем printf, хотя последний прописан раньше

[70-C9-4E-54-03-2F]

ваня бро,

int main()
{
    printf("First number >> ");
    scanf("%d", &num1);

    printf("Second number >> ");
    scanf("%d", &num2);

    fflush(stdin);

    printf("Action >> ");
    op = getchar();

    printf("%d\n", calculate(num1, num2, op));

    return 0;
}

Следует добавить fflush(stdin), после второго scanf. Таким образом перед тем, как будет запустится getchar буфер ввода очистится и никакие "\n" в переменную op туда не попадут :)

[70-C9-4E-54-03-2F]

jcmvbkbc, не совсем понятно зачем в scanf("%d ", &num) нужен пробел. scanf будет бесконечно ожидать ввода любого непробельного символа после ввода num. Поэтому программа, как бы, замирает. Можно попробовать написать:
scanf("%d%*c, &num);
Тогда пробельный символ (включая символ перехода строки) прочтется, но проигнорируется и никуда не запишется.
Но лучше, скорее всего, очистить буфер используя fflush(stdin), как я указал выше.

[jcmvbkbc]

лучше, скорее всего, очистить буфер используя fflush(stdin), как я указал выше.

70-C9-4E-54-03-2F, man fflush говорит, что это не работает для терминального ввода. И это подтверждается практикой.

не совсем понятно зачем в scanf("%d ", &num) нужен пробел. scanf будет бесконечно ожидать ввода любого непробельного символа после ввода num.

Да, лучше было бы выкинуть оттуда пробел, а getchar() заменить на scanf("%*[^+-*:]%1[+-*:]", &op);

[70-C9-4E-54-03-2F]

jcmvbkbc, у меня fflush(stdin) прекрасно работает в командной строке, все работает, как ожидается и '\n' не попадает в getchar.

[jcmvbkbc]

70-C9-4E-54-03-2F, я вас поздравляю, вы наблюдаете поведение не описанное стандартом, такое бывает, важно не принимать исключение за правило и не рассчитывать, что так будет всегда.

[70-C9-4E-54-03-2F]

jcmvbkbc, так ваш вариант с пробелом после спецификатора формата в scanf не работает должным образом, тогда уже добавить вместо getchar(),scanf(" %c", &op); и да будет вам счастье с ожидаемым поведением, описанное стандартом и чем-угодно. Важно одно: то, что вы предложили не работает.

[jcmvbkbc]

так ваш вариант с пробелом после спецификатора формата в scanf не работает должным образом

70-C9-4E-54-03-2F, я знаю. И я написал вариант на замену который считаю более хорошим, вот здесь.

и да будет вам счастье с ожидаемым поведением, описанное стандартом и чем-угодно

Вы поймите, я же не просто так это написал. Я поделился практическим наблюдением, что fflush(stdin) не работает и показал, что этому есть обоснование в мануале (кстати, в стандарте C99 вообще написано, что fflush для потока ввода -- UB). Не нужно из-за этого чувствовать себя уязвлённым, так работает этот сайт: люди указывают вам на ваши ошибки, а вы им -- на их.

[70-C9-4E-54-03-2F]

jcmvbkbc, хорошо, за fflush спасибо, одно я понял точно: универ, где я учусь, параша: нас учили использовать в таких случаях fflish(stdin) и у всех как ни странно работало. Мануал позже почитаю, вы работаете с какой операционной системы или что у вас за терминал?
Уязвленным себя не ощущал. Грубо всего лишь ответил, так как мне казалось, что вы высокомерите, за это попрошу прощения)

[Hemul GM]

70-C9-4E-54-03-2F, может хватит уже тут спамить?

[jcmvbkbc]

одно я понял точно: универ, где я учусь, параша: нас учили использовать в таких случаях fflish(stdin)

70-C9-4E-54-03-2F, по хорошему универ вообще не должен "учить" деталям такого уровня. Но, увы, ровно то же самое говорили и мне в студенчестве в моём универе, 25 лет назад.

и у всех как ни странно работало

Я помню у меня тоже работало, в Borland C для MS/DOS.

с какой операционной системы или что у вас за терминал

Уверен, что имеет значение только то, какая используется libc -- в моём случае это glibc 2.36, но для полноты картины: моя ОС -- Debian 12.

[Hemul GM]

moderator2 тут спам не по теме

[Модератор]

Hemul GM, это не аккаунт модератора и это не спам.

[Hemul GM]

Модератор, это спам. Ответ давно получен ТС. А это оффтоп, на который я получаю почту на каждую реплику. Это не относится к вопросу никаким образом.