Модераторы: Daevaorn

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Предпочтение исключению, свое или стандартное? 
V
    Опции темы
Dray
Дата 5.7.2006, 17:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Материалист
**


Профиль
Группа: Участник
Сообщений: 652
Регистрация: 7.10.2003
Где: г. Всеволожск

Репутация: 2
Всего: 6



Например у меня есть класс в котором хранятся числа в каком-то диапазоне. И в случае если кто-то попытается записать число вне диапазона, я хочу кинуть исключение. Как обычно принято делать: создать свой класс-исключение или кидать какое-нибудь стандартное, например out_of_range? 


--------------------
忍者

user posted image
PM MAIL   Вверх
Xenon
Дата 5.7.2006, 18:10 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник Клуба
Сообщений: 1529
Регистрация: 12.4.2006

Репутация: 11
Всего: 50



Dray, ну ты можешь просто кинуть исключение throw "Error", можешь не создавать класс. Но если тебе ничего не нужно, кроме надписи "Error", то класс не создавай. Если же нужно передать нечто большее, тогда создавай класс. 
PS. Блин, бред написал - исправляю 

Это сообщение отредактировал(а) XenonSk - 5.7.2006, 20:39


--------------------
user posted image  
PM MAIL   Вверх
En_t_end
Дата 5.7.2006, 18:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Участник Клуба
Сообщений: 2074
Регистрация: 4.12.2004

Репутация: 1
Всего: 20



Dray
Если хочешь мнение, то я бы не стал возвращать в твоем случае false, 0, NULL, минимум - вернул бы enum ErrType{...}; Исключение вообще не стал бы кидать. Просто заведи в классе тот же enum ErrType{NO_ERROR=0,OUT_OF_RANGE=-100}; В каждом методе возьми за привычку(если это не void) возвращать ErrType и получишь практически "бесплатный" расширяемый класс(можешь кроме OUT_OF_RANGE и ещё что-нибудь добавить). Ибо исключение - это всё же дорогое удовольствие, может так получиться, что некий(знаю парочку  smile ) компилятор их не держит и тогда придется адаптировать, кроить, перепроектировать... не только код метода, но и код юзающий метод.

Добавлено @ 18:53 
Кстати, кто-нибуль тестировал на скорость обработку(+перхват) исключения и сравнивания значений двух переменных ? Что-то мне подсказывает, что второе явно быстрее.   

Это сообщение отредактировал(а) En_t_end - 5.7.2006, 18:54
PM MAIL ICQ Skype GTalk Jabber   Вверх
Void
Дата 5.7.2006, 19:42 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


λcat.lolcat
****


Профиль
Группа: Участник Клуба
Сообщений: 2206
Регистрация: 16.11.2004
Где: Zürich

Репутация: 40
Всего: 173



En_t_end, с таким подходом проще всего писать все на чистом Си (спасибо, что не на ассемблере).

Цитата(En_t_end @  5.7.2006,  20:48 Найти цитируемый пост)
Кстати, кто-нибуль тестировал на скорость обработку(+перхват) исключения и сравнивания значений двух переменных ? Что-то мне подсказывает, что второе явно быстрее.   

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


--------------------
“Coming back to where you started is not the same as never leaving.” — Terry Pratchett
PM MAIL WWW GTalk   Вверх
En_t_end
Дата 5.7.2006, 20:34 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Участник Клуба
Сообщений: 2074
Регистрация: 4.12.2004

Репутация: 1
Всего: 20



Void
Цитата(Void @  5.7.2006,  23:42 Найти цитируемый пост)
 с таким подходом проще всего писать все на чистом Си (спасибо, что не на ассемблере).

Если бы в чистых Сях была бы поддержка ООП, да шаблоны, то я бы писал именно на них. Но тогда это были бы не Cи  smile 

Цитата(Void @  5.7.2006,  23:42 Найти цитируемый пост)
Но ведь исключение выбрасывается в исключительных ситуациях, если это не так, то напрашивается вывод об ошибках дизайна. А значит потери производительности минимальны. 

Ладно... согласен. Но все же для портируемости лучше юзать перечесление...
К тому же, что чисто психологически делать проще ?
Код

switch(class_obj.Metod())
{
case fyb_do_err: log_err << " 'do it really' -exception";break;
case more: break;
//default: log_err << "unreferenced error"; /*кстати вот здесь есть преимущество у исключений, unreferenced error в данном случае отловить невозможно*/
}

или:
Код

