Précision sur le try/catch

Bonjour,

Souvent dans mon code je déclenche des exceptions sur certains cas que je pourrait gerer simplement à l'aide d'un if else, et je ne sais pas si c'est réellement avisé.

Prenons l'exemple d'un buffer dans lequel chaque ligne doit etre découpée suivant les '\t' et stockée. On ignorera chaque ligne malformée.

J'aimerais savoir, entre ces deux écritures, laquelle est la mieux, et pourquoi :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

while ( (line = buf.readline()) != null) { tab = line.split('\t'); if (tab.length == 4) { // on vérifie que tab[0] et tab[2] sont des entiers if (tab[0].matches("\\d+") && tab[2].matches("\\d+")) { a = Integer.parseInt(tab[0]); b = tab[1]; c = Integer.parseInt(tab[2]); d = tab[3]; } } }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

while ( (line = buf.readline()) != null) { tab = line.split('\t'); try { a = Integer.parseInt(tab[0]); b = tab[1]; c = Integer.parseInt(tab[2]); d = tab[3]; } catch (Exception e) {} }

---

voila, j'espere avoir été clair

merci d'avance :)

C'est une tautologie, mais les exceptions, c'est fait pour des cas exceptionnels. Qui merite donc un traitement particulier (comme avertir l'utilisateur que son fichier est corrumpu).

Attraper une exception pour finalement l'ignorer, c'est une tres mauvaise habitude.

D'ailleurs, dans le cas que tu presentes, tu attrapes n'importe quel type d'exception, et tu peux donc te retrouver à ignorer des exceptions importantes (une OutOfMemoryException par exemple)

Si les lignes peuvent ne pas respecter le format, vaut mieux la version avec des if.

L'utilisation des if doit être un peu plus rapide.

Je ne suis pas sûr que le coté exceptionnel (dans le sens rare) soit la meilleure approche. Pour moi, c'est une question de philosophie, de savoir ce qui est le cas imprévu.

Reprenons ton exemple avec une ligne formattée.

• On considère qu'elle est formattée, et au cours du traitement on s'aperçoit qu'elle est mal formatée :fleche: exception.
• On ne sait pas si elle est bien formattée, et on fait une méthode pour le tester :fleche: pas d'exception, on renvoie un booléen.
• Des fois pour la sécurité on fait tout en un : une méthode pour vérifier qu'un utilisateur est autorisé et qui renvoie une exception sinon.