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.