Discussion :

[AsPrO]

Bonjour,

Je vois dans beaucoup de code cette pratique

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
throw new exception("Hum..hum");

Mais cela ne provoquerait-il pas des memory leak ? Etant donné qu'une zone mémoire est réservé mais jamais plus utilisé ni pointé (du moins en C++).
Je demande car le fait de le voir tellement souvent me fait douter, peut-être il y a t'il quelque chose à laquelle je ne pense pas.

[Matthieu Brucher]

Effectivement, je ne pense pas que ce soit super, d'autant qu'il faut récupérer l'exception par pointeur alors qu'un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

throw exception("Hum..hum");

peut être récupérée par un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

catch(const exception& exc)

sans qu'il y ait non plus d'allocation mémoire supplémentaire.

[Médinoc]

Une exception lancée ainsi doit être catchée par pointeur et deletée dans le catch.

J'ai tendance à dire qu'en C++, c'est une mauvaise pratique. La bibliothèque MFC possède une excuse, puisque sa première version date d'avant la standardisation du C++, voire même d'avant le support des exceptions C++ par Visual.

[Alp]

La bonne pratique c'est de virer le new.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

throw exception("Ca s'est mal passé");

Car le new n'apporte rien de bien/plus, si ce n'est qu'on aura à attrapper un exception* et à le deleter derrière...

[ram-0000]

J'avais ouvert un post à ce sujet : Durée de vie d'une exception il y a quelques jours.