try{class_obj.Metod();}
catch(fyb){log_err << " 'do it really' -exception";}//f.y.b - fuck you brain
catch(more){//...;}
catch(...){log_err << "unreferenced exception";}

Меня пугает множество фигурных скобок.
ЗЫ Скобки, скобки, скобки... вложенности, много вложенностей, они, они,идут за мной(после Макконелла я не люблю вложенности, скобки и всё что с ними связанно).

Добавлено @ 20:46 
Void
А вот умеет у тебя механизм отлова исключений действовать так ? smile
Код

ErrType tmperr = class_obj.Metod();
if(tmperr>ERR_RANGE1 && tmperr < ERR_RANGE20) do_some();
 

Это сообщение отредактировал(а) En_t_end - 5.7.2006, 20:49
PM MAIL ICQ Skype GTalk Jabber   Вверх
Xenon
Дата 5.7.2006, 20:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник Клуба
Сообщений: 1529
Регистрация: 12.4.2006

Репутация: 11
Всего: 50



En_t_end, Но зато, если нормально продумать ... Надо будет тебе, допустим, изменить сообщение об ошибки - меняешь только в catch блоке и все. И централизация. 


--------------------
user posted image  
PM MAIL   Вверх
En_t_end
Дата 5.7.2006, 20:54 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Участник Клуба
Сообщений: 2074
Регистрация: 4.12.2004

Репутация: 1
Всего: 20



XenonSk
что-то я не пойму, где децентрализация, при использовании перечеслений ?

Добавлено @ 20:56 
XenonSk
Надо мне будет изменить сообщение об ошибке я прежде всего изменю его в константных полях, а если надо заюзать новое сообщение, то изменю одну или две строчки в case блоке. 
PM MAIL ICQ Skype GTalk Jabber   Вверх
Xenon
Дата 5.7.2006, 21:09 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник Клуба
Сообщений: 1529
Регистрация: 12.4.2006

Репутация: 11
Всего: 50



En_t_end, ну просто, допустим, у тебя 8 методов, в которых может произойти одна и таже ошибка. По-мойму проще будет изменить в catch Одну сторку, чем там в каждом методе менять.
А централизациия - в смысле обработка ошибок в одном месте ... 


--------------------
user posted image  
PM MAIL   Вверх
DeadSoul
Дата 5.7.2006, 21:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник
Сообщений: 1217
Регистрация: 25.9.2005
Где: Москва

Репутация: 2
Всего: 11



Цитата(En_t_end @  5.7.2006,  20:34 Найти цитируемый пост)
Ладно... согласен. Но все же для портируемости лучше юзать перечесление...

С каких пор исключения стали непереносимыми?

Не надо учить других своей ненависти к исключениям 


--------------------
 Если Вы получили ответ на Ваш вопрос, то нажмите на "Вопрос решен". 

Бьем спамеров их же оружием. Пусть весь спам сыпется им
[email protected] 
PM   Вверх
Fazil6
Дата 5.7.2006, 21:56 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1653
Регистрация: 3.5.2006
Где: Минск

Репутация: 35
Всего: 60



Цитата

Если хочешь мнение, то я бы не стал возвращать в твоем случае false, 0, NULL, минимум - вернул бы enum ErrType{...}; Исключение вообще не стал бы кидать. Просто заведи в классе тот же enum ErrType{NO_ERROR=0,OUT_OF_RANGE=-100}; В каждом методе возьми за привычку(если это не void) возвращать ErrType и получишь практически "бесплатный" расширяемый класс(можешь кроме OUT_OF_RANGE и ещё что-нибудь добавить).

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

Хорошо, проверил я и вижу, что мне вернули ошибку. Дальше что? Самому по этапу передавать наверх? А там проверят и дальше ошибку возвратят? Так вся программа превратится в сплошной if-else.
Исключение нужно для обработки ошибки не там, где она возникла, а там, где ее можно и нужно обработать.  
En_t_end,
Цитата

А вот умеет у тебя механизм отлова исключений действовать так ?

ты это серьезно?
 
Код

try
 {
   class_obj.Metod();
 }
catch(const RangeError &err)
{
   if((err.tmperr>ERR_RANGE1) && (err.tmperr < ERR_RANGE20)) DoSome();
   else
       DoElse()
}


причем самый главный витамин в том, что throw RangeError(); совсем не обязательно кидался в class_obj.Metod(), а где-нибудь в глубинах кода, вызываемого в ней и не надо без конца проверять где и что вернулось.

XenonSk
Цитата

ну ты можешь просто кинуть исключение throw "Error", можешь не создавать класс. Но если тебе ничего не нужно, кроме надписи "Error", то класс не создавай.

это неправильно. Никогда не надо так делать даже если это работает. Всегда нужно создавать объект исключения.  
PM MAIL   Вверх
Dray
Дата 5.7.2006, 22:34 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Материалист
**


Профиль
Группа: Участник
Сообщений: 652
Регистрация: 7.10.2003
Где: г. Всеволожск

Репутация: 2
Всего: 6



Цитата(Fazil6 @  5.7.2006,  21:56 Найти цитируемый пост)
Всегда нужно создавать объект исключения.

Так все таки лучше свой? 


--------------------
忍者

user posted image
PM MAIL   Вверх
Fazil6
Дата 5.7.2006, 23:03 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1653
Регистрация: 3.5.2006
Где: Минск

Репутация: 35
Всего: 60



Цитата

Так все таки лучше свой? 

эта моя фраза относится в первую очередь к throw "Error", а свое или нет
это от ситуации зависит. если не надо информацию передавать, а только сигнализировать, то сойдет и пустой класс
Цитата

class MyError{};


throw MyError();

catch(const MyError &err)
{}


сдругой стороны учитывай как и где исключения будут обрабатываться. Если обработка для разных эксепшенов одинаковая, то может быть нет смысла для каждой ошибки генерить отдельное исключение, а использовать какое-то одно. 
Если ты хочешь в объекте исключения передавать какие-то данные и несколько, то здесь без своего класса трудно обойтись.
А бывают ситуации когда идет какая-то обработка данных и при ошибках надо просто прервать работу сообщив что где не так , тут и std::runtime_error вполне сгодится и писать что-то свое совсем не обязательно   
PM MAIL   Вверх
En_t_end
Дата 6.7.2006, 07:40 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Участник Клуба
Сообщений: 2074
Регистрация: 4.12.2004

Репутация: 1
Всего: 20



Fazil6
Цитата(Fazil6 @  6.7.2006,  01:56 Найти цитируемый пост)
ты это серьезно?

Ептс... конечно... видимо воображение у меня слабова-то smile
Цитата(Fazil6 @  6.7.2006,  01:56 Найти цитируемый пост)
! Очень интересно. А если я забуду проверить? А если после меня кто-то будет этот код сопровождать?

Я не уверен, но в бизнес-проектах, насколько мне известно, документируют каждый метод от сих до сих. К тому же не увидеть, что  метод возвращает ErrType или тому подобную вещь, с приставкой Err - просто невозможно, бросается в глаза.
И к тому же я не говорю о том, чтобы использовать перечесление, как способ сигнализации об ошибке только в одном методе. Когда ты примешь эту концепцию во всём классе, библиотеке классов, то никаких проблем с вижу-не вижу,отлавливаю-не отлавливаю ИМХО быть не должно.
Далее... почему ты думаешь, что с исключениями дела обстоят лучше ? Человек также может не заметить, что метод может бросать некое исключение и не сделать вокруг него try{}catch()catch(...) - ТАК что тут у исключений и перечеслений одинаковые шансы на нерадивого программиста.
Цитата(DeadSoul @  6.7.2006,  01:39 Найти цитируемый пост)
С каких пор исключения стали непереносимыми?

Они наоборот СТАЛИ переносимыми только с некоторых пор. 
PM MAIL ICQ Skype GTalk Jabber   Вверх
takedo
Дата 6.7.2006, 08:28 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 501
Регистрация: 1.6.2005

Репутация: 1
Всего: 3



в данный момент я согласен с  En_t_end, потому что считаю удобным сделать вообще в енуме NotError = 0, и когда функция выполняется делать так: 
Код

if(Func())
{
//а здесь обрабатываем ошибку
}

Если же у нас исключение, то надо эту функцию обрамлять в трай сатч, что не всегда удобно.

Добавлено @ 08:31 
да, кстати, чуть не забыл: smile нам ведь вообще не всегда надо обрабатывать ошибку! А если это так, то генерация исключения дороже просто ничего не делания smile  


--------------------
я не гольфист - я хоккеист
PM MAIL   Вверх
Fazil6
Дата 6.7.2006, 09:54 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1653
Регистрация: 3.5.2006
Где: Минск

Репутация: 35
Всего: 60



En_t_end
Цитата

Далее... почему ты думаешь, что с исключениями дела обстоят лучше ? Человек также может не заметить, что метод может бросать некое исключение и не сделать вокруг него try{}catch()catch(...) - ТАК что тут у исключений и перечеслений одинаковые шансы на нерадивого программиста.

у меня такое чувство, что ты слабо себе представляешь о чем дискутируешь. Кто тебе сказал, что каждую функцию, бросающую исключения обязательно надо обрамлять в try ? 
try ставится там, где будет обрабатываться ошибка, а не там где вызывается функция бросающая исключение и по объекту исключения определяется что произошло, где и как на это реагировать.

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

На самом деле эксепшены - лучшие друзья программиста.
 
PM MAIL   Вверх
Страницы: (3) Все [1] 2 3 
Ответ в темуСоздание новой темы Создание опроса
Правила форума "С++:Общие вопросы"
Earnest Daevaorn

Добро пожаловать!

  • Черновик стандарта C++ (за октябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика(4.4мб).
  • Черновик стандарта C (за сентябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика (3.4мб).
  • Прежде чем задать вопрос, прочтите это и/или это!
  • Здесь хранится весь мировой запас ссылок на документы, связанные с C++ :)
  • Не брезгуйте пользоваться тегами [code=cpp][/code].
  • Пожалуйста, не просите написать за вас программы в этом разделе - для этого существует "Центр Помощи".
  • C++ FAQ

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, Earnest Daevaorn

 
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | C/C++: Общие вопросы | Следующая тема »


 




[ Время генерации скрипта: 0.0901 ]   [ Использовано запросов: 21 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.