Discussion :

[semaj_james]

Bonjour,

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

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

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

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

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.

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

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

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

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

[thibaut]

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 :

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

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+

[®om]

Pourtant, ce code compile:

Code :

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

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 :

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

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

[jgavard]

J'avoue ma grand faute. Désolé pour mes ânneries.

[adiGuba]

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 :

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

	final int a = 1;

a est une constante qui vaut 1.


Alors qu'avec ce code :

Code :

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

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