throw new std :: exception vs throw std :: exception

Question

patrząc na niektóre kodu Natknąłem się na:

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

a ja zawsze myślałem, że nie trzeba/nie powinno się używać new tutaj.
Jaki jest właściwy sposób, są zarówno OK, jeśli tak jest jakaś różnica?

BTW z tego co widzę, podczas gdy „grepping” z PowerShell przypominającego bibliotekami nigdy używać throw new.

P.S. również znalazłem kod CLI, który używa throw gcnew. Czy to w porządku?

Answer 1

Konwencjonalny sposób rzut złapać wyjątków jest rzut obiektu wyjątku i złapać ją odniesienia (zwykle const odniesienia). Język C++ wymaga, aby kompilator generował odpowiedni kod w celu skonstruowania obiektu wyjątku i właściwego czyszczenia go we właściwym czasie.

rzucanie wskaźnik do dynamicznie przydzielonego obiektu nigdy nie jest dobrym pomysłem. Wyjątki mają na celu umożliwienie pisania bardziej odpornego kodu w obliczu błędów. Jeśli rzucasz obiekt wyjątkowy w konwencjonalny sposób, możesz być pewien, że niezależnie od tego, czy zostanie on złapany przez klauzulę catch nazwaną właściwym typem, przez catch (...), czy zostanie on ponownie zgłoszony, czy nie, zostanie odpowiednio zniszczony we właściwym czasie. . (Jedynym wyjątkiem jest sytuacja, w której nigdy nie zostanie złapany, ale jest to sytuacja nie do naprawienia niezależnie od tego, jak na nią patrzysz.)

Jeśli rzucisz wskaźnik na dynamicznie przydzielany obiekt, musisz mieć pewność, że stos wywołań wygląda tak, jak w punkcie, w którym chcesz rzucić wyjątek, znajduje się blok catch, który nazywa właściwy typ wskaźnika i ma odpowiednie wywołanie delete.Twój wyjątek nigdy nie może zostać przechwycony przez catch (...), chyba że ten blok ponownie rzuci wyjątek, który następnie zostanie przechwycony przez inny blok catch, który zajmuje się tym wyjątkiem.

Oznacza to, że skorzystałeś z funkcji obsługi wyjątków, która powinna ułatwić pisanie niezawodnego kodu i bardzo utrudnić pisanie kodu, który jest poprawny we wszystkich sytuacjach. Pozostawia to poza kwestią, że prawie niemożliwe będzie działanie jako kod biblioteki dla kodu klienta, który nie będzie oczekiwał tej funkcji.

Answer 2

Nie ma potrzeby korzystania new kiedy rzuca wyjątek.

Wystarczy napisać:

throw yourexception(yourmessage); 

i złapać jak:

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

Zauważ, że yourexception powinny pochodzić bezpośrednio lub pośrednio z std::exception.

Answer 3

Throwing new std::exception jest poprawny, jeśli strona połączenia oczekuje na złapanie std::exception*. Ale nikt nie spodziewa się, że złapie wskaźnik do wyjątku. Nawet jeśli udokumentować to, co się robi i funkcja osoby zapoznać się z dokumentacją, są one nadal mogą zapomnieć i spróbować złapać odniesienie do std::exception obiektu zamiast.

Answer 4

C++ FAQ ma ładny dyskusję na ten temat.

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

zasadzie „chyba nie jest dobry powód, aby nie, łapać przez odniesienie Unikać łapania przez wartość, ponieważ powoduje to wykonanie kopii, a kopia może mieć inne zachowanie od tego, co zostało rzucone, tylko w wyjątkowych okolicznościach powinnaś złapać wskaźnik. "