Discussion :

[mister3957]

Bonjour � tous,

Consid�rons cette �num�ration:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
 
enum MonEnum
{
    ValeurA,
    ValeurB,
    ValeurC,
    Valeur1,
    Valeur2,
    Valeur3
};

J'ai une fonction qui prend un entier :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void MaFonction(unsigned int uiEntier)
{
    MonEnum eValue;
    switch(uiEntier)
    {
        case 1:
           eValue = Valeur1; break;
        case 2:
           eValue = Valeur2; break;
        case 2:
           eValue = Valeur2; break;
    }
}

Que j'aimerai transformer en

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 
void MaFonction(unsigned int uiEntier)
{
     MonEnum eValue = Valeur1 + (uiEntier - 1);
}

Mais le compilateur m'indique qu'il ne peut pas transformer un unsigned int en MonEnum.

Que dois-je faire ?


Merci beaucoup


A bient�t

[M�dinoc]

Il te faudra faire un cast explicite avec static_cast<>.
Le C++ n'accepte d�j� pas les op�rations arithm�tique sur les enums, il les convertit en int; l�, tu fais carr�ment une op�ration arithm�tique sur une enum et un int, donc...

Code C++ :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
void MaFonction(unsigned int uiEntier)
{
     MonEnum eValue = static_cast< MonEnum >(Valeur1 + (uiEntier - 1));
}

[ram-0000]

A mon avis, c'est risqu� ce que tu veux faire (m�me si cela marche)

Peut �tre que plus tard tu voudras modifer ton enum comme cela :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
 
enum MonEnum
{
    ValeurA = 12,
    ValeurB = 17,
    ValeurC = 20,
    Valeur1 = 5,
    Valeur2 = 8,
    Valeur3 = 1
};

Et la ta fonction ne marchera plus du tout.

Je pr�f�re :

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
 
void MaFonction(unsigned int uiEntier)
{
    MonEnum eValue;
    switch(uiEntier)
    {
        default:
           eValue = ValeurA; break; // valeur par defaut
        case 1:
           eValue = Valeur1; break;
        case 2:
           eValue = Valeur2; break;
        case 2:
           eValue = Valeur2; break;
    }
    return eValue;
}

Qui a le m�rite d'�tre plus "safe"

[screetch]

d'un autre cot� peut etre un jour voudras tu transformer ton enum comme cela :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
enum MonEnum
{
    ValeurA,
    ValeurB,
    ValeurC,
    Valeur1,
    Valeur2,
    Valeur3,
    Valeur4,
    Valeur5,
    Valeur6
};

et la c'est la seconde version qui fonctionnera. et ce genre de modif arrive plus souvent que rendre les index aleatoires...

[mister3957]

Ouais c'est clair que �a encourage les effets de bords ce que je souhaite faire.
Mais en m�me temps j'ai une �num�ration qui va de Valeur1 � Valeur31... un switch de 31 "case"... c'est moche quand m�me.

Sinon j'ai vu sur le net MonEnum(Valeur1 + x)

[M�dinoc]

Ne le fais pas et utilise le static_cast � la place.
Un cast function-style, c'est aussi dangereux qu'un cast C-style: En C++, c'est g�n�ralement � bannir.