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

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

Восстановить пароль Регистрация
 
Рейтинг: Рейтинг темы: голосов - 50, средняя оценка - 4.84
Опа!!
14 / 14 / 0
Регистрация: 30.09.2009
Сообщений: 89
23.05.2011, 02:06     Обработка исключений при вводе некорректных данных #1
День добрый, начал разбираться с исключениями, пока не совсем понимаю в чем их удобство и зачем они нужны, какие преимущества и т.д... но вопрос, вобще-то, не в этом. Возможно ли применить обработку исключительной ситуации в случае некорректного (в плане типа) вводимых данных?

Пример - есть переменная 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
 Аватар для solar_wind
740 / 731 / 39
Регистрация: 06.07.2009
Сообщений: 2,937
Завершенные тесты: 1
23.05.2011, 06:10     Обработка исключений при вводе некорректных данных #2
throw удобно, как мне кажется, только тем что передает управление сразу в catch выходя сразу из всех циклов программы.
Вообще наиболее часто throw используется что бы передать ошибку например из библиотеки и основную программу, встречал такое.

Отлавливать исключения нужно так как какая либо библиотека, которую ты используешь, может запросто тебе его прислать. Если исключения не отлавливать, итогом возникновения ошибок, могут стать крахи программы, которые ты мог предотвратить. Однако во-первых тут двоякая ситуация, так как после некоторых ошибок программа может неправильно функционировать, хотя отлов исключения дает возможность тебе выдать для пользователя не какую то непонятную табличку на английском, а что либо удобочитаемое для него. Во-вторых исключений есть два типа, и отлавливать ты можешь только один из них.
В любом случае, тема не сказал бы что маленькая, рекомендую почитать статьи по исключениям, в сети их довольно много.
pito211
 Аватар для pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
23.05.2011, 07:33     Обработка исключений при вводе некорректных данных #3
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. Их преимущество в том, что ты в одном участке кода выбрасываешь исключение, например в собственном классе, а в самой программе или другом классе обрабатываешь?
Опа!!
14 / 14 / 0
Регистрация: 30.09.2009
Сообщений: 89
23.05.2011, 09:39  [ТС]     Обработка исключений при вводе некорректных данных #4
Цитата Сообщение от pito211 Посмотреть сообщение
разве этот кусок работает правильно? если допустим
countBuf[0] = '5'
countBuf[1] = 'a'
if будет пройден успешно(на первой итерации), throw не сработает, сработает atoi
да, это сработает правильно. встретили 5 - записали в Student_count _весь_ результат, да, даже если там на какой-то позиции дальше встретится не цифра. но в любом случае если мы при прохождении цикла далее встретим не букву то будет сгенерировано исключение и обработка передается в блок catch. неверные данные в переменной сохранятся, но по логике программы мы вернемся в этот же блок для повторного ввода. но лучше, конечно, вынести за цикл, согласен



Цитата Сообщение от pito211 Посмотреть сообщение
что ты в одном участке кода выбрасываешь исключение, например в собственном классе, а в самой программе или другом классе обрабатываешь?
ну, заполнение - один из методов моего класса, а обработка ведется в void main().

так мне никто и не ответил =) как же все-таки сделаеть красивую проверку на корректность вводимых данных? как не допустить вылета при вводе, скажем, символов в интовую переменную?
pito211
 Аватар для pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
23.05.2011, 14:11     Обработка исключений при вводе некорректных данных #5
покажи весь листинг, не видно где у тебя блок 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* и вобще это более "сиплюсплюсный вариант", а у тебя сейчас комбинация из си и си++
Опа!!
14 / 14 / 0
Регистрация: 30.09.2009
Сообщений: 89
23.05.2011, 18:20  [ТС]     Обработка исключений при вводе некорректных данных #6
Цитата Сообщение от 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
 Аватар для pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
23.05.2011, 18:35     Обработка исключений при вводе некорректных данных #7
Цитата Сообщение от Опа!! Посмотреть сообщение
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
Эксперт С++
 Аватар для Nameless One
5754 / 3403 / 255
Регистрация: 08.02.2010
Сообщений: 7,393
23.05.2011, 18:37     Обработка исключений при вводе некорректных данных #8
(Не смотрел по части исключений) а зачем глобальные переменные?
pito211
 Аватар для pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
23.05.2011, 18:38     Обработка исключений при вводе некорректных данных #9
Цитата Сообщение от Опа!! Посмотреть сообщение
int i = 0;
bool toFile(0);
ifstream f_In;
ofstream f_Out;
class CUniversity;
typedef vector <CUniversity> Base;
vector <CUniversity>::iterator index;
Base bs;
жёсткие связи, твой класс завязан на эти переменные, сам по себе он бесполезен, если не тащить за собой все эти объекты
Опа!!
14 / 14 / 0
Регистрация: 30.09.2009
Сообщений: 89
23.05.2011, 19:38  [ТС]     Обработка исключений при вводе некорректных данных #10
Цитата Сообщение от 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
Эксперт С++
 Аватар для Nameless One
5754 / 3403 / 255
Регистрация: 08.02.2010
Сообщений: 7,393
24.05.2011, 02:10     Обработка исключений при вводе некорректных данных #11
Цитата Сообщение от Опа!! Посмотреть сообщение
ак, а это как-то возбраняется? ухудшает читаемость? сказывается на производительности? вносит неудобства?
Ухудшает читаемость и отладку программы, делает ее менее безопасной

Добавлено через 1 минуту
Цитата Сообщение от Опа!! Посмотреть сообщение
ну глобальные переемнные ладно, но куда я денусь от объявления входных и выходных файлов и уж тем более от вектора? тоже объявлять все внутри класса? или передавать каким-то образом параметрами?
передаешь конструктору имя файла, открываешь файл в конструкторе, закрываешь файл в деструкторе
Опа!!
14 / 14 / 0
Регистрация: 30.09.2009
Сообщений: 89
24.05.2011, 03:12  [ТС]     Обработка исключений при вводе некорректных данных #12
Спасибо огромное за советы, сейчас попробую
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
24.05.2011, 05:35     Обработка исключений при вводе некорректных данных
Еще ссылки по теме:

C++ Проверка типа данных при вводе
Ошибка при вводе данных и в функциях C++
C++ Неверная обработка данных при вводе двух и более величин

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

Или воспользуйтесь поиском по форуму:
pito211
 Аватар для pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
24.05.2011, 05:35     Обработка исключений при вводе некорректных данных #13
ну и раз у тебя вся обработка исключений состоит в том, что выводится строчка, я бы на твоём месте передавал исключение типа 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;
}
Yandex
Объявления
24.05.2011, 05:35     Обработка исключений при вводе некорректных данных
Ответ Создать тему
Опции темы

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