enum et champs de bits

**FamiDoo** · 12/07/2006, 09h33

Bonjour,
j'ai le problème suivant :

eEtat1 = (pstFormule->acFormule[0] != '\0' ? F_NoM : 0) ;

eEtat1 |= (pstPrestation->usPresence & PRESENCE_AMC ? NoF_M : 0);

eEtat1 est un enum déclaré de la sorte :

enum { NoF_NoM = 0, F_NoM, NoF_M, F_M } eEtat1;

Toutefois, j'obtiens les erreurs suivantes :

error C2440: '=' : cannot convert from 'int' to ''
Conversion to enumeration type requires an explicit cast (static_cast, C-style cast or function-style cast)

error C2676: binary '|=' : '' does not define this operator or a conversion to a type acceptable to the predefined operator

merci pour toute aide

**Médinoc** · 12/07/2006, 09h40

Bonjour,
À moins d'être en .Net, les enums ne supportent pas les opérateurs bit-à-bit : L'enum est implicitement convertie en int (sans compter que 0 est déjà un int.
Il faut donc forcément utiliser un static_cast (et pour cela, ce sera beaucoup plus simple si tu donne un nom à ton enum).

**FamiDoo** · 12/07/2006, 09h43

même en C ?

**Matthieu Brucher** · 12/07/2006, 09h46

Envoyé par FamiDoo

même en C ?

Aucune idée, mais ça m'étonnerait que ce ne soit pas le cas.

**Médinoc** · 12/07/2006, 09h50

En C, il me semble que la conversion implicite int --> enum est encore possible.

**FamiDoo** · 12/07/2006, 10h02

et pour le binary je peux faire quoi ?

**Médinoc** · 12/07/2006, 10h08

Tu donnes un nom à ton enum et tu rajoutes des casts.
de plus, tu vas sûrement devoir remplacer

Code C :

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

eEtat1 |= calcul;

par

Code C++ :

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

eEtat1 = static_cast< nom_de_l_enum >( eEtat1 | calcul );

**FamiDoo** · 12/07/2006, 10h23

eEtat1 = static_cast<etat> (pstFormule->acFormule[0] != '\0' ? F_NoM : 0) ;

eEtat1 = static_cast<etat> ( eEtat1 | (pstPrestation->usPresence & PRESENCE_AMC ? NoF_M : 0) );

Voila la correction

.

Un grand merci.

**Laurent Gomila** · 12/07/2006, 10h39

Moi je préfère ce genre de chose beaucoup plus claire quant aux intentions du codeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if (pstFormule->acFormule[0] != '\0')
{
    if (pstPrestation->usPresence & PRESENCE_AMC)
        eEtat1 = F_M;
    else
        eEtat1 = F_NoM;
}
else
{
    if (pstPrestation->usPresence & PRESENCE_AMC)
        eEtat1 = NoF_M;
    else
        eEtat1 = NoF_NoM;
}

**Médinoc** · 12/07/2006, 10h53

Sauf que ce code ne respecte PAS les intentions du codeur...

Où as-tu vu un troisième else dans son code ? Il est question de flags, ici...

Edit: J'ai rien dit...