Discussion :

[alex4444]

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 :

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

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!

[Rei Ichido]

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 :

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

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

[alex4444]

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

Code :

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

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

[Rei Ichido]

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.

[alex4444]

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.

[adiGuba]

Salut,

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.

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

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

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

a++

[edit] grillé

[thelvin]

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 :

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

static final int ZERO = 0;

ZERO
ZERO + 5
ZERO * ZERO