Discussion :

[chuky]

Bonjour.

Comment faut-il s'y prendre pour utiliser un return dans un Try, par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
private Color stringToColor(String paramValue)
	throws NumberFormatException
	{
		int red = 0;
		int green = 0;
		int blue = 0;
 
		try
		{
			red = (Integer.decode("0x" + paramValue.substring(1,3))).intValue();
			green = (Integer.decode("0x" + paramValue.substring(3,5))).intValue();
			blue = (Integer.decode("0x" + paramValue.substring(5,7))).intValue();
			return new Color(red,green,blue);	
		}
		catch (StringIndexOutOfBoundsException e)
		{
			System.err.println(e);
		}
	}

Evidement, Eclipse n'est pas d'accord, il me dit que la fonction doit retourner une variable Color. Je suis bien d'accord, mais comment gérer l'exception dans ce cas ?

Je crois qu'il y a une notion en java que j'ai pas bien compris.


Un grand merci d'avance pour votre aide.

Antoine.

[FreshVic]

En gros ce que le compilateur n'aime pas c'est que si une StringIndexOutOfBoundException est lancé il ne retourne rien alors que la fonction dois retourné un objet Color !!

Donc la solution est de (dans ton catch) lancer une Exception ou alors de retourné un objet color qui serais le Color par defaut en cas d'erreur !!

Si tu lance une exception n'oublie pas de la specifié dans le throws de la methode

[thibaut]

Tu as deux solutions:

Soit tu renvoie une valeur équivalente a une erreur : null par exemple, soit tu retransforme ton exception en une exception propre a ton appliation...

Une autre solution si tu ne veux pas de ces deux cas: c'est de ne pas catcher l'exception : StringIndexOutOfBoundsException n'est pas à catcher car c'est une RuntimeException !!!

[nuke_y]

Dans ton cas :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
private Color stringToColor(String paramValue) 
   throws NumberFormatException 
   { 
      int red = 0; 
      int green = 0; 
      int blue = 0; 
 
      try 
      { 
         red = (Integer.decode("0x" + paramValue.substring(1,3))).intValue(); 
         green = (Integer.decode("0x" + paramValue.substring(3,5))).intValue(); 
         blue = (Integer.decode("0x" + paramValue.substring(5,7))).intValue(); 
      } 
      catch (StringIndexOutOfBoundsException e) 
      { 
         System.err.println(e);
         return new Color(0,0,0);
      } 
return new Color(red,green,blue);
   }

ou

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
private Color stringToColor(String paramValue) 
   throws NumberFormatException 
   { 
      int red = 0; 
      int green = 0; 
      int blue = 0; 
      Color color=null;
 
      try 
      { 
         red = (Integer.decode("0x" + paramValue.substring(1,3))).intValue(); 
         green = (Integer.decode("0x" + paramValue.substring(3,5))).intValue(); 
         blue = (Integer.decode("0x" + paramValue.substring(5,7))).intValue(); 
      color = new Color(red,green,blue);
} 
      catch (StringIndexOutOfBoundsException e) 
      { 
         System.err.println(e);
         color = new Color(0,0,0);
      } 
return color;
   }

Mais je plussoie pour le fait que StringIndexOutOfBoundsException nbe doit pas être catchée !!

[bobuse]

une autre solution est de faire remonter l'exception comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
private Color stringToColor(String paramValue) 
   throws NumberFormatException, StringIndexOutOfBoundsException
   { 
      int red = 0; 
      int green = 0; 
      int blue = 0; 
 
         red = (Integer.decode("0x" + paramValue.substring(1,3))).intValue(); 
         green = (Integer.decode("0x" + paramValue.substring(3,5))).intValue(); 
         blue = (Integer.decode("0x" + paramValue.substring(5,7))).intValue(); 
         return new Color(red,green,blue);    
   }

Je trouve ça plus propre, car catcher une exception et ne rien en faire (ou ne pas savoir quoi en faire) : bof ! Autant rebalancer la patate chaude, en espérant qu'au dessus ils sauront quoi en faire

[thibaut]

Ne pa faire de throws NumberFormatException, StringIndexOutOfBoundsException

[nuke_y]

lol, faut qu'on arrête là les mecs, il va nous cogner sinon ...

[bobuse]

Ne pa faire de throws NumberFormatException, StringIndexOutOfBoundsException

Heu où est le problème ?
Catcher une exception sans rien faire, je vois pas l'intérêt !
Retourner un objet null n'arrange pas les choses, car ça ne donne pas d'info sur l'origine du dysfonctionnement. A moins que ce soit dans les specs de la fonction, mais faut m'expliquer ...
En faisant remonter les exceptions, on peut en faire ce qu'on veut !

[thibaut]

Oui tu peux les catcher les RuntimeException mais ça ne sers a rien de faire un throws sur ce type d'exception, car elles sont remontées obligatoirement...

Danc ce cas remonte une exception personnalisée...

[thibaut]

Est ce que dans ton cas l'origine du dusfonctionnement (comme tu dis) est pertinante...

A mon avis non, qu'est ce que va vouloir savoir un utilisateur de ton appli ???

Il va vouloir savoir qu'il y à eu un problème, mais le fond du problème, il s'en tape...

La solution avec des erreurs techniques est de les logger dans un fichier ou sur la console est de renvoyer doit null, soit de remonter une exception personnalisée... Mais l'utilisateur s'en fout de StringIndexOutOfBounds...

[nuke_y]

C'est aussi qu'un StringIndexOutOfBounds ça doit pouvoir s'éviter en faisant un test avant !

[bobuse]

Je rapelle que le code n'est pas de moi

Est ce que dans ton cas l'origine du dusfonctionnement (comme tu dis) est pertinante...

A mon avis non, qu'est ce que va vouloir savoir un utilisateur de ton appli ???

Il va vouloir savoir qu'il y à eu un problème, mais le fond du problème, il s'en tape...

Bien sûr que ça ne sert à rien de la donner à l'utilisateur l'exception. Mais j'ai supposé que la fonction en question (stringToColor) était appelée par une autre fonction qui a un contexte que je ne connais pas.

Oui tu peux les catcher les RuntimeException mais ça ne sers a rien de faire un throws sur ce type d'exception, car elles sont remontées obligatoirement...

Ben c'est mieux que les choses soient claires ! On prend un risque d'avoir une exception sans la catcher, on le sait à cet endroit du code. En amont (à un autre endroit du code), on ne le sait pas forcément. Sauf si la signature de la fonction signale les exceptions qu'elle est succeptible de lancer.

[thibaut]

Quoi qu'il en soit tu as toujours le risque d'avoir des exceptions sans les catcher a savoir pour exemple NullPointerException si paramValue est nulle alors...

[chuky]

Salut.

Merci beaucoups pour vos réponses, en fait je ne voyais pas la chose comme ça, j'avais pas bien compris le système d'exception.

En tout cas, merci de m'avoir répondu aussi vite, je commençais à criser devant mon code

Je vais utiliser la technique qui conciste à retourner une couleur par defaut, elle me parait pas mal du tout.


Ciaooo