EnumToString : problème de conversion

Bonjour,

Souhaitant convertir en string des enums, j'ai suivi les étapes du lien http://www.codeproject.com/Articles/...ums-to-strings.

Seulement, à la compilation, j'ai le warning suivant :

Citation:

---

***\Test\Days.h:22: avertissement : deprecated conversion from string constant to 'char*' [-Wwrite-strings]

---

Pour information, je développe sous QtCreator.

Savez-vous comment je pourrais corriger ce problème ?

Aussi, dans la ligne

Code:

---

#define BEGIN_ENUM( ENUM_NAME ) typedef enum tag##ENUM_NAME

---

, je ne comprends pas tout à fait ce que représente "tag##ENUM_NAME". Pouvez-vous m'expliquer ?

Merci :)

C'est une macro (évaluée par le préprocesseur, donc)

## à pour effet de placer dans le texte de remplacement un unique token, composé par la concaténation des deux tokens indiqué.

ainsi, en supposant #define join(a,b) a ## b, le code int join(x,1) = 2; est strictement équivalent à int x1 = 2;pour ton warning, c'est un problème de const-correctness. a priori, c'est que tu utilise une fonction prenant un char* auquel tu donnes une constante (genre "message").

Pour résoudre le problème, il faut soit déclarer un char[], soit changer l'argument pour être un const char* (ce qui est généralement possible, si c'est ton code)

En ultime recours, utilise l'option -E de gcc.
Cela devrait générer un code source préprocessé, donc sans plus de macro.
Cela pourrait t'aider.

Remplace char * GetString, char * gs_ etc. par des déclarations de const char *.

Citation:

---

je ne comprends pas tout à fait ce que représente "tag##ENUM_NAME". Pouvez-vous m'expliquer ?

---

Si tu fais BEGIN_ENUM(Days), tu auras une enum de type enum tagDays avec dessus un typedef enum tagDays Days;

Merci leternel et Médinoc :).

Je comprends mieux les macros de la première condition (GENERATE_ENUM_STRINGS). Dans mon cas par contre, vu qu'il y a le warning dû à la conversion de string en const char* (#element retourne la chaîne de caractères associée?), il va plutôt évaluer les macros du "else" donc il ne va pas créer d'enum ? Pourquoi la première condition n'est pas remplie ?

Sinon, j'ai essayé de remplacer dans l'exemple par des const char* mais pour la deuxième macro (gs_...), j'ai l'erreur :

Citation:

---

***\Test\Days.h:23: erreur : use of enum 'tagDays' without previous declaration

---

Pourtant, je reprends l'exemple avec :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

BEGIN_ENUM(Days) { DECL_ENUM_ELEMENT(sunday), DECL_ENUM_ELEMENT(monday), DECL_ENUM_ELEMENT(tuesday), DECL_ENUM_ELEMENT(wednesday), DECL_ENUM_ELEMENT(thursday), DECL_ENUM_ELEMENT(friday), DECL_ENUM_ELEMENT(saturday) } END_ENUM(Days)

---

Comme l'indique le code, selon des defines, le code utilisera la version du #if ou celle du #else.

Tu mets ta définition d'enum dans le fichier header sans définir GENERATE_ENUM_STRINGS.

Puis tu inclus ça dans un fichier source à part, d'abord sans définir GENERATE_ENUM_STRINGS puis en le définissant (si j'ai bien compris). Ce fichier source contiendra la fonction qui retourne les strings.

tu dois placer un "const" pour les trois mentions de char.
les deux char* et le char []

cela dit, ca ne résoudra pas ton problème.

Ton problème me semble être à l'autre bout du code, dans la signature de la fonction à qui GetStringDays(…) est donnée en argument.

Oui bien sûr, je comprends la démarche.

Dans l'exemple, on voit d'ailleurs clairement le fichier source EnumToString qui appelle la macro GENERATE_ENUM_STRINGS donc pourquoi on se trouve dans l'autre condition ?

C'est d'ailleurs bien cette deuxième condition qui me génère un warning car si je la commente, je n'ai plus de problèmes.
Bien sûr, le mieux serait de corriger ce warning et de savoir dans quelle condition cette deuxième suite de macro peut être utile. Pour l'instant, ces deux points restent problématiques.

Peux-tu poster ton code exact?

Bien sûr, il est joint à cette réponse ;)

Citation:

---

Ton problème me semble être à l'autre bout du code, dans la signature de la fonction à qui GetStringDays(…) est donnée en argument.

---

