enum - macros

Bonjour à tous !
Je dois créer une liste de tables dans un fichier séparé(par macro).

Jusqu’à maintenant, les tables de mon programme étaient crées avec des enums :

Fields.h

Code:

---

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

enum Table { // user data tables table_events = 1, table_activities = 2, // table_notes = 3, // no longer in use table_variables = 4, table_availability_ranges = 5,   // local table metadata, not part of the repository table_local_store_data = 9000, };*/

---

fields.cpp

Code:

---

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

static NameTablePair name_table_pairs[name_table_pair_count] = { // user data tables { "events", table_events }, // no longer in use, except by unit tests { "activities", table_activities }, // { "notes", table_notes }, // no longer in use { "variables", table_variables }, { "availability_ranges", table_availability_ranges },   // local table metadata, not in repository { "local_store_data", table_local_store_data }, };

---

je dois modifier ces fichiers de façon à créer les tables par macro, je voulais savoir si je m'y prenais bien, et si ce n'est pas le cas, pourquoi :)

j'ai rajouté un fichier field.inc comportant les macros
Fields.inc

Code:

---

1 2 3 4 5 6 7

#define Table \ name_table_pairs(events, 1) \ name_table_pairs(activities, 2) \ name_table_pairs(variables, 4) \ name_table_pairs(availability_ranges, 5) \ name_table_pairs(table_local_store_data, 5)

---

j'ai modifié le fichier .h pour 'remplir' mes enum

Code:

---

1 2 3 4 5 6 7

enum Table { #define name_table_pairs(a, b) table_##a, #undef name_table_pairs, #include "Fields.inc", #undef name_table_pairs };

---

j'ai pas mal d'erreurs au niveau de mon enum dans le .h, savez-vous d’où ça vient ?

Merci de vos conseils :)

Bonjour,

avant toute chose, sais-tu ce qu'est une macro et comment ça fonctionne ?
C'est juste quelque chose qui sera remplacé avant la compilation.

Donc quelque soit la forme choisie, tu dois avoir le même résultat

Code:

---

1 2 3

int i = 0; #define VAR(type, name, val) type name = val VAR(int, i, 0);

---

Et autour de ça, tu as 2 mot-clés principalement qui sont #define et #undef, respectivement pour déclarer et supprimer une macro.

Code:

---

1 2 3 4

#define name_table_pairs(a, b) table_##a, #undef name_table_pairs, #include "Fields.inc", #undef name_table_pairs

---

Tes macro font quoi ?
Tu déclares une macro qui prend 2 paramètres, et n'en utilise qu'un seul
la ligne suivante tu la supprimes
Une virgule après un include
Le fichier inclus lui ne fait que déclarer une macro
Puis tu supprimes une macro qui n'existe pas

Puisque tu as l'original, vas-y par étape..

Code:

---

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

enum Table { // user data tables table_events = 1, table_activities = 2, // table_notes = 3, // no longer in use table_variables = 4, table_availability_ranges = 5,   // local table metadata, not part of the repository table_local_store_data = 9000, };

---

pourrait devenir

Code:

---

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

#define BEGIN_ENUM(name) enum name { #define END_ENUM() }; #define ENUM(name, value) name = value,   BEGIN_ENUM(table) // user data tables ENUM(table_events, 1) ENUM(table_activities, 2) // ENUM(table_notes, 3) // no longer in use ENUM(table_variable, 4) ENUM(table_availability_ranges, 5) // local table metadata, not part of the repository ENUM(table_local_store_data, 9000) END_ENUM()

---

etc..

Merci beaucoup, je vais suivre à la lettre vos conseils.

En effet, je n'ai jamais programmé avec des macros, j'ai essayé de m'inspirer de ce que je pouvais trouver sur le net, les forums etc etc :/

Merci pour votre réponse, je vais faire comme ça :)

Je ne met pas le sujet résolu tout de suite au cas ou j'ai d'autres soucis concernant ceci

Mouais, enfin, tous les bons auteurs, à commencer par Stroustrup, conseillent d'éviter au maximum les macros en C++...

c'est ce que j'ai cru comprendre vu les réactions sur le chat :/
Mais je suis en stage, et c'est ce qu'on me demande.

Ne connaissant pas assez la matière, je préfère exécuter ^^

Là on comprend...

En fait, le soucis en fait, est que je dois obligatoirement générer le code de l'enum par une macro d'un fichier externe (le .inc)

J'ai trouvé de l'aide sur le chat, je vais poster la réponse ici

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

// Fields.inc // tes name_table_pairs(...) pas dans le #define et ',' manquants #ifndef table #define table name_table_pairs(events, 1) , name_table_pairs(activities, 2) , name_table_pairs(variables, 4) , name_table_pairs(availability_ranges, 5) , name_table_pairs(table_local_store_data, 9000) #endif   // ton .cpp   enum Table { #define name_table_pairs(a, b) table_##a = b // oublié = b #include "./Fields.inc" #undef name_table_pairs };

---

Merci à vous tous :)

Citation:

---

Envoyé par ptyxs

Mouais, enfin, tous les bons auteurs, à commencer par Stroustrup, conseillent d'éviter au maximum les macros en C++...

---

Il ne faut pas dramatiser les macros sous prétexte qu'on est en C++.

Une macro a une utilité simple qu'il faut utiliser correctement : remplacer une donnée avant la compilation.
Ca permet de réaliser certaines opérations sympas, de déclarer des infos et actions à la compilation plutôt qu'au run-time, ...
Ou tout simplement d'avoir une syntaxe plus sympa, éviter les répétitions de copier/coller de code, ...

Bref, à ne pas jeter arbitrairement.

Il y a tout un tas de bonnes raisons d'utiliser ces 'disgusting macros' (selon les termes de Stroustrup) le moins souvent possible en C++ et seulement dans les rares cas où c'est réellement souhaitable. Voir par exemple là-dessus Scott Meyers, Effective C++, Third Edition, Item 2.

Voir aussi dans la FAQ de Stroustrup, ce passage :
http://www.stroustrup.com/bs_faq2.html#macro
et celui-ci :
http://www.stroustrup.com/bs_faq.html#unsafe