Discussion :

[allstar]

Voila tout est dans titre ?

J'ai une regle checkstyle m'interdisant de catcher une Exception de type
Throwable...

Je le soucis c'est que je ne comprends pas pourquoi cela n'est pas judicieux ?

merci

[tchize_]

parce que tu va catcher absolument tout. Non seulement les Exception auxquelles tu ne t'attends pas nécessairement et qui sont du à des erreurs de design (exemple des NullPointerException), ce qui pourrait encore être judicieux pour faire retomber ton code sur ses pattes +- dans tous les cas; mais tu va aussi catcher tous les Error (OutOfMemoryError, NoClassDefFoundError, ....) qui traduisent parfois des problèmes chroniques dans l'application, une situation catastrophique et un programme dans un état totalement instables. Ce checkstyle est là pour te dire "attention mon petit, si tu veux vraiment arrêter aussi les truc dangereux comme le OutOfMemoryError, montre moi vraiment que c'est ce que tu veux".

Mettre un catch(Throwable) ou un catch(Exception) juste pour 'pas se casser le cul', c'est souvent du mauvais design, car t'y va à la grosse louche et, comme pour la pêche au filet, t'as ton poisson mais tu ramasse aussi tous les dauphins et les méduses dont t'as pas besoin

[Astartee]

J'ai une regle checkstyle m'interdisant de catcher une Exception de type
Throwable...

Par règle "checkstyle" tu veux dire une convention de codage interne à ton équipe ? Je ne connais pas ce mot...


Catcher un Throwable n'est pas très judicieux dans le sens où ça catche tout et n'importe quoi, sans que tu puisses savoir ce qui s'est passé exactement...

Par exemple :

Code :

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

1
2
3
4
5
6
try {
  throw new MyException();
}
catch (Throwable e) {
  System.out.println("Erreur ! une exception MyException est survenue.");
}

Tu peux penser, au vu du code, que dans tous les cas tu catcheras une MyException.
Mais si jamais une erreur que tu n'as pas prévue dans ton code intervenait (supposons qu'au lieu du throw bête et méchant de cet exemple il y ait un appel à une fonction plus complexe susceptible de lever une MyException, et que lors de l'exécution de cette fonction une RuntimeException soit levée) alors tu vas catcher cette exception et croire qu'il s'agit d'un cas d'erreur donné, alors que ce n'est pas le cas...

Il vaut donc mieux, pour être sûr de ce que l'on catche, être le plus précis possible dans la clause catch :

Code :

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

1
2
3
4
5
6
try {
  throw new MyException();
}
catch (MyException e) {
  System.out.println("Erreur ! une exception MyException est survenue.");
}

Ainsi une erreur non prévue ne sera pas "cachée" par un catch trop global.

Par contre, un catch sur Throwable ne sera sans doute pas gênant si ensuite on relance l'exception :

Code :

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

1
2
3
4
5
6
7
try {
  throw new MyException();
}
catch (Throwable e) {
  System.out.println("Erreur ! une exception est survenue.");
  throw e;
}

[Deadpool]

Par règle "checkstyle" tu veux dire une convention de codage interne à ton équipe ? Je ne connais pas ce mot...

Checkstyle est le nom d'un outil permettant de vérifier si un code se conforme à un certain nombre de normes de codage Java.

[tchize_]

Par contre, un catch sur Throwable ne sera sans doute pas gênant si ensuite on relance l'exception :

Code :

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

1
2
3
4
5
 
catch (Throwable e) {
  System.out.println("Erreur ! une exception est survenue.");
  throw e;
}

Si, au contraire, ce sera très génant, car ta méthode aura alors comme signature "throws Throwable" et l'appelant sera obligé de catcher les Throwable

[allstar]

Oui bien sur il s'agit d'une convention de codage qu'on a dans le checkstyle.xml.

En tout cas c'es super clair

merci

[Astartee]

Checkstyle est le nom d'un outil permettant de vérifier si un code se conforme à un certain nombre de normes de codage Java.

OK, merci.

Si, au contraire, ce sera très génant, car ta méthode aura alors comme signature "throws Throwable" et l'appelant sera obligé de catcher les Throwable

En effet
J'avais dit "sans doute" parce que là comme ça dans ma tête je ne voyais pas de problème, mais c'est sûr que si on s'amusait à coder ça on s'en rendrait tout de suite compte, un "throws Throwable" je ne laisse pas passer ça !
[HS] et c'est pas juste pour parler, en ce moment je suis en pleine refonte d'un code produit en Inde où j'ai trouvé, entre autres bêtises, un beau throws Throwable et tout plein de catch (Exception e)...