La macro pour le GetString a l'air pourtant correcte. Surtout que je ne l'ai pas modifié depuis.

Chez moi avec MFC et sans Unicode, ces trois fichiers sources compilent parfaitement sous Visual 2005.

Et cette version du tester marche sans MFC:

Code:

---

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 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43

#include <iostream> #include <tchar.h> #include "Days.h" using namespace std;   void MyFunction( Days day );   int _tmain(int argc, _TCHAR* argv[], _TCHAR* envp[]) { int nRetCode = 0;   Days currentDayOfWeek = tuesday; //But for me, it was Tuesday. MyFunction( currentDayOfWeek );   (cout << "Presser [Entree] pour quitter: ").flush(); int n; cin >> n;   return nRetCode; }   void MyFunction( Days day ) { switch( day ) { case monday: case tuesday: case wednesday: case thursday: case friday: { cout << "Today is " << GetStringDays(day) << ", I have to work!" << endl; } break;   case saturday: case sunday: { cout << "Today is " << GetStringDays(day) << ", very nice!!!" << endl; } break; } }

---

Avec ou sans la modif const du header:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

// File name: "EnumToString.h" #undef DECL_ENUM_ELEMENT #undef BEGIN_ENUM #undef END_ENUM   #ifndef GENERATE_ENUM_STRINGS #define DECL_ENUM_ELEMENT( element ) element #define BEGIN_ENUM( ENUM_NAME ) typedef enum tag##ENUM_NAME #define END_ENUM( ENUM_NAME ) ENUM_NAME; const char* GetString##ENUM_NAME(enum tag##ENUM_NAME index); #else #define DECL_ENUM_ELEMENT( element ) #element #define BEGIN_ENUM( ENUM_NAME ) const char* gs_##ENUM_NAME [] = #define END_ENUM( ENUM_NAME ) ; const char* GetString##ENUM_NAME(enum tag##ENUM_NAME index){ return gs_##ENUM_NAME [index]; } #endif

---

Citation:

---

Chez moi avec MFC et sans Unicode, ces trois fichiers sources compilent parfaitement sous Visual 2005.

---

Je n'ai aucun problème aussi à le faire fonctionner sous VS2010.

Le problème intervient lorsque je développe avec l'environnement de développement QtCreator en fait.

Sur le même projet simpliste disponible ci-joint, j'ai exécuté sous VS2010 puis sous QtCreator. Les deux captures d'écran réalisées montrent la différence..

J'ai le même problème avec GCC -Wall -Wextra -Werror.

Cela dit, j'imagine que le compilo de visual possède une option similaire.

... Il n'y a pas de ligne 26 dans days.h.

Ah, j'ai pigé! g++ n'apprécie pas une déclaration de fonction prenant une enum en paramètre si l'enum n'est pas définie avant (et le pire, c'est qu'il me semble que le standard est d'accord avec lui).

Essaie en changeant EnumToString.cpp comme ceci:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

// File name: "EnumToString.cpp"   /// The strings associated with the enums are gererated here ///////////////////////////////////////////////////////////////////// #include "Days.h" //#include "OtherEnum.h" //#include "AnotherOne.h"   #define GENERATE_ENUM_STRINGS // Start string generation #include "Days.h" //#include "OtherEnum.h" //#include "AnotherOne.h" #undef GENERATE_ENUM_STRINGS // Stop string generation

---

Ca a l'air mieux, en effet.
C'est "marrant" que je ne l'ai pas vu avec gcc -E :aie:

Citation:

---

Ah, j'ai pigé! g++ n'apprécie pas une déclaration de fonction prenant une enum en paramètre si l'enum n'est pas définie avant (et le pire, c'est qu'il me semble que le standard est d'accord avec lui).

---

Bien vu Medinoc :D, ça fonctionne enfin correctement!

Merci beaucoup à vous deux pour votre aide :ccool:.

Citation:

---

Envoyé par Médinoc

Ah, j'ai pigé! g++ n'apprécie pas une déclaration de fonction prenant une enum en paramètre si l'enum n'est pas définie avant (et le pire, c'est qu'il me semble que le standard est d'accord avec lui).

---

Idem pour Visual C++.

La version 2005, celle sous laquelle j'ai testé le code, l'accepte sans discuter.

Edit: ↓Oui, 2010 respecte mieux les standards il me semble.

Il me semble pourtant avoir eu le problème il y a quelques jours sur VS 2010.

Bonjour,

Toujours sur le même sujet, avec un exemple simple visible sur http://www.codeproject.com/Articles/...ums-to-strings, j'ai du mal à comprendre l'intérêt des directives suivantes :

Code:

---

1 2 3 4 5 6

// File name: "Days.h" #if ( !defined(DAYS_H) || defined(GENERATE_ENUM_STRINGS) )   #if (!defined(GENERATE_ENUM_STRINGS)) #define DAYS_H #endif

---

Surtout concernant la première double condition avec en deuxième partie defined(GENERATE_ENUM_STRINGS). Admettons que le header soit déjà définie, si c'est en cours de génération de string, la première double condition sera remplie pour sortir juste après ?

Aussi, dans EnumToSting, on aurait envie d'ajouter les directives

Code:

---

1 2	#ifndef ENUMTOSTRING_H #define ENUMTOSTRING_H

---

pour pas que le fichier soit parcouru à chaque inclusion (chaque nouveau header qui définit une nouvelle énumération...). Mais quand on fait ça, on obtient l'erreur suivante :

Citation:

---

erreur : multiple definition of 'enum tagDays'

---

Quelqu'un pourrait m'éclairer un peu sur le comportement ?

Merci :)

Il FAUT que EnumToString.h puisse être inclus plusieurs fois, parce qu'il a deux modes: Avec et sans GENERATE_ENUM_STRINGS.

Days.h est compliqué pour la même raison: Quand GENERATE_ENUM_STRINGS est défini, on ignore la protection contre l'inclusion multiple, car dans le .cpp on a besoin de l'inclure deux fois (une fois sans GENERATE_ENUM_STRINGS, une fois avec).

Citation:

---

Il FAUT que EnumToString.h puisse être inclus plusieurs fois, parce qu'il a deux modes: Avec et sans GENERATE_ENUM_STRINGS.

---

Oui effectivement!

Citation:

---

Days.h est compliqué pour la même raison: Quand GENERATE_ENUM_STRINGS est défini, on ignore la protection contre l'inclusion multiple, car dans le .cpp on a besoin de l'inclure deux fois (une fois sans GENERATE_ENUM_STRINGS, une fois avec).

---

Justement, quand GENERATE_ENUM_STRINGS est défini, la deuxième condition n'est pas remplie (

Code:

---

1 2 3

#if (!defined(GENERATE_ENUM_STRINGS)) #define DAYS_H #endif

---

) et donc le header ne sera pas redéfini. Du coup, je dirais plutôt qu'on se protège ainsi de l'inclusion multiple non ? En gros, tel que je le comprends, avec le premier include du cpp, le header Days va être défini et ensuite avec l'inclusion dans le GENERATE_ENUM_STRINGS, le non remplissage de la deuxième condition va l'empêcher d'être redéfini.
Du coup, je ne vois pas ce qu'apporte la deuxième partie de la double condition avec le GENERATE_ENUM_STRINGS..

Tu interprètes mal le truc.

• Si GENERATE_ENUM_STRINGS n'est pas défini, on a un garde d'inclusion normal: On vérifie que DAYS_H n'est pas défini (sinon on saute tout l'en-tête) et on le définit.
• Si GENERATE_ENUM_STRINGS est défini, on inclut tout l'en-tête même si DAYS_H est déjà défini, et on ne le définit pas soi-même.

D'un autre côté, c'est vrai qu'avec l'erreur sur l'enum, le fait de ne pas définir DAYS_H soi-même ne sert pas à grand-chose, vu que l'inclusion avec GENERATE_ENUM_STRINGS doit forcément avoir lieu après l'inclusion sans. Mais ce code était prévu pour Visual, où cette erreur n'existe(ait) pas.

Citation:

---

Tu interprètes mal le truc.
Si GENERATE_ENUM_STRINGS n'est pas défini, on a un garde d'inclusion normal: On vérifie que DAYS_H n'est pas défini (sinon on saute tout l'en-tête) et on le définit.
Si GENERATE_ENUM_STRINGS est défini, on inclut tout l'en-tête même si DAYS_H est déjà défini, et on ne le définit pas soi-même.

---

Ok donc avec l'inclusion nécessaire avec le GENERATE_ENUM_STRINGS, on est obligé en fait de parcourir deux fois le header (une première pour définir l'énum et une deuxième quand la génération des strings est en cours) c'est ça ?

Du coup, à la première inclusion, on a les directives du cas ou GENERATE_ENUM_STRINGS n'est pas définie et ensuite, on redéfinie après des undef en utilisant cette fois-ci les directives du cas GENERATE_ENUM_STRINGS.

C'est exactement ça.

Parfait!

Merci beaucoup Médinoc :ccool: