Discussion :

[tentos]

Bonjour,

je travaille avec sonar pour auditer la qualité de mon code et je suis tombeé sur la métrique suivante :
Avoid Catching Throwable : A catch statement should never catch throwable since it includes errors.

!je récupère cette erreur sur la portion de code

Code :

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

1
2
3
4
5
6
 
try {
	super.finalize();
} catch (Throwable exception) {
	LOG.error(exception);
}

Y-a t'il une solution pour contourner le problème ?

[fr1man]

Oui catcher uniquement des classes dérivant d'Exception.
Une Error est une erreur grave qu'on ne doit pas essayer catcher, car l'application n'est plus en mesure de fonctionner.

[adiGuba]

Salut,


La solution est de virer le try/catch qui ne sert à rien !
Tu n'en as pas besoin dans la méthode finalize()...

a++

[professeur shadoko]

Avoid Catching Throwable : A catch statement should never catch throwable since it includes errors.

à part le fait que le catch ne s'applique pas dans le cas d'espèce je trouve la recommandation un tantinet contradictoire (avoid/never). Je rappelle toujours la recommandation d'Oscar Wilde: "appuyez-vous sur vos principes il finiront bien par céder!".
(oui dans certains cas -pas courants, courants- ça peut être pertinent de faire catch de Throwable et, en particulier, de récolter des Errors )