Discussion :

[jul54]

Salut,
dans mon programme j'ai le bout de code suivant:

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
 
int Number = pTree->GetNumberIcon(IID_PREPARATION);
switch (RowIcone)
{
	case Number:
	{
	m_pPreparationPageGeneral = new CPreparationPageGeneral(this);
	AddPage(m_pPreparationPageGeneral);
	m_pPreparationPageEtiquetage = new CPreparationPageEtiquetage(this);
	AddPage(m_pPreparationPageEtiquetage);
	m_pPreparationPageCaracterPhy = new CPreparationPageCaracterPhy(this);
	AddPage(m_pPreparationPageCaracterPhy);
	}
	break;
}

Seulement à la compilation, j'ai le message d'erreur suivant:

error C2051: case expression not constant

Donc d'après lui l'entier Number n'est pas constant, j'avoue ne pas trop saisir la raison de ce message et MSDN n'est pas plus locasse pour expliquer ce qui pourrait déclencher cette erreur.

Merci d'avance pour toute proposition

[farscape]

salut ,
le pb c'est qu'a mon avis ça ne peut marcher que si l'on fait ce type de sequence:

Code :

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

1
2
3
4
5
6
7
 
        const int nCase=10;
 
	switch(nVar)
	{
	  case nCase:// 
        }

ou nCase est une constante definit.
parce qu'un switch avec un case variable moi j'ai jamais vu du moins avec vc .
faudrait que je regarde dans le bouquin de stroustrup si il en parle ...

[jul54]

Merci de ta réponse,
mon problème c'est que mon numéro n'est pas variable, il est tout le temps égal à 6.

Ma fonction sert juste à récupérer la variable de la classe (variable private que je récupère donc par une fonction publique).
Je vais essayer de déclarer cette variable membre constante pour voir.

[farscape]

apres verifications dans le soustrup il ne parle que de valeurs constantes donc
soit par des enums soit la constante elle même ou une variable constante pre initialise comme dans mon code en dehors de ça je vois pas.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
enum token_value{
NAME=2,NUMBER,END
};
switch(curr_tok)
{
case NAME:
case NUMBER:
// etc....
}

[jul54]

Merci pour tout,
je vais voir ce que je peux faire, parce que mettre la valeur en dure dans le programme, c'est vraiment moche, dès fois qu'on les modifie un jour, ça va être la merde.