Оцените

[ACE Valery]

Просто любопытно. Почему '+' принимается в функции как int? Если код вашей функции будет скрыт от меня, я никогда не догадаюсь, что в нее надо передать, тем более, что имена принимаемых переменных не отражают своей сути. А еще мне не понятно зачем возвращать int, если вы возвращаете printf("%d\n", b); Ну нелогично же блин... Опять же, не видя кода функции я подумаю, что она возвращает число и выведу на печать его после вызова этой функции. И выведется оно два раза. А я буду голову ломать, откуда оно у меня там дважды взялось.

[ArmanPrestige]

Зря столько ифов. использовал бы switch. плюс я думаю лучше было бы сделать так

case '/': 
      return i/c

нет необходимости в локальной переменной.
и еще - деление будет целочисленное, если не заметил.

[RussDragon]

Да говорил же, писал от нечего делать, INT у + потому, что он возвращает цифровое значение символов, да, это можно переделать и в другой тип, но зачем, если это чисто для себя?=)

[ACE Valery]

тогда зачем спрашивать у форумчан, хорош ли код?

[RussDragon]

То что написано за 5 минут, тоже имеет значение)

[Stilet]

тоже имеет значение)

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

P.S. Хотя я бы конечно его переписал более читабельно и общепринято.

[ArmanPrestige]

Если что то делаешь, то делать это надо либо качественно, либо вообще не делать.

[three_cats]

Сообщение от ArmanPrestige

Если что то делаешь, то делать это надо либо качественно, либо вообще не делать.

Либо делать, но молча.

[zipo666]

Заместо ифов вообще можно юзать (...)?....:....

[RussDragon]

#include <stdio.h>
int CALCULATE(char, char, char);

int main()
{
    int c, i, g;
    char calc[3];
    for (i = 0; i<4; ++i) {
        calc[i] = 0;
    }
    g = 0;
    
    while((c = getchar()) != EOF)
    {
    
        if (c >= '0' && c <= '9') {
            if(calc[g] == 0){
            calc[g] = c;
            g++;
                
            }
        }
        else if(c == '-' || c == '+' || c == '*' || c == '/')
        {
            if(calc[1] == 0)
            {
            calc[1] = c;
            g++;
            }
        
        }
        
    }
    printf("calc0 %c, calc1 %c, calc2 %c\n", calc[0], calc[1], calc[2]);
 CALCULATE(calc[0], calc[1], calc[2]);
    return 0;
}


int CALCULATE(char n1, char n2, char n3) {
    int b;
    n1 -= '0';
    n3 -= '0';
    switch (n2) {
        
        case '+':
            b = n1 + n3;
            break;
        case '-':
            b = n1 - n3;
            break;
        case '*':
            b = n1 * n3;
            break;
        case '/':
            b = n1 / n3;
            break;
    }
    return printf("%d", b);
    
}

Ловите, привереды=)
Теперь тот же калькулятор, с тем же не работающим делением, но который считает то, что пришлось на вход=) Пока что однозначные числа, потому что лень было заморачиваться с большими массивами=)
UPD:
printf("calc0 %c, calc1 %c, calc2 %c\n", calc[0], calc[1], calc[2]);
Не обращайте внимания, она только для отладки)