Utilisation d'une énumération

Bonjour à tous,

Considérons cette énumération:

Code:

---

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:

---

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:

---

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

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:

---

1 2 3 4

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

---

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:

---

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:

---

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"

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

Code:

---

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...

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)

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.

OK, va pour le static_cast alors ;)


Merci beaucoup :)


A bientôt