enum et champs de bits

Version imprimable

12/07/2006, 09h33
FamiDoo

enum et champs de bits

Bonjour,
j'ai le problème suivant :

Citation:

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 :

Citation:

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

Toutefois, j'obtiens les erreurs suivantes :

Citation:

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
12/07/2006, 09h40
Médinoc

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).
12/07/2006, 09h43
FamiDoo

même en C ?
12/07/2006, 09h46
Matthieu Brucher

Citation:

Envoyé par FamiDoo

même en C ?

Aucune idée, mais ça m'étonnerait que ce ne soit pas le cas.
12/07/2006, 09h50
Médinoc

En C, il me semble que la conversion implicite int --> enum est encore possible.
12/07/2006, 10h02
FamiDoo

et pour le binary je peux faire quoi ?
12/07/2006, 10h08
Médinoc

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

Code:

eEtat1 |= calcul;

par

Code:

eEtat1 = static_cast< nom_de_l_enum >( eEtat1 | calcul );
12/07/2006, 10h23
FamiDoo

Citation:

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.

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

Code:

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;
}

12/07/2006, 10h53
Médinoc

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...
12/07/2006, 11h24
FamiDoo

Il n'est pas équivalent ton code Laurent
12/07/2006, 12h41
Laurent Gomila

Je veux bien vous croire, mais je n'arrive pas à voir ce qui est différent.
12/07/2006, 13h22
Médinoc

C'est simple: Nos yeux.
Ils ne sont pas en face des trous...