Discussion :

[Mindiell]

Bonjour à tou(te)s,

J'ai entendu dire qu'en c++ il faut éviter les #define et utiliser les enum à la place. Grand bien me fasse, j'ai donc utiliser uniquement des enums.

Par contre, là je suis bloqué : les enum ne peuvent contenir que des entiers, pas de chaine de caractères. Comment faire ca proprement alors ?

Merci d'avance,

[dtrosset]

Avec des

Code :

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

char const * const maVariableTexte = "La valeur de la variable texte";

Didier

[3DArchi]

Salut,
En C++, ce serait plutôt :

Code :

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

std::string const maVariableTexte = "La valeur de la variable texte"

Dit autrement, ce ne sont pas les enum qui remplacent forcément des #define mais tout simplement une variable constante (joli oxymore) éventuellement statique.

[Mindiell]

Ah oui, mais ce n'est pas un peu dommage pour la structuration de ma classe ? (les enums font partie de ma classe, puisque ca la concerne directement)

Et puis 328 constantes, je ne sais pas, ca me parait pas propre

EDIT : surtout que j'appelle mes enums ainsi :
SetToto (MonEnum::MaValeur);

C'est plus parlant on sait tout de suite que MaValeur est une valeur de "type" MonEnum

EDIT 2 : Impossible d'initialiser ma variable dans la déclaration de ma classe

[3DArchi]

Ah oui, mais ce n'est pas un peu dommage pour la structuration de ma classe ? (les enums font partie de ma classe, puisque ca la concerne directement)

La constante peut être dans la classe (en static). Il n'y a pas de pb.

Et puis 328 constantes, je ne sais pas, ca me parait pas propre

Pourquoi ? D'autant qu'il y a de grande chance que ceci soit optimisé par le compilo.

EDIT : surtout que j'appelle mes enums ainsi :
SetToto (MonEnum::MaValeur);

C'est plus parlant on sait tout de suite que MaValeur est une valeur de "type" MonEnum

Tu peux toujours associer un enum à une chaîne de caractère si tu le souhaites. Remarque que le define ne t'assurait pas un typage non plus.

Quel est ton pb exactement ? Sérialiser ton enum en chaîne de caractères ? Utiliser des chaînes comme constantes mais vouloir garantir qu'elles sont bien celles que tu as définies ? Autre ?

EDIT 2 : Impossible d'initialiser ma variable dans la déclaration de ma classe

Non, ce doit être dans le .cpp :
.h :

Code :

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

1
2
3
4
struct my_struct
{
    static std::string const my_var;
};

.cpp :

Code :

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

1
2
 
std::string const my_struct::my_var("valeur");

[Mindiell]

Voici mon code, ca sera plus simple :

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
18
19
20
21
22
23
24
25
26
27
28
class Telnet : private sf::Thread
{
public :
    enum TelnetCommand
    {
        COMMAND_SE = 240,
        [bla bla bla]
        COMMAND_IAC
    };
    enum TelnetOption
    {
        OPTION_BINARY_TRANSMISSION,
        [bla bla bla]
        OPTION_EXTENDED_OPTIONS_LIST = 255
    };
    enum TelnetSubNegotiation
    {
        SUBNEGOTIATION_IS,
        SUBNEGOTIATION_SEND,
        SUBNEGOTIATION_INFO
    };
    //C'est ici mon souci :)
    enum TelnetTerminalType
    {
        ADDS_CONSUL_980 = "ADDS-CONSUL-980",
        [bla bla bla]
        ZENTEC_30 = "ZENTEC-30"
    };

L'erreur étant, bien entendu :

error: enumerator value for `ADDS_CONSUL_980' not integer constant

Si je mets les std::string à la place :

Code :

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

    std::string const ADDS_CONSUL_980 = "ADDS-CONSUL-980";

L'erreur sera dans ce cas :

error: ISO C++ forbids initialization of member `ADDS_CONSUL_980'
error: making `ADDS_CONSUL_980' static
error: invalid in-class initialization of static data member of non-integral type `const std::string'

[Mindiell]

.h :

Code :

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

1
2
3
4
struct my_struct
{
    static std::string const my_var;
};

Avec ca je ne pourrai pas utiliser une forme du type : my_struct::my_var ?
Ca sera plutôt my_struct.my_var, non ?

L'ennui c'est que j'ai tout le reste en enum.

En même temps, en tapant ca, je viens de me rendre compte que je ne suis pas obligé d'avoir des chaines. Je ne les utilise qu'à un seul endroit finalement ! Je vais tester. Merci !

[Laurent Gomila]

Avec ca je ne pourrai pas utiliser une forme du type : my_struct::my_var ?
Ca sera plutôt my_struct.my_var, non ?

C'est une variable statique, donc ce sera bien my_struct::my_var.

Sinon une autre solution est d'avoir un enum dont les valeurs sont des entiers arbitraires (plutôt que des chaînes), et faire une fonction qui renvoie la chaîne associée à chaque valeur de l'enum. Ca te permet d'avoir une solution plus fortement typée et donc plus sûre (puisque l'utilisateur devra forcément passer un membre de l'enum et non une chaîne qui pourrait très bien être n'importe quoi).

[Mindiell]

Sinon une autre solution est d'avoir un enum dont les valeurs sont des entiers arbitraires (plutôt que des chaînes), et faire une fonction qui renvoie la chaîne associée à chaque valeur de l'enum. Ca te permet d'avoir une solution plus fortement typée et donc plus sûre (puisque l'utilisateur devra forcément passer un membre de l'enum et non une chaîne qui pourrait très bien être n'importe quoi).

Eh eh
Je suis assez fier de moi-même : C'est exactement ce que je viens de tester !
Je demande un enum, et j'ai une méthode qui retourne la std::string équivalente !

Je suis aussi fort que Laurent !