певд зЬй :: исключение против броска станд :: исключение

Question

, глядя на какой-то код, который я наткнулся на:

throw /*-->*/new std::exception ("//... 

, и я всегда думал, что вам не нужно/не следует использовать new здесь.
Каков правильный путь, оба в порядке, если это так, то есть разница?

Кстати, из того, что я вижу, когда «grepping» с PowerShell boost libs никогда не используют throw new.

P.S. также я нашел код CLI, который использует throw gcnew. Это нормально?

Answer 1

Обычный способ исключения и исключения исключений заключается в том, чтобы выбросить объект исключения и уловить его ссылкой (обычно const). Язык C++ требует, чтобы компилятор сгенерировал соответствующий код для создания объекта исключения и правильно очистил его в соответствующее время.

Бросок указателя на динамически выделенный объект никогда не является хорошей идеей. Предполагается, что исключения позволят вам написать более надежный код перед лицом ошибок. Если вы выбрали объект исключения обычным способом, вы можете быть уверены, что если он пойман по ключу, именовав правильный тип, с помощью catch (...), будет ли он повторно перезаписан или нет, он будет правильно уничтожен в соответствующее время , (Единственное исключение - если это никогда не поймано, но это не восстанавливаемая ситуация, если вы посмотрите на нее.)

Если вы бросаете указатель на динамически выделенный объект, вы должны быть уверены, что независимо от того, стек вызовов выглядит так, как будто вы хотите выбросить свое исключение, есть блок catch, который называет правильный тип указателя и имеет соответствующий вызов delete.Ваше исключение никогда не должно быть уловлено catch (...), если только этот блок не повторяет исключение, которое затем попадает в другой блок catch, который правильно справляется с исключением.

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

Answer 2

Не нужно использовать new при исключении бросания.

Просто написать:

throw yourexception(yourmessage); 

и поймать как:

catch(yourexception const & e) 
{ 
     //your code (probably logging related code) 
} 

Обратите внимание, что yourexception следует извлечь из std::exception прямо или косвенно.

Answer 3

Throwing new std::exception является правильным, если сайт вызова ожидает поймать std::exception*. Но никто не ожидает, что поймает указатель на исключение. Даже если вы документируете, что делает ваша функция, и люди читают документацию, они по-прежнему могут забыть и попытаться поймать ссылку на объект std::exception.

Answer 4

C++ FAQ имеет хорошую дискуссию по этому вопросу:.

1. https://isocpp.org/wiki/faq/exceptions#what-to-catch
2. https://isocpp.org/wiki/faq/exceptions#catch-by-ptr-in-mfc

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