[bloc finally] Comment contourner le finally clause cannot complete normally"

Bonjour,

J'ai besoin de tester si Un String contient un entier, j'ai utilisé cette méthode :

Code:

---

1 2 3 4 5 6 7 8 9 10

public static boolean isValidInt(String str) { boolean isValid = false; try { Integer newInput = Integer.valueOf(str); int i = newInput.intValue(); isValid = true; } finally{ return isValid; } }

---

Eclipse me retourne le warning suivant : "Le bloc finally ne se termine pas correctement."

En parcourant la FAQ Java, j'ai constaté qu'il vallait mieux éviter le return dans le bloc finally. FAQ Java

Je m'embrouille un peu avec la gestion des exceptions... Quelqu'un peut-il m'indiquer comment faire sans utiliser le "return isValid" dans le bloc finally ?

Merci

Oui mais le finally ne sers pas à ça !!!

Le finally sers pour libérer les ressources, à savoir connection SGBDR, fichier ouvert... mais pas a faire un return.

Citation:

---

Envoyé par pistache42

Bonjour,

J'ai besoin de tester si Un String contient un entier, j'ai utilisé cette méthode :

Code:

---

1 2 3 4 5 6 7 8 9 10

public static boolean isValidInt(String str) { boolean isValid = false; try { Integer newInput = Integer.valueOf(str); int i = newInput.intValue(); isValid = true; } finally{ return isValid; } }

---

Eclipse me retourne le warning suivant : "Le bloc finally ne se termine pas correctement."

En parcourant la FAQ Java, j'ai constaté qu'il vallait mieux éviter le return dans le bloc finally. FAQ Java

Je m'embrouille un peu avec la gestion des exceptions... Quelqu'un peut-il m'indiquer comment faire sans utiliser le "return isValid" dans le bloc finally ?

Merci

---

En effet il faut mieux éviter les return dans finally, ça peut poser quelque soucis (décrits dans la FAQ).

Moi je remplacerais le finally par un catch et je mettrait le return hors de la clause try-catch. En effet, le problème de ton code c'est que tu n'attrappe pas l'Exception qui peut se produire si la chaine de caractère n'est pas dans un bon format du fait de l'absence de clause catch.

Code:

---

1 2 3 4 5 6 7 8 9 10

public static boolean isValidInt(String str) { boolean isValid; try { int i = Integer.parseInt(str) isValid = true; } catch (Exception e){ isValid = false; } return isValid; }

---

Edit: et comme le dit thibault, la clause finally sert à placer les instructions devant être réalisées dans tout les cas donc typiquement à libérer les ressources utilisées

Dans ton cas ce serait plutôt ceci qu'il faudrait faire

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public static boolean isValidInt(String str) { boolean isValid = false; try { Integer newInput = Integer.valueOf(str); int i = newInput.intValue(); isValid = true; } catch (NumberFormatException e){ //faire quelquechose ou pas } return isValid }

---

En effet, c'est ce que je cherchais à faire...

j'avais trouvé ce bout de code sur le Net, et il me semblait bien que c'était une mauvaise utilisation du bloc finally.

Ca m'aura au moins permis de bien comprendre le fonctionnement des blocs try/catch/finally.

Merci pour votre aide.;)

Salut,


Juste une remarque (même si c'est un peu H-S) : Pourquoi utiliser Integer.valueOf() ?

En effet il serait ici préférable d'utiliser Integer.parseInt() qui renvoit un int (au lieu d'un Integer) et qui éviterait donc de créer un objet inutilement...

Code:

---

1 2 3 4 5 6 7 8 9 10

public static boolean isValidInt(String str) { boolean isValid = false; try { Integer.parseInt(str); // throw NumberFormatException isValid = true; } catch (NumberFormatException e){ // faire quelquechose ou pas } return isValid; }

---

a++

Merci adiGuba, c'est vrai que j'ai l'habitude d'utiliser Integer.parseInt(), comme je l'ai dit c'est une méthode que j'ai récupérée et j'ai pas fait attention à ce point.

J'ai pas encore bien le reflexe entre les int et Integer, je vais d'ailleurs revoir ce point de suite que j'ai déjà étudié mais que visiblement je n'ai pas intégré...:?

Mon expérience du java est encore très restreinte, mais j'avance petit à petit...:aie: