Cast des enum en int

**olibara** · 09/01/2009, 19h55

Bonjour

C'est un problème sans doute idiot, mais je ne comprends pas pourquoi je suis systématiquement forcé de caster mes enum en int alors que le type est de l' int ?

Meme dans un switch case !

Je trouve ca fastidieux et inutile !

**StormimOn** · 09/01/2009, 20h01

Pour un switch tu n'as pas besoin de caster, à moins que j'ai mal compris.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
MonEnum truc;
...
switch(truc)
{
    case MonEnum.Toto:
        break;
    case MonEnum.Tata:
        break;
    case MonEnum.Titi:
        break;
    default:
        break;
}

Pour les cas où tu dois manipuler l'enum comme un entier tu n'as pas le choix par contre. C'est ça d'utiliser un typage fort, mais c'est drôlement bien pratique ^^

**olibara** · 09/01/2009, 20h31

Salut Stormimon

Pour le switch, c'est peut etre moi qui ai pas bien compris la bonne maniere d'utiliser l'enum

Je declare un Enum

Code :

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

1
2
3
4
5
6
7
8
   enum Alerts
    {
      V_ON=71,
      V_OFF=72,
      S_SHUTDOWN=73,
      VIOLATION=75,
      MOTION=76
    }

Et dans un switch utilisant un int je teste les valeurs de l'enum
Et je suis obligé de caster !

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
       switch (l.type)
        {
          case (int) Alerts.V_ON:
            lAlarm.Add(l);
            break;
          case (int)Alerts.VIOLATION:
            lAlarm.Add(l);
            break;
          case (int)Alerts.MOTION:
            lAlarm.Add(l);
            break;
          case (int)Alerts.S_SHUTDOWN:
            lAlarm.Add(l);
            break;
          default:
            break;
        }

**Greybird** · 09/01/2009, 22h51

Envoyé par olibara

Et dans un switch utilisant un int je teste les valeurs de l'enum
Et je suis obligé de caster !

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
       switch (l.type)
{
          case (int) Alerts.V_ON:
            lAlarm.Add(l);
            break;
          case (int)Alerts.VIOLATION:
            lAlarm.Add(l);
            break;
          case (int)Alerts.MOTION:
            lAlarm.Add(l);
            break;
          case (int)Alerts.S_SHUTDOWN:
            lAlarm.Add(l);
            break;
          default:
            break;
        }

Ca me semble assez inefficient, dans la mesure où ça risque de faire un cast par comparaison...

Même si ce point serait à vérifier en regardant le code IL, il me semblerait plus naturel de transformer l.type en une valeur de l'enum avant de faire le switch sans cast.

Code :

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

1
2
3
4
5
6
7
 
if (!Enum.IsDefined(typeof(Alerts), l.type)
{
  throw new NotSupportedException("...");
}
Alerts alert = (Alerts) l.type
// ou

L'avantage de l'enum par rapport à l'int c'est que la plage de valeurs possibles est fixée et donc vérifiable à la compilation, au moins tant que tu utilises bien les valeurs de l'enum dans ton code (i.e. Alerts.XXX), et que tu n'assignes pas directement des valeurs numériques qui pourraient se trouver en dehors des valeurs possibles de l'enum. Par ailleurs la classe Enum fournit un ensemble de méthodes permettant de récupérer les valeurs possibles, de vérifier la validité d'une valeur, etc.

**tomlev** · 09/01/2009, 23h08

Envoyé par olibara

Pour le switch, c'est peut etre moi qui ai pas bien compris la bonne maniere d'utiliser l'enum

Je pense aussi... pourquoi tu ne déclares pas type comme étant de type Alerts ? Contrairement aux enums de C++, ceux de C# sont fortement typés, autant en profiter

**Greybird** · 09/01/2009, 23h17

Envoyé par tomlev

Contrairement aux enums de C++, ceux de C# sont fortement typés, autant en profiter

Attention tout de même à la notion de "fortement typé" : il est tout à fait possible de positionner une valeurs n'appartenant pas à l'enum dans une instance de cet enum :

Code :

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

Alerts alert = (Alerts) 5;

est tout à fait valide.
D'ailleurs une instance d'Alerts non initialisée à une autre valeur aura pour valeur numérique 0, qui, n'appartenant pas à l'enum, ne correspond à aucune valeur attendue.

Très intéressant donc, mais à condition d'en appréhender les limites.

**olibara** · 10/01/2009, 00h13

Et bien voila un bon débat et eclaire ma lanterne et améliore ma connaissance de l'enum en Charp

Je n'ai jamais eté tres pratiquant des enums mais c'est vrai qu'avec un peu de pratique c'est fort utile.

Merci a tous.

**StormimOn** · 10/01/2009, 09h37

Histoire de finir avec les switch et pour en revenir à ton exemple de départ, cela donnerait donc ceci au final

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
switch ((Alerts)l.type)
{
    case Alerts.V_ON:
        lAlarm.Add(l);
        break;
    case Alerts.VIOLATION:
        lAlarm.Add(l);
        break;
    case Alerts.MOTION:
        lAlarm.Add(l);
        break;
    case Alerts.S_SHUTDOWN:
        lAlarm.Add(l);
        break;
    default:
        break;
}

**olibara** · 10/01/2009, 09h53

Tout a fait !

Merci Stormimonn, merci a tous.

**smyley** · 10/01/2009, 19h15

Envoyé par StormimOn

Histoire de finir avec les switch et pour en revenir à ton exemple de départ, cela donnerait donc ceci au final

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
switch ((Alerts)l.type)
{
    case Alerts.V_ON:
        lAlarm.Add(l);
        break;
    case Alerts.VIOLATION:
        lAlarm.Add(l);
        break;
    case Alerts.MOTION:
        lAlarm.Add(l);
        break;
    case Alerts.S_SHUTDOWN:
        lAlarm.Add(l);
        break;
    default:
        break;
}

Juste en passant, le code si dessous est équivalent :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
switch ((Alerts)l.type)
{
    case Alerts.V_ON:
    case Alerts.VIOLATION:
    case Alerts.MOTION:
    case Alerts.S_SHUTDOWN:
        lAlarm.Add(l);
        break;
    default:
        break;
}