[Syntaxe] Un return dans un try... Comment faire ?

Version imprimable

Bonjour.

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

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.

13/01/2005, 14h28
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
13/01/2005, 14h37
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 !!!

Dans ton cas :

Code:

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);
   }

Code:

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

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

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 :wink:

13/01/2005, 14h58
thibaut

Ne pa faire de throws NumberFormatException, StringIndexOutOfBoundsException
13/01/2005, 15h02
nuke_y

lol, faut qu'on arrête là les mecs, il va nous cogner sinon ...
13/01/2005, 17h20
bobuse

Citation:

Envoyé par thibaut

Ne pa faire de throws NumberFormatException, StringIndexOutOfBoundsException

8O 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 !
13/01/2005, 17h28
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...
13/01/2005, 17h32
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...
13/01/2005, 17h38
nuke_y

C'est aussi qu'un StringIndexOutOfBounds ça doit pouvoir s'éviter en faisant un test avant !
13/01/2005, 17h51
bobuse

Je rapelle que le code n'est pas de moi :wink:

Citation:

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.

Citation:

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.
13/01/2005, 17h54
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...
14/01/2005, 10h33
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