Оцените код

@EvilViking · Регистрация: 29.10.2019

Короче первый код который я забацл сам считай без помощи интернета!
Теперь пришло время его показать хоть кому-то. Если посмотреть по struct я даже сам не знал что это будет. Его я(struct) оставил просто так. Думаю если я всё просто переведу во float я бы все ещё смог бы использовать его(код).
Прошу хоть как-то оценить мой (со всем)маленький труд.

Ожидаю критики... любой)

Кстати можно ли назвать "(0 * i)" костылем или просто тефтелька в спагетти? ))

C++

#include <iostream>
using namespace std;
;struct Choto{
    //const float pi = 3.14;
    int num_1;
    int num_2;
};
  int main(int argc, const char * argv[]){
    Choto p;
    float i,u,o;
    cout << "Выбери действие: Выйти или начать." << "\n" << " " << "\n" << "9. Начать. " << "\n" << " " << "\n" << "0. Выйти. " << "\n" << " " << endl;
    cin >> i;
    cout << "Вы зашли в калькулятор!" << "\n" << " " << endl;
while (i){
 if (i == 9) {
 
        while(i){
        cout << "\n" << " " << "Введите первое число" <<endl;
        cin >> o;
        cout << "Введите второей число" <<endl;
        cin >> u;
        cout << "Выбибери действие:" <<  "\n" << " " << "1. +" <<  "\n" << " " << "2. -" <<  "\n" << " " << "3. *" <<  "\n" << " " << "4. /" << "\n" << endl;
        cin >> i;
    //p.pi;
    p.num_1 = o;
    p.num_2 = u;
    if(i){
        if(i == 1){
            float i = (0 * i) + p.num_1 + p.num_2; // p.pi;
                cout << i << "\n" << " " << endl;
}
        if(i == 2){
            float i = (0 * i) + p.num_1 - p.num_2; // p.pi;
                cout << i << "\n" << " " << endl;
}
        if(i == 3){
            float i = (0 * i) + p.num_1 * p.num_2; // p.pi;
                cout << i << "\n" << " " << endl;
}
        if(i == 4){
            float i = (0 * i) + p.num_1 / p.num_2; // p.pi;
                cout << i << "\n" << " " << endl;
        }
    else
        if(i > 4)
            cout << "Нет такого действия!!!" << endl;
    }
    cout << "Вы хотите продлжить? " << "\n" << "9.да " << "\n" << "" << "\n" << "0.нет " << "\n" << "" << endl;
    cin >> i;
    if(i == 9){
        continue;
    }
    if(i == 0){
        break;
 
        }
    }
}
}
    return 0;
}

СПАСИБО))))

@oleg-m1973 · 29.10.2019, 18:20

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

Кстати можно ли назвать "(0 * i)" костылем или просто тефтелька в спагетти? ))

Зачем это у тебя?

Алексей1153 · 29.10.2019, 18:29

EvilViking, начни с человеческого форматирования кода ))
Инициализацию переменных тоже обязательно делать

@TheCalligrapher · 29.10.2019, 18:37

Форматировать код вас не учили?

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

C++

1	;struct Choto{

Что здесь делает эта ;?

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

C++

1 2	Choto p; float i,u,o;

Почему вдруг объявления почти всех переменных вынесены в начало функции?

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

C++

float i =

Почему в программе использован тип float, когда все вычисления целочисленны?

Также: повторное объявление переменной с уже "занятым" именем - плохая идея. Вы уже объявляли переменную i выше.

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

C++

  while (i) {
    if (i == 9) {
      ...
    }
  }

Для любого ненулевого i, кроме 9 это - бесконечный цикл. Что, если пользователь введет 5?

И использование для таких целей переменной типа float - плохая идея.

Алексей1153 · 29.10.2019, 18:37

вот тоже интересный случай

C++

1	float i = (0 * i) + p.num_1 + p.num_2; // p.pi;

Байт · 29.10.2019, 18:42

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

можно ли назвать "(0 * i)" костылем или просто тефтелька в спагетти?

Это можно назвать нелепостью, если не бредом.
Тем не менее, желаю вам удачи на выбранном трудном пути!

@liv · 29.10.2019, 18:55

EvilViking, чуток причесал Ваш код:

C++

#include <iostream>
using namespace std;
struct Choto {
    //const float pi = 3.14;
    float num_1;
    float num_2;
};
int main(int argc, const char * argv[]) {
    Choto p;
    int i = 1;
    while (i) {
        cout << "Выбери действие: Выйти или начать." << endl << "9. Начать. " << endl << "0. Выйти. " << endl;
        cin >> i;
 
        if (i == 9) {
            cout << "Вы зашли в калькулятор!" << endl;
            while (i) {
                cout << "Введите первое число: ";
                cin >> p.num_1;
                cout << "Введите второе число: ";
                cin >> p.num_2;
                cout << "Выбери действие:" << endl << " " << "1. +" << endl << " " << "2. -" << endl << " " << "3. *" << endl << " " << "4. /" << endl;
                cin >> i;
                //p.pi;
                switch (i)
                {
                    case 1:
                        cout << p.num_1 + p.num_2 << endl;
                        break;
                    case 2:
                        cout << p.num_1 - p.num_2 << endl;
                        break;
                    case 3:
                        cout << p.num_1 * p.num_2 << endl;
                        break;
                    case 4:
                        cout << p.num_1 / p.num_2 << endl;
                        break;
                    default:
                        cout << "Нет такого действия!!!" << endl;
                }
                cout << "Вы хотите продолжить? " << endl << "9.да " << endl << "" << "0.нет " << endl;
                cin >> i;
            }
        }
    }
    return 0;
}

@EvilViking · 29.10.2019, 19:52 **[ТС]**

oleg-m1973, Я сделал чтобы 9 не шла в решение) Просто не знал как решить это. Сейчас я вроде поверхностно понял как работает switch-case теперь я убрал этот кусок))

Добавлено через 3 минуты
liv, спасибо! Я сегодня как раз поверхностно понял как работает switch-case теперь я убрал этот кусок))

Добавлено через 22 минуты
TheCalligrapher, Ну по порядку.

1) да меня никто не учил.

Но также я указал что могу и убрать struct или вы про ; перед struct?

Просто все работает и я не хотел пока править! Хотел продумать чуть позже... например через вечность или сейчас.
2)Это грязь из-за того что я тыкался и пытался решить проблему почему код не хочет нормально компилироваться

3.1) Не очень понял?)
3.2)Это забывчивость

А ещё потому что я дурак!) На самом деле до того как кинуть сюда код я за чем-то поменял float на int в struct.(
4)Это да!) Я уже поменял if-ы на switch-case.
5)Спасибо за совет)) Но можно кратко почему? Буду ещё больше благодарен)

Спс))

Добавлено через 5 минут
Алексей1153, Я уже ответил на похожий вопрос.)) Я надеюсь вы не будете против если повторю?Хорошо?)
"Я сделал чтобы 9 не шла в решение) Просто не знал как решить это. Сейчас я вроде поверхностно понял как работает switch-case теперь я убрал этот кусок))
"

Добавлено через 2 минуты
Алексей1153
Если бы я был человеком?! Шучу я али... Так не туда) Я просто не знаю что такое "человеческого форматирования кода", и вот момент "Инициализацию переменных тоже обязательно делать" объясните пожалуйста.))

@liv · 29.10.2019, 19:55

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

Но можно кратко почему?

Это об этом?

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

И использование для таких целей переменной типа float - плохая идея.

Ну так, вещественные числа - неточные! Сравнивать их на равенство чему-либо - неблагодарное дело!

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

что такое "человеческого форматирования кода"

Чтобы у человека-читателя кода не было вопросов что и до чего относится, чтобы все было аккуратно выровнено, с уступами, где требуется.

@EvilViking · 29.10.2019, 20:11 **[ТС]**

liv, "Сравнивать их на равенство чему-либо - неблагодарное дело!", Ну по большей части я это сделал как эксперимент) Я не желал становиться "неприкасаемым")

@liv · 29.10.2019, 20:15

EvilViking, да ради Бога, эксперементируйте, но запомните, что сравнивать числа с плавающей точкой на равенство - очень плохо. Это потенциальный источник возможных ошибок, т.к. в нужный момент можно не получить точного равенства.

Алексей1153 · 29.10.2019, 20:24

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

и вот момент "Инициализацию переменных тоже обязательно делать" объясните пожалуйста

да запросто. При объявлении любой переменной или при вписывании поля в классе первым делом производи инициализацию. Это должно быть как безусловный рефлекс. Поле или переменная без инициализации в общем случае содержит мусор, что влечёт за собой разные баги

