Discussion :

[GDS_Strou]

Bonjour.
je voudrais savoir quelle est la conséquence de mettre throw e dans un catch ?

Ceci car j’utilise sonar pour vérifier la qualité de mon code et il me renvoi un bug majeur lorsque je mets printstackstrace(e) dans mon catch.
En revanche, il ne me dit plus rien si je mets throw e;

Merci pour vos explications.

[Songbird]

Salut,

je voudrais savoir quelle est la conséquence de mettre throw e dans un catch ?

Si tu n'as pas un autre catch dans le premier catch, rien de spécial. Ça fait planter ton programme en conservant la ligne à laquelle l'exception a été créée.

Je ne connais pas sonar, mais il se peut qu'il n'apprécie pas trop la gestion d'erreur à coup de printStackTrace(). Il doit s'attendre à toujours recevoir une exception, j'imagine.

[GDS_Strou]

Merci, je vais certainement mettre partout où j’avais un printstacktrace un throw du coup.
Le fait de rentrer dans un catch fait systématiquement planter un programme ?

[joel.drigo]

Salut,

La différence essentielle entre :

Code :

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

1
2
3
catch(UneException e) {
   e.printStackTrace();
}

et

Code :

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

1
2
3
catch(UneException e) {
   throw e;
}

c'est que dans le premier cas, l'exécution de la méthode en cours ne s'arrête pas. En cas d'exception, il se peut donc qu'on exécute un code potentiellement non exécutable (incohérent ou pouvant causer des exceptions). Il faut donc probablement chercher ailleurs dans la méthode (après le catch) si l'absence de rupture d'exécution de la méthode n'est pas la cause d'un bug majeur (une variable à null par exemple)

Quand au second cas, il ne devrait même pas être écrit : ça ne sert à rien de catcher une exception pour la relancer (et ça consomme un peu de ressource, à moins que le compilateur fasse sauter directement), autant la laisser passer à l'appelant. Mixer les deux ne devraient pas être fait non plus : on risque d'avoir une stacktrace alors que le traitement dans l'appelant est peut-être un contournement qui fait qu'un résultat est bien produit : on se retrouve avec une trace d'exception alors que tout s'est bien passé. Si on passe une exception à l'appelant, ne pas traiter cette exception, le but de la passer à l'appelant voulant dire "je ne sais pas comment réagir pour cette exception à ce niveau du code alors je laisse un appelant ayant un contexte suffisant pour le faire.

Le fait de rentrer dans un catch fait systématiquement planter un programme ?

Non, ça dépend justement de comment on traite l'erreur.

[GDS_Strou]

Merci beaucoup, c’est très clair comme ça.
Juste une question, le throw permet de continuer le programme tout comme le printstacktrace ?

[Songbird]

Juste une question, le throw permet de continuer le programme tout comme le printstacktrace ?

Sauf si l'envoi est récupéré dans un autre contexte, non. Lorsqu'une exception non-gérée est lancée, ça fait planter le thread courant.