Калькулятор для начинающих, прошу оценить

@Adamtotu · Регистрация: 12.05.2016

Студворк — интернет-сервис помощи студентам

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
#include <iostream>
#include <math.h>
using namespace std;
 
int menu ();
int sum ();
int razn ();
int vozved ();
int delen ();
 
 
int main (){
    for (;;){
            
            menu ();
            
    }
    return 0;
    
}
 int menu(){
    
    int choice;
    
    cout <<"Calculator V.01\n";
    cout <<"1.Suma\n";
    cout <<"2.Raznica\n";
    cout <<"3.Ymnozenie\n";
    cout <<"4.Delenie\n";
    
    cout <<"Enter the number:1 - 4;\n";
    cin>> choice;
    cout <<"\n";
    
    switch (choice){
        case 1:
            cout <<sum()<<"\n";
            break;
        case 2:
            cout <<razn()<<"\n";
            break;
        case 3:
            cout <<vozved()<<"\n";
            break;
        case 4:
            cout <<delen()<<"\n";
            break;
    }
    
 }
 int sum (){
    float a,b,x;
    
    for (x = 0; x != 0005;){
    cout <<"Enter the first number: \n";
    cin>>a;
    cout <<"Enter the second number: \n";
    cin>>b;
    
    cout <<"Answer: "<< a + b<<" \n";
 
    menu ();
}
}
int razn (){
    float a,b,x;
    
    for (x = 0; x != 0005;){
    cout <<"Enter the first number: \n";
    cin>>a;
    cout <<"Enter the second number: \n";
    cin>>b;
    
    cout <<"Answer: "<< a - b<<" \n";
 
    menu ();
}
}
 
int vozved (){
    float a,b,x;
    
    for (x = 0; x != 0005;){
    cout <<"Enter the first number: \n";
    cin>>a;
    cout <<"Enter the second number: \n";
    cin>>b;
    
    cout <<"Answer: "<< a * b<<" \n";
 
    menu ();
}
}
 
int delen (){
    float a,b,x;
    
    for (x = 0; x != 0005;){
    cout <<"Enter the first number: \n";
    cin>>a;
    cout <<"Enter the second number: \n";
    cin>>b;
    
    cout <<"Answer: "<< a / b<<" \n";
 
    menu ();
}
}

// Прошу оценить!

@Alamant · 18.05.2016, 20:57

Много лишних фигурных скобок.

@Adamtotu · 20.05.2016, 13:44 **[ТС]**

А калькулятор, как тебе?

@GbaLog- · 20.05.2016, 14:53

Adamtotu, А что если я 0 введу в делитель? И почему в условии цикла стоит x != 0005? Зачем эти нули? Кстати, все циклы в функциях бесконечные(может не все, но лень смотреть).

Добавлено через 2 минуты
Зачем подключалась библиотека math.h? Почему все функции возвращают int, хотя обрабатываемые значения с плавающей точкой?
Так себе, если честно.

@Adamtotu · 20.05.2016, 16:12 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
#include <iostream>
using namespace std;
 
int menu ();
int summa ();
int razn ();
int mnoz ();
int delen ();
 
int  main (){
    setlocale (0,"");
    for (;;){
        
        menu ();
        
    }
    return 0;
}
 
int menu (){
    int choice;
    cout <<"Выберите действие\n";
    cout <<"1.Сумма\n";
    cout <<"2.Разница\n";
    cout <<"3.Произведение\n";
    cout <<"4.Деление\n";
    
    cout <<"Выберите число (1-4);\n";
    cin >> choice;
    cout <<'\n';
    
    
    switch (choice){
        
        case 1:
            cout << summa()<<"\n";
            break;
        case 2:
            cout <<razn() <<"\n";
            break;
        case 3:
            cout <<mnoz()<<"\n";
            break;
        case 4:
            cout <<delen() <<"\n";
            break;
    }
    
}
 
int summa(){
    float a,b;
    cout <<"Вы перешли в меню - Сумма.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВторое число: ";
    cin >> b;
    cout <<"\nРезультат: "<<a + b<<"\n";
    main ();
    
}
int razn(){
    
    float a,b;
    cout <<"Вы перешли в меню - Разница.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВведите второе число: ";
    cin >> b;
    cout <<"\nРезультат: "<<a - b<<"\n";
    main ();
    
}
 
int mnoz(){
    
    float a,b;
    cout <<"Вы перешли в меню - Произведение.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВведите второе число: ";
    cin >> b;
    cout <<"\nРезультат: "<<a * b<<"\n";
    main ();
    
}
 
int delen(){
    
    float a,b;
    cout <<"Вы перешли в меню - Деление.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВведите второе число: ";
    cin >> b;
    
    if (b) cout <<"Результат: "<<a/b<<"\n";
    else cout <<"На ноль не делится\n";
    main ();
    
}

// Вот я исправил, а что на счет функций???

@Alamant · 20.05.2016, 17:04

Adamtotu, я скопировал код, но VS2015 ругается на первый же for. Мне кажется, там не хватает закрывающей скобки. В качестве учебного калькулятора, наверное, это неплохая программа, но она таки должна работать. Также вызывает сомнения идея рекурсивного вызова main.

@Adamtotu · 20.05.2016, 17:28 **[ТС]**

на dev 5 c++, код идеально работает. За VS 2015 не знаю. Но спасибо)

Добавлено через 2 минуты
Можешь посмотреть, на счет этого смайлика. Смайлик образовался в качестве написанных ; и ).

Ибо for(;; ){} это бесконечный цикл.

@Alamant · 21.05.2016, 13:48

Adamtotu, теперь код компилируется) Правда, когда я ввел в меню букву, программа ушла в бесконечный цикл. Зато она работает. Это хорошо.

@Adamtotu · 21.05.2016, 15:32 **[ТС]**

Потому что программа не настроена на буквы. Правда можна также условие поставить. Например если пользователь введет букву или не подходящий символ, то программа просто вернет его в меню. Как доработка))

@Alamant · 21.05.2016, 19:33

Adamtotu, всегда полезно думать об обработке ошибок.

@DaVillka · 21.05.2016, 19:51

Adamtotu, что то он слишком сложныи как для четырех операторов, когда я делал, у меня была только одна fункция в которои проверялся сам оператор и выводил просто а+b на пример.

@Adamtotu · 21.05.2016, 20:16 **[ТС]**

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

Adamtotu, что то он слишком сложныи как для четырех операторов, когда я делал, у меня была только одна fункция в которои проверялся сам оператор и выводил просто а+b на пример.

Мы говорим о калькуляторе для начинающих ибо он не такой уж и сложный.. А на счет "одна функция", то бажано вообще не нагружать главную функцию(main). А я сделал простой калькулятор, его можно использовать для начинающих.

@DaVillka · 21.05.2016, 21:41

Adamtotu, я понимаю, но создавать кучу ненужных переменных и функций...

Добавлено через 7 минут
Adamtotu, много лишнего, но все равно молодцом ^^

Добавлено через 43 минуты
Alamant, как тебе аналог твоего кода?

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
#include <iostream>
 
float calculator(float chislo1, char simvol, float chislo2)
{
    switch (simvol)
    {
    case '+': return chislo1 + chislo2;
    case '-': return chislo1 - chislo2;
    case '*': return chislo1 * chislo2;
    case '/': return chislo1 / chislo2;
    default: return 1;
    }
    return 0;
}
int main()
{
    setlocale (LC_ALL, "RUS");
    std::cout << "Калькулятор\n";
 
    float chislo1 = 0, chislo2 = 0;
    char simvol   = 0;
 
    for (;;)
    {
        std::cout <<"Введите первое число: " << std::endl;
        std::cin >> chislo1;
        std::cout <<"Введите символ: " << std::endl;
        std::cin >> simvol;
        std::cout <<"Введите второе число: " << std::endl;
        std::cin >> chislo2;
 
        std::cout << "Результат: " << calculator(chislo1,  simvol,  chislo2) << std::endl;
    }
    return 0;
}

@Nosey · 21.05.2016, 21:55

Adamtotu,
Я просто попросил компилятор рассказать о ошибках:

1) ISO C++ forbids taking address of function ‘::main’ [-Wpedantic] Test2.cpp /Test2/src line 61 - страшная жуть
2) ISO C++ forbids taking address of function ‘::main’ [-Wpedantic] Test2.cpp /Test2/src line 73 - страшная жуть
3) ISO C++ forbids taking address of function ‘::main’ [-Wpedantic] Test2.cpp /Test2/src line 86 - страшная жуть
4) ISO C++ forbids taking address of function ‘::main’ [-Wpedantic] Test2.cpp /Test2/src line 101 - страшная жуть
5) comparing floating point with == or != is unsafe [-Wfloat-equal] Test2.cpp /Test2/src line 99 - может быть страшной жутью
6) no return statement in function returning non-void [-Wreturn-type] Test2.cpp /Test2/src line 51 - жуткова-то но пройдет.
7) no return statement in function returning non-void [-Wreturn-type] Test2.cpp /Test2/src line 63 - жуткова-то но пройдет.
8) no return statement in function returning non-void [-Wreturn-type] Test2.cpp /Test2/src line 75 - жуткова-то но пройдет.
9) no return statement in function returning non-void [-Wreturn-type] Test2.cpp /Test2/src line 88 - жуткова-то но пройдет.
10) no return statement in function returning non-void [-Wreturn-type] Test2.cpp /Test2/src line 103 - жуткова-то но пройдет.
11) switch missing default case [-Wswitch-default] Test2.cpp /Test2/src line 35 - чем строже код - тем лучше.

Поправил их:
Разве что оставил сравнение с нулем, чтоб не усложнять:

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
#include <iostream>
using namespace std;
 
void menu ();
void summa ();
void razn ();
void mnoz ();
void delen ();
 
int  main (){
    setlocale (0,"");
    for (;;){
 
        menu ();
 
    }
    return 0;
}
 
void menu (){
    int choice;
    cout <<"Выберите действие\n";
    cout <<"1.Сумма\n";
    cout <<"2.Разница\n";
    cout <<"3.Произведение\n";
    cout <<"4.Деление\n";
 
    cout <<"Выберите число (1-4);\n";
    cin >> choice;
    cout <<'\n';
 
 
    switch (choice){
 
        case 1:
            summa();
            break;
        case 2:
            razn();
            break;
        case 3:
            mnoz();
            break;
        case 4:
            delen();
            break;
        default:
            break;
    }
 
}
 
void summa(){
    float a,b;
    cout <<"Вы перешли в меню - Сумма.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВторое число: ";
    cin >> b;
    cout <<"\nРезультат: "<<a + b<<"\n";
}
 
void razn(){
    float a,b;
    cout <<"Вы перешли в меню - Разница.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВведите второе число: ";
    cin >> b;
    cout <<"\nРезультат: "<<a - b<<"\n";
}
 
void mnoz(){
    float a,b;
    cout <<"Вы перешли в меню - Произведение.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВведите второе число: ";
    cin >> b;
    cout <<"\nРезультат: "<<a * b<<"\n";
}
 
void delen(){
    float a,b;
    cout <<"Вы перешли в меню - Деление.";
    cout <<"\nВведите первое число: ";
    cin >> a;
    cout <<"\nВведите второе число: ";
    cin >> b;
 
    if (b) cout <<"Результат: "<<a/b<<"\n";
    else cout <<"На ноль не делится\n";
}

И немного оформил:

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
#include <iostream>
using namespace std;
 
void menu();
void summa();
void razn();
void mnoz();
void delen();
 
int main()
{
    setlocale(0, "");
    for (;;)
    {
        menu();
    }
    return 0;
}
 
void menu()
{
    int choice;
    cout << "Выберите действие\n";
    cout << "1.Сумма\n";
    cout << "2.Разница\n";
    cout << "3.Произведение\n";
    cout << "4.Деление\n";
 
    cout << "Выберите число (1-4);\n";
    cin >> choice;
    cout << '\n';
 
    switch (choice)
    {
        case 1:
        {
            summa();
            break;
        }
        case 2:
        {
            razn();
            break;
        }
        case 3:
        {
            mnoz();
            break;
        }
        case 4:
        {
            delen();
            break;
        }
        default:
        {
            break;
        }
    }
}
 
void summa()
{
    float a, b;
    cout << "Вы перешли в меню - Сумма.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВторое число: ";
    cin >> b;
    cout << "\nРезультат: " << a + b << "\n";
}
 
void razn()
{
    float a, b;
    cout << "Вы перешли в меню - Разница.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВведите второе число: ";
    cin >> b;
    cout << "\nРезультат: " << a - b << "\n";
}
 
void mnoz()
{
    float a, b;
    cout << "Вы перешли в меню - Произведение.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВведите второе число: ";
    cin >> b;
    cout << "\nРезультат: " << a * b << "\n";
}
 
void delen()
{
    float a, b;
    cout << "Вы перешли в меню - Деление.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВведите второе число: ";
    cin >> b;
 
    if (b)
    {
        cout << "Результат: " << a / b << "\n";
    }
    else
    {
        cout << "На ноль не делится\n";
    }
}

@Adamtotu · 21.05.2016, 23:16 **[ТС]**

DaVillka, Я что то не понял, а что ты изменил. Прости если туплю, но не бросается в глаза.

@DaVillka · 21.05.2016, 23:18

Adamtotu, ничего я не менял, просто несколько проще сделал твои код, и он уменьшился в два раза )

@Adamtotu · 21.05.2016, 23:22 **[ТС]**

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

Adamtotu, я понимаю, но создавать кучу ненужных переменных и функций...

Добавлено через 7 минут
Adamtotu, много лишнего, но все равно молодцом ^^

Добавлено через 43 минуты
Alamant, как тебе аналог твоего кода?

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
#include <iostream>
 
float calculator(float chislo1, char simvol, float chislo2)
{
    switch (simvol)
    {
    case '+': return chislo1 + chislo2;
    case '-': return chislo1 - chislo2;
    case '*': return chislo1 * chislo2;
    case '/': return chislo1 / chislo2;
    default: return 1;
    }
    return 0;
}
int main()
{
    setlocale (LC_ALL, "RUS");
    std::cout << "Калькулятор\n";
 
    float chislo1 = 0, chislo2 = 0;
    char simvol   = 0;
 
    for (;;)
    {
        std::cout <<"Введите первое число: " << std::endl;
        std::cin >> chislo1;
        std::cout <<"Введите символ: " << std::endl;
        std::cin >> simvol;
        std::cout <<"Введите второе число: " << std::endl;
        std::cin >> chislo2;
 
        std::cout << "Результат: " << calculator(chislo1,  simvol,  chislo2) << std::endl;
    }
    return 0;
}

Во первых, что бы не писать всегда (std::cout) можна ввести такую своего рода библиотеку(using namespace std

Во вторых у меня в калькуляторе присутствует "меню", которого в твоём варианте нет, т.е. мой вариант рассматривает более интересней материал.
В третьих, твоя идея очень хороша, спасибо что подказала/подсказал такую идейку, да просто не догадался как сделать, может не думал.
В четвертых. как завершить программу в твоём варианте? Предлогаю к нашим вариантам добавить возможность добавить выход из программы.

@DaVillka · 21.05.2016, 23:40

Adamtotu, ну все это дельные замечания, я просто хотел показать, как можно сделать код проще, а на счет using namespace std, я стараюсь явно указывать пространства имен, иначе можно нахвататься лишних проблем в будущем.

@Adamtotu · 22.05.2016, 00:35 **[ТС]**

DaVillka, Не знаю, пробовал сделать выход но в твоёй версии не варит мозг, или мои навыки слишком малы.

Добавлено через 12 минут
DaVillka, Вот что хочу узнать, если тебе известно подкинь идею. Как сделать так, что бы Юзер мог выбрать количество чисел, которые будет использовать. Ибо мне не достаточно двоих, я хочу что бы Юзер сам мог выбирать.

Добавлено через 29 минут

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
#include <iostream>
#include <cstdlib>
using namespace std;
 
void menu ();
void summa ();
void razn ();
void umnoz ();
void delen ();
 
int main (){
    setlocale (0,"");
    menu ();
    return 0;
 
}
 
void menu (){
    
    int option;
    
    do {
        cout << "Добро пожаловать в Калькулятор (0.5)\n";
        cout << "1.Сумма\n";
    cout << "2.Разница\n";
        cout << "3.Произведение\n";
        cout << "4.Деление\n";
        cout << "5.Выход\n";
        do {
                cout << "Выберите действие (1 - 5): \n";
                cin >> option;
        }
        while (option < 1, option > 5);
        
        switch (option) {
            case 1: summa();
                break;
            case 2:razn ();
                break;
            case 3:umnoz();
                break;
            case 4:delen ();
                break;
            case 5:cout <<"Спасибо за использование нашей программы!!!";
                break;
        }
    }
    while (option != 5);
}
 
void summa()
{
    float a, b;
    cout << "Вы перешли в меню - Сумма.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВторое число: ";
    cin >> b;
    cout << "\nРезультат: " << a + b << "\n";
}
 
void razn()
{
    float a, b;
    cout << "Вы перешли в меню - Разница.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВведите второе число: ";
    cin >> b;
    cout << "\nРезультат: " << a - b << "\n";
}
 
void umnoz()
{
    float a, b;
    cout << "Вы перешли в меню - Произведение.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВведите второе число: ";
    cin >> b;
    cout << "\nРезультат: " << a * b << "\n";
}
 
void delen()
{
    float a, b;
    cout << "Вы перешли в меню - Деление.";
    cout << "\nВведите первое число: ";
    cin >> a;
    cout << "\nВведите второе число: ";
    cin >> b;
 
    if (b)
    {
        cout << "Результат: " << a / b << "\n";
    }
    else
    {
        cout << "На ноль не делится\n";
    }
}

// Вот как я переделал всю программу. Можно сказать я переписал весь код. Что скажешь?

@DaVillka · 22.05.2016, 00:42

Adamtotu, можно предложить юзеру вводить числа умножать делить и тд пока он не введет '='

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Alamant 13 / 13 / 9 Регистрация: 16.05.2016 Сообщений: 49
	18.05.2016, 20:57
	Много лишних фигурных скобок. 0

@Adamtotu 0 / 0 / 0 Регистрация: 12.05.2016 Сообщений: 41
	20.05.2016, 13:44 [ТС]
	А калькулятор, как тебе? 0

@GbaLog- Любитель чаепитий 3745 / 1801 / 566 Регистрация: 24.08.2014 Сообщений: 6,020 Записей в блоге: 1
	20.05.2016, 14:53
	Adamtotu, А что если я 0 введу в делитель? И почему в условии цикла стоит x != 0005? Зачем эти нули? Кстати, все циклы в функциях бесконечные(может не все, но лень смотреть). Добавлено через 2 минуты Зачем подключалась библиотека math.h? Почему все функции возвращают int, хотя обрабатываемые значения с плавающей точкой? Так себе, если честно. 0

@Alamant 13 / 13 / 9 Регистрация: 16.05.2016 Сообщений: 49
	20.05.2016, 17:04
	Adamtotu, я скопировал код, но VS2015 ругается на первый же for. Мне кажется, там не хватает закрывающей скобки. В качестве учебного калькулятора, наверное, это неплохая программа, но она таки должна работать. Также вызывает сомнения идея рекурсивного вызова main. 1

@Adamtotu 0 / 0 / 0 Регистрация: 12.05.2016 Сообщений: 41
	20.05.2016, 17:28 [ТС]
	на dev 5 c++, код идеально работает. За VS 2015 не знаю. Но спасибо) Добавлено через 2 минуты Можешь посмотреть, на счет этого смайлика. Смайлик образовался в качестве написанных ; и ). Ибо for(;; ){} это бесконечный цикл. 0

@Alamant 13 / 13 / 9 Регистрация: 16.05.2016 Сообщений: 49
	21.05.2016, 13:48
	Adamtotu, теперь код компилируется) Правда, когда я ввел в меню букву, программа ушла в бесконечный цикл. Зато она работает. Это хорошо. 1

@Adamtotu 0 / 0 / 0 Регистрация: 12.05.2016 Сообщений: 41
	21.05.2016, 15:32 [ТС]
	Потому что программа не настроена на буквы. Правда можна также условие поставить. Например если пользователь введет букву или не подходящий символ, то программа просто вернет его в меню. Как доработка)) 0

@Alamant 13 / 13 / 9 Регистрация: 16.05.2016 Сообщений: 49
	21.05.2016, 19:33
	Adamtotu, всегда полезно думать об обработке ошибок. 0

@DaVillka -1 / 8 / 6 Регистрация: 31.10.2015 Сообщений: 151
	21.05.2016, 19:51
	Adamtotu, что то он слишком сложныи как для четырех операторов, когда я делал, у меня была только одна fункция в которои проверялся сам оператор и выводил просто а+b на пример. 0

@Adamtotu 0 / 0 / 0 Регистрация: 12.05.2016 Сообщений: 41
	21.05.2016, 23:16 [ТС]
	DaVillka, Я что то не понял, а что ты изменил. Прости если туплю, но не бросается в глаза. 0

@DaVillka -1 / 8 / 6 Регистрация: 31.10.2015 Сообщений: 151
	21.05.2016, 23:18
	Adamtotu, ничего я не менял, просто несколько проще сделал твои код, и он уменьшился в два раза ) 0

@DaVillka -1 / 8 / 6 Регистрация: 31.10.2015 Сообщений: 151
	21.05.2016, 23:40
	Adamtotu, ну все это дельные замечания, я просто хотел показать, как можно сделать код проще, а на счет using namespace std, я стараюсь явно указывать пространства имен, иначе можно нахвататься лишних проблем в будущем. 0

@DaVillka -1 / 8 / 6 Регистрация: 31.10.2015 Сообщений: 151
	22.05.2016, 00:42
	Adamtotu, можно предложить юзеру вводить числа умножать делить и тд пока он не введет '=' 0