C++

struct Choto
{
    int num_1{0};
    int num_2{0};
};

@EvilViking · 30.10.2019, 15:44 **[ТС]**

C++

float

@oleg-m1973 6574 / 4559 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	29.10.2019, 18:20	2
	Сообщение от EvilViking Кстати можно ли назвать "(0 * i)" костылем или просто тефтелька в спагетти? )) Зачем это у тебя? 0

Алексей1153 фрилансер 4470 / 3973 / 868 Регистрация: 11.10.2019 Сообщений: 10,483
	29.10.2019, 18:29	3
	EvilViking, начни с человеческого форматирования кода )) Инициализацию переменных тоже обязательно делать 0

Байт Диссидент 27200 / 16954 / 3748 Регистрация: 24.12.2010 Сообщений: 38,140
	29.10.2019, 18:42	6
	Сообщение от EvilViking можно ли назвать "(0 * i)" костылем или просто тефтелька в спагетти? Это можно назвать нелепостью, если не бредом. Тем не менее, желаю вам удачи на выбранном трудном пути! 0

@EvilViking 0 / 0 / 0 Регистрация: 29.10.2019 Сообщений: 7
	29.10.2019, 19:52 [ТС]	8
	oleg-m1973, Я сделал чтобы 9 не шла в решение) Просто не знал как решить это. Сейчас я вроде поверхностно понял как работает switch-case теперь я убрал этот кусок)) Добавлено через 3 минуты liv, спасибо! Я сегодня как раз поверхностно понял как работает switch-case теперь я убрал этот кусок)) Добавлено через 22 минуты TheCalligrapher, Ну по порядку. 1) да меня никто не учил. Но также я указал что могу и убрать struct или вы про ; перед struct? Просто все работает и я не хотел пока править! Хотел продумать чуть позже... например через вечность или сейчас. 2)Это грязь из-за того что я тыкался и пытался решить проблему почему код не хочет нормально компилироваться 3.1) Не очень понял?) 3.2)Это забывчивость А ещё потому что я дурак!) На самом деле до того как кинуть сюда код я за чем-то поменял float на int в struct.( 4)Это да!) Я уже поменял if-ы на switch-case. 5)Спасибо за совет)) Но можно кратко почему? Буду ещё больше благодарен) Спс)) Добавлено через 5 минут Алексей1153, Я уже ответил на похожий вопрос.)) Я надеюсь вы не будете против если повторю?Хорошо?) "Я сделал чтобы 9 не шла в решение) Просто не знал как решить это. Сейчас я вроде поверхностно понял как работает switch-case теперь я убрал этот кусок)) " Добавлено через 2 минуты Алексей1153 Если бы я был человеком?! Шучу я али... Так не туда) Я просто не знаю что такое "человеческого форматирования кода", и вот момент "Инициализацию переменных тоже обязательно делать" объясните пожалуйста.)) 0

@liv Модератор 5107 / 4538 / 854 Регистрация: 07.10.2015 Сообщений: 9,462
	29.10.2019, 19:55	9
	Сообщение от EvilViking Но можно кратко почему? Это об этом? Сообщение от TheCalligrapher И использование для таких целей переменной типа float - плохая идея. Ну так, вещественные числа - неточные! Сравнивать их на равенство чему-либо - неблагодарное дело! Сообщение от EvilViking что такое "человеческого форматирования кода" Чтобы у человека-читателя кода не было вопросов что и до чего относится, чтобы все было аккуратно выровнено, с уступами, где требуется. 0

@EvilViking 0 / 0 / 0 Регистрация: 29.10.2019 Сообщений: 7
	29.10.2019, 20:11 [ТС]	10
	liv, "Сравнивать их на равенство чему-либо - неблагодарное дело!", Ну по большей части я это сделал как эксперимент) Я не желал становиться "неприкасаемым") 0

@liv Модератор 5107 / 4538 / 854 Регистрация: 07.10.2015 Сообщений: 9,462
	29.10.2019, 20:15	11
	EvilViking, да ради Бога, эксперементируйте, но запомните, что сравнивать числа с плавающей точкой на равенство - очень плохо. Это потенциальный источник возможных ошибок, т.к. в нужный момент можно не получить точного равенства. 0