Хочу узнать мнение хороших программистов на C(СИ) о моем калькуляторе, можно каких-нибудь советов?

Question

[Kaktys_DH]

Хочу узнать мнение хороших программистов на C(СИ) о моем калькуляторе, можно каких-нибудь советов? Это мой первый проект на С.

#include <stdio.h>
int main()
{

  int num_1;
  int num_2;
  int result;
  char wait[2];
  char vopros[2];
  char vopros_2[2];


do {
  printf("\n\t\t\t\t\tКАЛЬКУЛЯТОР 2.0");


  printf("\nВведите первое число: ");
  scanf("%d", &num_1);

  printf("\nВведите второе число: ");
  scanf("%d", &num_2);


printf("\nВыберите действие - +, -, *, /: ");
scanf("%s", wait);

if (wait[0] == '+') {
  result = num_1 + num_2;
}

else if (wait[0] == '-') {
  result = num_1 - num_2;
}

else if (wait[0] == '*') {
  result = num_1 * num_2;
}

else if (wait[0] == '/') {
  result = num_1 / num_2;
}

printf("%d %s %d = %d\n", num_1, wait, num_2, result);


do {
  printf("Хотите воспроизвести действие еще с одним числом?(Y/n)");
  scanf("%s", vopros_2);

  if (vopros_2[0] == 'n') {
    vopros_2[0] = 'N';
  }

  else if (vopros_2[0] != 'n' && vopros_2[0] != 'N') {
    num_1 = 0;
    num_2 = 0;
    char wait[2] = {0};

printf("Введите число\n");
scanf("%d", &num_1);


printf("\nВыберите действие - +, -, *, /: ");
scanf("%s", wait);

if (wait[0] == '+') {
  result = result + num_1;
}
else if (wait[0] == '-') {
  result = result - num_1;
}
else if (wait[0] == '*') {
  result = result * num_1;
}
else if (wait[0] == '/') {
  result = result / num_1;
}

printf("Result - %d\n", result);
  }
}
while (vopros_2[0] != 'N');


printf("Хотите попробывать еше раз? (Y/n): ");
scanf("%s", vopros);

if (vopros[0] == 'n') {
  vopros[0] = 'N';
}

}
while (vopros[0] != 'N');

return 0;
}

Comments

[Модератор]

Kaktys_DH дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац).
В противном случае ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

[Kaktys_DH]

Модератор, тега C не было

Answer 1

[CityCat4]

Ну... для первого проекта пойдет - все равно как правило первый проект - это место набивания шишек и натыкания на общие места...

Юзер - дебил! Исходят всегда из этого. Он вводит чушь. Он делает совсем не то, что от него оиждают. Он игнорирует предупреждения. ВСЕ его действия должны проверяться. ВСЕ результаты должны проверяться.

Переменные, которые будут управлять программой - обязательно инициализировать! Выборы, которые управляют программой - проверять! Что произойдет, если юзер введет любой другой символ, кроме знака операции? Правильно, SIGSEGV :) Потому что result не имеет значения, а в нелепом каскадном if обработка ситуации "юзер ввел ерунду" не предусмотрена и в printf() пойдет неинициализированная переменная :)

Избегай каскадных if, когда можно использовать case

scanf() оправдан только тогда, когда нужно преобразовать число, в твоем случае нужно использовать getchar()

Вместо усложнения кода двойным условием (n или N) стоит использовать tolower()/toupper(), чтобы условие было одно

Вычисления целочисленные и будут давать неожиданный результат с нецелыми операндами. Например 4 / 2 даст 2 (как и должно быть), но 5 / 2 - тоже даст 2!

Нет обработки операций с нулем - например при делении на нуль программа скорее всего получит SIGILL

В общем, есть над чем работать :)

Comments

[Kaktys_DH]

1.Зачем их инициализировать?
2.Case я ещё не изучал, я только до циклов дошел, просто по пути изучения делаю маленькие программы)
3.Про getchar вообще не слышал, но загуглю
4. Тоже не знаю что это - tolower()/toupper().
5. Обработку с нулем сделаю.

Когда я построил этот алгоритм мне казалось что я гений :D

[CityCat4]

Зачем их инициализировать?

Неинициализированная переменная имеет непредсказуемое значение и если лоханешься (например, делая выбор на основе ее значения) - можешь внезапно схватить SIGSEGV :)

.Про getchar вообще не слышал, но загуглю

Интересно, printf() уже знаем, а вот getchar() - нет ... Изучай библиотеку стандартного ввода-вывода. И кстати, для вывода строки тоже printf() использовать не надо - он достаточно "дорогой" в смысле кода и времени, лучше использовать puts()

Тоже не знаю что это - tolower()/toupper().

Функции перевода символа в нижний/верхний регистр. Во времена DOS можно было просто вычесть из кода фиксированное число, сейчас так может не сработать.

Обработку с нулем сделаю.

И проверку ввода юзера не забудь :)