Discussion :

[pistache42]

Bonjour,

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

Code :

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

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

[thibaut]

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.

[Deadpool]

Bonjour,

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

Code :

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

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 :

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

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

[sinok]

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

Code :

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

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
    }

[pistache42]

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.

[adiGuba]

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 :

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

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++

[pistache42]

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...