case expressions must be constant expressions

Version imprimable

Bonjour,

J'ai un soucis avec un switch, j'ai l'erreur suivante sur chaque case: case expressions must be constant expressions

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
private final int stop=new Color(128,128,128+8).getRGB();
private final int cp=new Color(128,128,128+9).getRGB();  
private final int feuV=new Color(128,128,128+10).getRGB();
private final int feuO=new Color(128,128,128+11).getRGB();
private final int feuR=new Color(128,128,128+12).getRGB();
...
 
 
fonction(){
 
switch(getPatchColorAt(X,Y).getRGB()){
 
case feuR:
	return feuR+";"+X+";"+Y;
case feuO:
	return feuO+";"+X+";"+Y;
case feuV:
	return feuV+";"+X+";"+Y;
case cp:
	return cp+";"+X+";"+Y;
case stop:
       return stop+";"+X+";"+Y;
case carrefour:
	return carrefour+";"+X+";"+Y;
default:
	return "OK";
}
return "erreur";
}

Pourtant mes variables sont declarees final et sont des int comme dit dans la faq. qu'est ce qui ne va pas ?

Cordialement

18/12/2006, 12h06
Baptiste Wicht
Je sais pas pourquoi, c'est ainsi, mais il faut les déclarer explicitement et non pas faire appel à une méthode pour les initialiser...

Par exemple, si tu change comme ça :
Code:

1 2 3 4 5 6 private final int stop = 1; private final int cp = 2; private final int feuV = 3; private final int feuO = 4; private final int feuR = 5;
Ca va fonctionner.

Par contre, je sais pas comment faire pour résoudre ton problème et que ca marche, mais il doit y avoir un truc certainement.
18/12/2006, 12h11
adiGuba

Citation:

Envoyé par semaj_james

Pourtant mes variables sont declarees final et sont des int comme dit dans la faq. qu'est ce qui ne va pas ?

Parce que le fait qu'elle soit final ne suffit pas pour que ce soit une constante.
Il faut également que sa valeur puisse être déterminé à la compilation (et non pas à l'exécution).

Ainsi, avec ce code :

Code:

final int a = 1;

a est une constante qui vaut 1.

Alors qu'avec ce code :

Code:

final int a = new Color(128,128,128+12).getRGB();

a est une variable immuable, dont la valeur est défini à l'exécution, et qui ne peut pas être connu à la compilation.

Pour être plus clair, le compilateur ne peut pas déterminer si le nombre renvoyé par getRGB() sera toujours le même, et ne le considère donc pas comme une constante.

De ce fait, il ne peut pas utiliser cette valeur dans le switch...

a++

Ce qui doit suivre les case sont des String, or feuV, feuR sont des références vers des int donc ça ne marche pas.

Par contre, peut-être devrais-tu creuser sur :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 fonction(){ switch(getPatchColorAt(X,Y).getRGB().toString()){ case feuR.toString(): return feuR+";"+X+";"+Y; case feuO.toString(): ... default: return "OK"; } return "erreur"; }
Ah oui, effectivement, ce que dit adiGuba est totalement exact, j'avais pas pensé à ça...

18/12/2006, 12h19
xv-mnt

jgavard, étonnant ce que tu dis !
Les switch ne marchent QUE pour les int, et les types enumérés depuis j2se 1.5 ! :?
Je ferais plutôt confiance aux propos de adiguba...:mrgreen:

Citation:
Envoyé par jgavard
Ce qui doit suivre les case sont des String, or feuV, feuR sont des références vers des int donc ça ne marche pas.

Par contre, peut-être devrais-tu creuser sur :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 fonction(){ switch(getPatchColorAt(X,Y).getRGB().toString()){ case feuR.toString(): return feuR+";"+X+";"+Y; case feuO.toString(): ... default: return "OK"; } return "erreur"; }
Ah oui, effectivement, ce que dit adiGuba est totalement exact, j'avais pas pensé à ça...
Non, non non je ne crois pas, depuis quand on peut faire un switch sur autre chose qu'un int ou un type Enum !!!!

Primo:
Cher jgavard tu ne dois pas connaitre java pour dire une telle ânerie... Merci quand tu veux aider d'essayer de ne pas dire n'importe quoi. Je sais pas où tu as tu pêcher ce code, qui ne doit en aucun cas compiler... Alors avant de balancer un exemple essaie de le compiler.

Secundo:
Des références vers des int, je ne connaissais pas... c'est nouveau ça vient de sortir !!! Je ne savais pas que le type int était un objet, merci jgavard pour tes lumières ça remet en cause presque 10 années de pratique de java.

A+

Pourtant, ce code compile:
Code:

1 2 3 4 5 6 7 public static void main(String... args) { Color color = Color.BLACK; final int k = 2; switch(color.getBlue()) { case k: } }
...
EDIT: Oui en fait g compris le problème, c'est que les valeurs des champs doivent être connus à la compilation... Avec un appel à une méthode, c'est pas connu à la compilation...

Ainsi, ce code ne compile pas...
Code:

1 2 3 4 5 6 7 public static void main(String... args) { Color color = Color.BLACK; final int k = new Color(128,128,128+8).getRGB(); switch(color.getBlue()) { case k: //erreur } }

18/12/2006, 13h41
jgavard

J'avoue ma grand faute. Désolé pour mes ânneries.
21/12/2006, 09h57
semaj_james

Je vous remercie.
17/07/2007, 17h28
BiM

Merci pour ces réponses ;)