Обработка исключений при вводе некорректных данных

@Опа!! · Регистрация: 30.09.2009

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

День добрый, начал разбираться с исключениями, пока не совсем понимаю в чем их удобство и зачем они нужны, какие преимущества и т.д... но вопрос, вобще-то, не в этом. Возможно ли применить обработку исключительной ситуации в случае некорректного (в плане типа) вводимых данных?

Пример - есть переменная int a; я считываю в нее значение: cin >> a; и ввожу вместо числа, скажем, букву. программа, естественно, вылетает. возможно ли при помощи исключений ухватить эту ошибку и не допустить падения программы? и возможно ли улавливать всякие непредвиденные ошибки, где я не использовал throw?

Добавлено через 1 час 14 минут
Вобщем вот такая вот идиотская защита от ввода не числовых значений при считывании в переменную int:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
cout << "Enter number of students: "; 
cin >> countBuf;
int x = countBuf.length();
for (int i = 0; i < x; i++)
{
    if (!isdigit(countBuf[i]))
    {
            throw 4;
    }
    Student_count = atoi(countBuf.c_str());
}
 
...
 
catch (int i)
{
         ...
         if (i == 4)
    {
         cout << endl << " ---Error!---" << endl << "Count must consist _only_ from digits! Please, try again." << endl;
    }
                
}

Я уверен, такую кривизну можно заменить более лакончиным и красивым способом.. но вот как? помогут ли мне в решении этой задачи try, catch и throw?

@solar_wind · 23.05.2011, 06:10

throw удобно, как мне кажется, только тем что передает управление сразу в catch выходя сразу из всех циклов программы.
Вообще наиболее часто throw используется что бы передать ошибку например из библиотеки и основную программу, встречал такое.

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

@pito211 · 23.05.2011, 07:33

C++
1
2
3
4
5
6
7
8
for (int i = 0; i < x; i++)
{
        if (!isdigit(countBuf[i]))
        {
                throw 4;
        }
        Student_count = atoi(countBuf.c_str());
}

разве этот кусок работает правильно? если допустим
countBuf[0] = '5'
countBuf[1] = 'a'
if будет пройден успешно(на первой итерации), throw не сработает, сработает atoi которой вернёт тебе, судя из cplusplus.com, zero. Ошибки вроде бы не будет в результате, но мне кажется это работает не так, как ты задумывал

Помоему надо вынести строку

C++
1
Student_count = atoi(countBuf.c_str());

после цикла? ты X раз переводишь(пытаешься переводить) строку в число. Это не рационально.
И ещё я думаю, что в такой програмулине не стоит нагромождать try и catch. Их преимущество в том, что ты в одном участке кода выбрасываешь исключение, например в собственном классе, а в самой программе или другом классе обрабатываешь?

@Опа!! · 23.05.2011, 09:39 **[ТС]**

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

разве этот кусок работает правильно? если допустим
countBuf[0] = '5'
countBuf[1] = 'a'
if будет пройден успешно(на первой итерации), throw не сработает, сработает atoi

да, это сработает правильно. встретили 5 - записали в Student_count _весь_ результат, да, даже если там на какой-то позиции дальше встретится не цифра. но в любом случае если мы при прохождении цикла далее встретим не букву то будет сгенерировано исключение и обработка передается в блок catch. неверные данные в переменной сохранятся, но по логике программы мы вернемся в этот же блок для повторного ввода. но лучше, конечно, вынести за цикл, согласен

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

что ты в одном участке кода выбрасываешь исключение, например в собственном классе, а в самой программе или другом классе обрабатываешь?

ну, заполнение - один из методов моего класса, а обработка ведется в void main().

так мне никто и не ответил =) как же все-таки сделаеть красивую проверку на корректность вводимых данных? как не допустить вылета при вводе, скажем, символов в интовую переменную?

@pito211 · 23.05.2011, 14:11

покажи весь листинг, не видно где у тебя блок try

если это какая-то функция, допустим она называется fuck() для определённости,

Сообщение от Опа!!

cout << "Enter number of students: ";
cin >> countBuf;
int x = countBuf.length();
for (int i = 0; i < x; i++)
{
* * * * if (!isdigit(countBuf[i]))
* * * * {
* * * * * * * * throw 4;
* * * * }
* * * * Student_count = atoi(countBuf.c_str());
}
...

и ты её вызываешь в main() то должно быть примерно так

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
try 
{
     fuck();
}
catch (int i)
{
     switch (i) 
     {
     case 1:
          cerr << "Error 1" << endl;
          break;
     case 2:
          cerr << "Error 2" << endl;
     ...
     default:
          cerr << "Fatal error" << endl;
          exit(i);
     }
}

как то так примерно?

Добавлено через 45 минут
и в качестве альтернативы твоему коду могу предложить такой вариант

C++
1
2
3
4
5
6
7
        int fuck;
    
    if ( !(cin >> fuck) )
    {
        cin.clear();
        throw(4);
    }

только учти, что если ты введёшь 4qwefds то никакого исключения выработано не будет, а fuck = 4, исключение будет выработано если введёшь йцу543ке34н4 например. Зато не надо никаких стрингов, преобразований к char* и вобще это более "сиплюсплюсный вариант", а у тебя сейчас комбинация из си и си++

@Опа!! · 23.05.2011, 18:20 **[ТС]**

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

int fuck;
if ( !(cin >> fuck) )
* * * * {
* * * * * * * * cin.clear();
* * * * * * * * throw(4);
* * * * }

за вот это спасибо, добавлю обязательно к себе, забыл что она тоже возвращает 0 и 1.

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

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
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
#include "stdafx.h"
#include <conio.h>
#include <string>
#include <iostream>
#include <vector>
#include <algorithm>
#include <ostream>
#include <sstream>
#include <fstream>
 
using namespace std;
 
int i = 0;
bool toFile(0);
 
ifstream f_In;
ofstream f_Out;
class CUniversity;
typedef vector <CUniversity> Base;
vector <CUniversity>::iterator index;
Base bs;
 
class CUniversity
{
    private:
        string Name;
        string City;
        char Type;
        int Student_count;
 
    public:
        CUniversity () {Student_count = 0;};
        
        CUniversity (string N, string C, char T, int S)
        {
            Name = N;
            City = C;
            Type = T;
            Student_count = S;
        }
        
        string GetCity()
        {
            return City;
        }
        
        void InsertVector(CUniversity Univer) 
        {
            bs.push_back(Univer);
        }
 
        void InputRecord() //throw (int) 
        {
            int nBuf = 0;
            string countBuf;
            
            cout << "Enter name: "; 
            cin >> Name;
            
            int nSize = Name.length();
            for (int i = 0; i < nSize; i++)
            {
                if (!isalpha (Name[i]))  throw 1;
            }
    
            cout << "Enter city: "; 
            cin >> City;
            nSize = City.length();
            for (int i = 0; i < nSize; i++)
            {
                if (!isalpha (City[i]) && !isdigit (City[i]))  throw 2;
            }
 
            string sBufType;
            cout << "Enter type ('g' - State, 'c' - Commercial): "; 
            cin >> sBufType;
            if (sBufType.length() == 1)
            {
                if ((sBufType[0] != 'g') && (sBufType[0] != 'c')) throw 3;
            }
            else throw 3;
            Type = sBufType[0];
            
            cout << "Enter number of students: "; 
            cin >> countBuf;
            int x = countBuf.length();
            for (int i = 0; i < x; i++)
            {
                if (!isdigit(countBuf[i]))
                {
                    throw 4;
                }
                Student_count = atoi(countBuf.c_str());
            }
        }
 
        void SortByCity() 
        {
            if (bs.size() != 0)
            sort (bs.begin(), bs.end());
        }
 
        void ShowInfo(bool toFile) 
        {
            if (toFile)
            {
                cout << "University name: " << Name << endl;
                f_Out << "University name: " << Name << endl;
                cout << "Destination city: " << City << endl;
                f_Out << "Destination city: " << City << endl;
                if (Type == 'c') 
                {
                    cout << "Type: Commercial." << endl;
                    f_Out << "Type: Commercial." << endl;
                }
                else 
                {
                    cout << "Type: State." << endl;
                    f_Out << "Type: State." << endl;
                }
                cout << "Number of students: " << Student_count << ".";
                f_Out << "Number of students: " << Student_count << ".";
                }
        
            else
            {
                cout << "University name: " << Name << endl;
                cout << "Destination city: " << City << endl;
                if (Type == 'c') 
                {
                    cout << "Type: Commercial." << endl;
                }
                else 
                {
                    cout << "Type: State." << endl;
                }
                cout << "Number of students: " << Student_count << ".";
            }
        }
 
        void ShowVector()
        {
            int n = bs.size();
            if (n != 0)
            {
                for (int j = 0; j < n; j++)
                {
                    cout << endl;
                    bs[j].ShowInfo(toFile);
                    cout << endl;
                }
            }
            else cout << endl << "Vector is empty!";
        }
 
        void SearchName(string SrchName)
        {
            toFile = true;
            int k = bs.size();
            bool isFinded = false;
            for (int l = 0; l < k; l++)
            {
                if (SrchName == bs[l].Name)
                {
                    bs[l].ShowInfo(true);
                    isFinded = true;
                }
            }
 
            if (!isFinded)
            {
                cout << "Nothing finded!" << endl;
                getch();
            }
            toFile = false;
        }
 
        void ReadFromFile()
        {
            string bufName;
            string bufCity;
            char bufType;
            int bufStudent_count;
 
            if (f_In.is_open())
            {
                while (!f_In.eof())
                {
                    f_In >> bufName >> bufCity >> bufType >> bufStudent_count;
                    bs.push_back(CUniversity(bufName, bufCity, bufType, bufStudent_count)); 
                }
                cout << "All ok";
                getch();
            }
            else 
            {
                cout << "Input file isn't opened!";
                getch();
            }
        }
 
        void ClearVector()
        {
            int nVecSize = bs.size();
            if (nVecSize == 0)
            {
                cout << endl << "Vector already empty!";
            }
            else bs.clear();
        }
 
        CUniversity& operator [] (int i) 
        {
            return bs[i];
        }
 
        bool operator < (CUniversity Univers)
        {
            if (Name < Univers.GetCity())
            {
                return true;
            }
            else return false;
        }
    //  ~CUniversity() {bs.clear();};
}; // class
 
void PrintMenu()
{
    system("cls");
    cout << "--- Main menu ---" << endl;
    cout << "Enter '1' to fill container from keyboard." << endl;
    cout << "Enter '2' to load information from file to container." << endl;
    cout << "Enter '3' to print all container." << endl;
    cout << "Enter '4' to find university by Name." << endl;
    cout << "Enter '5' to sort container by City." << endl;
    cout << "Enter '6' to clear container." << endl;
    cout << "Enter '7' to EXIT." << endl << endl;
    cout << "Your choice: ";
}
 
int main()
{
    f_In.open("f:\\in.txt");
    f_Out.open("f:\\out.txt", ios::in|ios::out|ios::trunc);
 
    if (!f_In || !f_Out) // если файл не найден
    {
        cout << "-- Error! File not found! --";
        getch();
    }
 
    CUniversity Univers;
    char iMenuChoice(0);
    string srchName;
 
    while (iMenuChoice != '7')
    {
        PrintMenu();
        cin >> iMenuChoice;
 
        switch (iMenuChoice)
        {
        case '1':
            while (i < 3)
            {
                
                try
                {
                    Univers.InputRecord();
                    i++;
                    Univers.InsertVector(Univers);
                }
                catch (int i)
                {
                    if (i == 1)
                    {
                        cout << endl << " ---Error!---" << endl << "Name must consist only from _letters_! Please, try again." << endl;
                    }
                    if (i == 2)
                    {
                        cout << endl << " ---Error!---" << endl << "Name of city must consist only from _letters_ or numbers! Please, try again." << endl;
                    }
                    if (i == 3)
                    {
                        cout << endl << " ---Error!---" << endl << "Type must be a single symbol - 'c' or 'g'! Please, try again." << endl;
                    }
                    if (i == 4)
                    {
                        cout << endl << " ---Error!---" << endl << "Count must consist _only_ from digits! Please, try again." << endl;
                    }
                    getch();
                }
                catch (...)
                {
                    cout << "Critical error.";
                    getch();
                }
                cout << endl;   
            }
            cout << endl << "Filling finished! Press anykey to continue..."; 
            getch();
            break;
 
        case '2':
            Univers.ReadFromFile();
            cout << endl << "...press anykey to continue"; 
            getch();
            break;
 
        case '3':
            toFile = false;
            Univers.ShowVector();
            cout << endl << "...press anykey to continue"; 
            getch();
            break;
 
        case '4':
            cout << "Enter Name for searching: "; cin >> srchName;
            Univers.SearchName(srchName);
            cout << endl << "...press anykey to continue"; 
            getch();
            break;
 
        case '5':
            Univers.SortByCity();
            cout << endl << "---------------" << endl << "Sorted by city: " 
                 << endl << "---------------" << endl;
            Univers.ShowVector();
            cout << endl << "...press anykey to continue"; 
            getch();
            break;
 
        case '6':
            Univers.ClearVector();
            cout << endl << "Done, press any key!";
            getch();
            break;
 
        case '7':
            break;
 
        default:
            cout << "Please, enter number from '1' to '5'!";
            getch();
            break;
        } // case
    } // while
 
    Univers.ClearVector();
    f_In.close();
    f_Out.close();
    return 0;
} // main

@pito211 · 23.05.2011, 18:35

Сообщение от Опа!!

for (int i = 0; i < nSize; i++)
* * * * * * * * * * * * {
* * * * * * * * * * * * * * * * if (!isalpha (Name[i])) *throw 1;
* * * * * * * * * * * * }

C++
1
2
3
4
if (std::string::npos != s.find_first_of("0123456789"))
{
    throw 1;
}

@Nameless One · 23.05.2011, 18:37

(Не смотрел по части исключений) а зачем глобальные переменные?

@pito211 · 23.05.2011, 18:38

Сообщение от Опа!!

int i = 0;
bool toFile(0);
ifstream f_In;
ofstream f_Out;
class CUniversity;
typedef vector <CUniversity> Base;
vector <CUniversity>::iterator index;
Base bs;

жёсткие связи, твой класс завязан на эти переменные, сам по себе он бесполезен, если не тащить за собой все эти объекты

@Опа!! · 23.05.2011, 19:38 **[ТС]**

Сообщение от Nameless One

(Не смотрел по части исключений) а зачем глобальные переменные?

так, а это как-то возбраняется? ухудшает читаемость? сказывается на производительности? вносит неудобства?

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

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

жёсткие связи, твой класс завязан на эти переменные, сам по себе он бесполезен, если не тащить за собой все эти объекты

эм.. не совсем понял... все вышеуказанное надо каким-то образом заключить в сам класс? о,О

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

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

int i = 0;
bool toFile(0);

ifstream f_In;
ofstream f_Out;
class CUniversity;
typedef vector <CUniversity> Base;
vector <CUniversity>::iterator index;
Base bs;

ну глобальные переемнные ладно, но куда я денусь от объявления входных и выходных файлов и уж тем более от вектора? тоже объявлять все внутри класса? или передавать каким-то образом параметрами?

@Nameless One · 24.05.2011, 02:10

Сообщение от Опа!!

ак, а это как-то возбраняется? ухудшает читаемость? сказывается на производительности? вносит неудобства?

Ухудшает читаемость и отладку программы, делает ее менее безопасной

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

Сообщение от Опа!!

ну глобальные переемнные ладно, но куда я денусь от объявления входных и выходных файлов и уж тем более от вектора? тоже объявлять все внутри класса? или передавать каким-то образом параметрами?

передаешь конструктору имя файла, открываешь файл в конструкторе, закрываешь файл в деструкторе

@Опа!! · 24.05.2011, 03:12 **[ТС]**

Спасибо огромное за советы, сейчас попробую

@pito211 · 24.05.2011, 05:35

ну и раз у тебя вся обработка исключений состоит в том, что выводится строчка, я бы на твоём месте передавал исключение типа string, тогда обработку можно было сократить до

C++
1
2
3
4
5
6
7
8
9
10
try
{
    Univers.InputRecord();
    i++;
    Univers.InsertVector(Univers);
}
catch (string str)
{
    cout << str << endl;
}

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера 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, то после закрытия окошка. . .

@solar_wind 770 / 760 / 59 Регистрация: 06.07.2009 Сообщений: 3,021
	23.05.2011, 06:10
	throw удобно, как мне кажется, только тем что передает управление сразу в catch выходя сразу из всех циклов программы. Вообще наиболее часто throw используется что бы передать ошибку например из библиотеки и основную программу, встречал такое. Отлавливать исключения нужно так как какая либо библиотека, которую ты используешь, может запросто тебе его прислать. Если исключения не отлавливать, итогом возникновения ошибок, могут стать крахи программы, которые ты мог предотвратить. Однако во-первых тут двоякая ситуация, так как после некоторых ошибок программа может неправильно функционировать, хотя отлов исключения дает возможность тебе выдать для пользователя не какую то непонятную табличку на английском, а что либо удобочитаемое для него. Во-вторых исключений есть два типа, и отлавливать ты можешь только один из них. В любом случае, тема не сказал бы что маленькая, рекомендую почитать статьи по исключениям, в сети их довольно много. 1

@Nameless One 5828 / 3479 / 358 Регистрация: 08.02.2010 Сообщений: 7,448
	23.05.2011, 18:37
	(Не смотрел по части исключений) а зачем глобальные переменные? 1

@Опа!! 17 / 17 / 1 Регистрация: 30.09.2009 Сообщений: 89
	24.05.2011, 03:12 [ТС]
	Спасибо огромное за советы, сейчас попробую 0