Форум программистов, компьютерный форум CyberForum.ru

Неполадка с default в switch-case - вывод срабатывает дважды - C++

Восстановить пароль Регистрация
 
Александр_Алекс
4 / 1 / 1
Регистрация: 08.12.2013
Сообщений: 136
15.08.2014, 15:31     Неполадка с default в switch-case - вывод срабатывает дважды #1
Здравствуйте! Такая вот ерундовая проблема, но хотелось бы узнать почему происходит именно так =)

Есть функция, проверяющая корректность ввода уравнения. Если уравнение не соответствует шаблону, программа об этом сообщает и просит ввести его заново. И все отлично работает, но в блоке default есть два cout и они срабатывают дважды. Почему такое происходит?

вот код:

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
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
// Uravnenie.cpp: определяет точку входа для консольного приложения.
//
 
#include "stdafx.h"
#include "iostream"
 
using namespace std;
 
int checkCorrectness(char equation[])
{
    int CheckX = 0; //chtobu ne bulo 2-x 'x'
    int CheckEquation = 0;
    //proverka pervogo simvola
    switch(equation[0])
    {
        case 'x':
            CheckEquation = 1;
            CheckX = 1;
            break;
        case '1':
            CheckEquation = 1;
            break;
        case '2':
            CheckEquation = 1;
            break;
        case '3':
            CheckEquation = 1;
            break;
        case '4':
            CheckEquation = 1;
            break;
        case '5':
            CheckEquation = 1;
            break;
        case '6':
            CheckEquation = 1;
            break;
        case '7':
            CheckEquation = 1;
            break;
        case '8':
            CheckEquation = 1;
            break;
        case '9':
            CheckEquation = 1;
            break;
        case '0':
            CheckEquation = 1;
            break;
        default:
            CheckEquation = 0;
            cout<<"You've entered wrong equation, please, try again! 1"<<endl;
            cout<<"Your equation: ";
            return CheckEquation;
    }
    //proverka vtorogo simvola
    switch(equation[1])
    {
        case '+':
            CheckEquation = 1;
            break;
        case '-':
            CheckEquation = 1;
            break;
        default:
            CheckEquation = 0;
            cout<<"You've entered wrong equation, please, try again! 2 "<<endl;
            cout<<"Your equation: ";
            return CheckEquation;
    }
    //proverka 3-go simvola
    switch(equation[2])
    {
        case 'x':
            if(CheckX == 1)
            {
                CheckEquation = 0;
                cout<<"You've entered wrong equation, please, try again! 3"<<endl;
                cout<<"Your equation: ";
                return CheckEquation;
                break;
            }
            else
            {
                CheckEquation = 1;
                break;
            }
        case '1':
            CheckEquation = 1;
            break;
        case '2':
            CheckEquation = 1;
            break;
        case '3':
            CheckEquation = 1;
            break;
        case '4':
            CheckEquation = 1;
            break;
        case '5':
            CheckEquation = 1;
            break;
        case '6':
            CheckEquation = 1;
            break;
        case '7':
            CheckEquation = 1;
            break;
        case '8':
            CheckEquation = 1;
            break;
        case '9':
            CheckEquation = 1;
            break;
        case '0':
            CheckEquation = 1;
            break;
        default:
            CheckEquation = 0;
            cout<<"You've entered wrong equation, please, try again! 4"<<endl;
            cout<<"Your equation: ";
            return CheckEquation;
    }
    //proverka 4-go simvola
    switch(equation[3])
    {
        case '=':
            CheckEquation = 1;
            break;
        default:
            CheckEquation = 0;
            cout<<"You've entered wrong equation, please, try again! 5"<<endl;
            cout<<"Your equation: ";
            return CheckEquation;
    }
    //proverka 5-go simvola
    switch(equation[4])
    {
        case '1':
            CheckEquation = 1;
            break;
        case '2':
            CheckEquation = 1;
            break;
        case '3':
            CheckEquation = 1;
            break;
        case '4':
            CheckEquation = 1;
            break;
        case '5':
            CheckEquation = 1;
            break;
        case '6':
            CheckEquation = 1;
            break;
        case '7':
            CheckEquation = 1;
            break;
        case '8':
            CheckEquation = 1;
            break;
        case '9':
            CheckEquation = 1;
            break;
        case '0':
            CheckEquation = 1;
            break;
        default:
            CheckEquation = 0;
            cout<<"You've entered wrong equation, please, try again! 6"<<endl;
            cout<<"Your equation: ";
            return CheckEquation;
    }
 
    return CheckEquation;
 
}
 
 
int _tmain(int argc, _TCHAR* argv[])
{
    char equation[6];
    cout<<"Enter the equation, please."<<endl;
    cout<<"Example: x+1=2."<<endl;
    cout<<"Your equation: ";
    do
    {
        cin>>equation;
        checkCorrectness(equation);
    }while(checkCorrectness(equation)!=1);
    cout<<equation;
 
    system("pause");
    return 0;
}
А вот, что получается:

Неполадка с default в switch-case - вывод срабатывает дважды

Заранее спасибо за ваши ответы!
Лучшие ответы (1)
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
15.08.2014, 15:31     Неполадка с default в switch-case - вывод срабатывает дважды
Посмотрите здесь:

C++ switch case
C++ Switch - case
C++ switch-case, как писать в самом case
C++ Switch default и cin
Switch-case C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
tehnar5
31 / 31 / 12
Регистрация: 03.05.2011
Сообщений: 84
15.08.2014, 15:42     Неполадка с default в switch-case - вывод срабатывает дважды #2
Сообщение было отмечено автором темы, экспертом или модератором как ответ
Дело в том, что вы сначала проверяете корректность после ввода, а потом еще и в цикле while, вот и все. Попробуйте сделать что-то типа
C++
1
2
    int res =    checkCorrectness(equation);
}while(res!=1);
A_Kamolins
 Аватар для A_Kamolins
4 / 4 / 4
Регистрация: 15.08.2014
Сообщений: 30
15.08.2014, 15:49     Неполадка с default в switch-case - вывод срабатывает дважды #3
Может так получше будет? Ничего лишнего, вроде:

C++
1
2
3
4
5
    while (true) {
        cin >> equation;
        if (checkCorrectness(equation) == 1) break;
    }
    cout << equation;
А про двойную проверку парень правду говорит
ValeryS
Модератор
6377 / 4843 / 442
Регистрация: 14.02.2011
Сообщений: 16,053
15.08.2014, 15:55     Неполадка с default в switch-case - вывод срабатывает дважды #4
Цитата Сообщение от Александр_Алекс Посмотреть сообщение
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
case '1':
            CheckEquation = 1;
            break;
        case '2':
            CheckEquation = 1;
            break;
        case '3':
            CheckEquation = 1;
            break;
        case '4':
            CheckEquation = 1;
            break;
        case '5':
            CheckEquation = 1;
            break;
        case '6':
            CheckEquation = 1;
            break;
        case '7':
            CheckEquation = 1;
            break;
        case '8':
            CheckEquation = 1;
            break;
        case '9':
            CheckEquation = 1;
            break;
        case '0':
            CheckEquation = 1;
            break;
не лень писать было?
это потенциальный источник ошибок
поправив в одной ветке забудешь поправить в другой
вот так более безопасно
C++
1
2
3
4
5
6
7
8
9
10
11
12
        case '1':
        case '2':
        case '3':
        case '4':
        case '5':
        case '6':
        case '7':
        case '8':
        case '9':
        case '0':
            CheckEquation = 1;
            break;
и проверка эта, повторяется несколько раз
может имеет смысл вынести в отдельную функцию?
Александр_Алекс
4 / 1 / 1
Регистрация: 08.12.2013
Сообщений: 136
15.08.2014, 16:06  [ТС]     Неполадка с default в switch-case - вывод срабатывает дважды #5
tehnar5, как я сам не додумался. Спасибо за ответ. Была мысля так сделать, но почему-то не сделал =)
ValeryS, не знал, что так тоже можно записать, спасибо.
A_Kamolins, и тебе спасибо за предложенный вариант, но что-то кнопочки "спасибо" возле тебя нет
A_Kamolins
 Аватар для A_Kamolins
4 / 4 / 4
Регистрация: 15.08.2014
Сообщений: 30
15.08.2014, 16:11     Неполадка с default в switch-case - вывод срабатывает дважды #6
Видимо я ещё новенький и из-за этого
Yandex
Объявления
15.08.2014, 16:11     Неполадка с default в switch-case - вывод срабатывает дважды
Ответ Создать тему
Опции темы

Текущее время: 09:01. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru