Problème de Switch en java

Bonjour voila mon problème:

je fais un exercice ou je dois faire une méthode d'instance pour comparer le nombre de pages entre 2 instances de Livre et cela avec un Switch.

voila mon code:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public int compare1(Livre l) { int a=l.getNbPages(); switch (nbPages){ case a : return 0; break; default: if(nbPages>l.nbPages) return 1; else return -1; } }

---

Mon erreur à la compilation me dit qu'il faut une constante dans le premier case, or a est une constante.

Merci de votre aide! :P

a n'est pas une constante, même si effectivement dans ton programme elle ne va pas changer, le compilateur n'a aucun moyen de le savoir.
Pour lui garantir, il faut lui déclarer qu'elle ne va jamais changer :

Code:

---

1 2	final int a=l.getNbPages();

---

Même en rajoutant final la même erreur apparait :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public int compare1(Livre l) { final int a=l.getNbPages(); switch (nbPages){ case a: return 0; break; default: if(nbPages>l.nbPages) return 1; else return -1; } }

---

Constant expression required :cry:

Mhhh effectivement, j'avais testé avec un final int a = 0. Dans ce cas j'imagine que le compilateur se rend compte qu'il peut remplacer dans le case par la valeur qui a été affectée en dur.

Dans ce cas, le mieux est de passer par la différence (l.getNbPages() - nbPages) et de comparer avec 0.
Cela dit, si c'est pour mettre un if à l'intérieur, c'est un peu curieux (enfin c'est bizarre à la base de vouloir passer par un switch dans ce cas). Si le nb de pages est maxé à petite valeur, tu peux mettre tous les cas ... ça reste vilain.

En fait la méthode je l'ai déjà faite avec un if et elle marche très bien mais il demande de la faire avec un while aussi dans l exercice...

Je ne comprends pas pourquoi cela ne marche pas.

Merci quand même, je demanderais lundi en cours. ;)

Les cases d'un switch doivent être des expressions constantes, pas seulement des variables finales.

Une variable finale n'aura pas la même valeur d'un appel à l'autre de la méthode. Une expression constante, si. En fait, sa valeur est connue du compilateur.

Exemples d'expressions constantes :

3
1 + 5

Et si on a quelque part :

Code:

---

static final int ZERO = 0;

---

ZERO
ZERO + 5
ZERO * ZERO

Salut,

Citation:

---

Envoyé par alex4444

Même en rajoutant final la même erreur apparait :

---

final ne permet pas à lui seul de définir une constante. Ici tu as une variable avec une valeur fixe, mais qui ne peut pas être déterminée à la compilation.

:arrow: Les valeurs des constantes sont connu à la compilation, ce qui permet au compilateur de les utiliser correctement.

Citation:

---

Envoyé par alex4444

Merci quand même, je demanderais lundi en cours. ;)

---

Oui c'est très bizarre de vouloir faire cela avec un switch :koi:

a++

[edit] grillé ;)

tu peux résoudre ton problème avec un switch sur le signe de la différence

Code:

---

1 2 3 4 5 6 7

int d = this.nbPages - l.nbPages; // attention si l == null int sign = d != 0? Math.abs(d)/d : 0; //remarque de Rei Ichido //Le switch ne sert vraiment à rien mais bon!! switch(sign) { default: return sign; }

---

Sauf que si la différence vaut 0, ça pose un petit souci :)

Vraiment, le switch dans ce cas là ... beuh :'(