Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Александр_Алекс
7 / 4 / 1
Регистрация: 08.12.2013
Сообщений: 165
#1

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

15.08.2014, 15:31. Просмотров 485. Ответов 5
Метки нет (Все метки)

Здравствуйте! Такая вот ерундовая проблема, но хотелось бы узнать почему происходит именно так =)

Есть функция, проверяющая корректность ввода уравнения. Если уравнение не соответствует шаблону, программа об этом сообщает и просит ввести его заново. И все отлично работает, но в блоке 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 - вывод срабатывает дважды

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

В операторе switch всегда срабатывает default - C++
Привет всем, вот такая проблема ввожу указанные в case буквы, в любом случий срабатывает default, почему так? #define WINVER 0x0501 ...

Оптимальная конструкция switch-case-while / while-switch-case - C++
Имеется конструкция типа: switch() { case 1: while() { ... }

Switch case внутри switch case - C++
Привет всем! Нужна помощь. Пишу калькулятор с консольным меню. Так вот, используется цикл while, внутри которого у меня switch...

switch-case, как писать в самом case - C++
Такс дело вот в чём,прога работает всё хорошо,вот только мне не нравится как записваю страны в кейсе,можно ли каким-нибудь образом писать в...

Оператор Switch - default - C++
У меня есть якобы такой код на С++ Builder`e. Я сделал к каждому сase свою формулу. в ListBoxe у меня есть 4 строчка, где должен быть...

Switch default и cin - C++
Проблема в следующем: Если в case есть cin то после завершения кейса выбирается default, как избежать этого? int i; ...

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

C++
1
2
3
4
5
    while (true) {
        cin >> equation;
        if (checkCorrectness(equation) == 1) break;
    }
    cout << equation;
А про двойную проверку парень правду говорит
0
ValeryS
Модератор
6705 / 5114 / 482
Регистрация: 14.02.2011
Сообщений: 17,183
15.08.2014, 15:55 #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;
и проверка эта, повторяется несколько раз
может имеет смысл вынести в отдельную функцию?
1
Александр_Алекс
7 / 4 / 1
Регистрация: 08.12.2013
Сообщений: 165
15.08.2014, 16:06  [ТС] #5
tehnar5, как я сам не додумался. Спасибо за ответ. Была мысля так сделать, но почему-то не сделал =)
ValeryS, не знал, что так тоже можно записать, спасибо.
A_Kamolins, и тебе спасибо за предложенный вариант, но что-то кнопочки "спасибо" возле тебя нет
0
A_Kamolins
4 / 4 / 4
Регистрация: 15.08.2014
Сообщений: 30
15.08.2014, 16:11 #6
Видимо я ещё новенький и из-за этого
0
15.08.2014, 16:11
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
15.08.2014, 16:11
Привет! Вот еще темы с ответами:

Недопонимание с switch и выводом default - C++
Задание: сделать так что бы при вводе целого числа (от 1 до 7 которые соответствует дням недели, 1- понедельник, 2 -вторник и т.д.)...

Возвращение значения из switch-default - C++
Здравствуйте! У меня задача на создание двох фунций, которые вычисляют среднее значение двух и трех чисел соответственно. Все это я сделал....

Для чего нужны break и default в switch - C++
что означает строка break и default в codeblocks? пример: #include using namespace std; int main() {

Switch - case - C++
Обязательно с использованием классов Есть задача. Дан массив вещественных чисел. а) Ко всем отрицательным элементам прибавить элемент...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
6
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru