Discussion :

[deubelte]

Yo! a tous.

Je travaille sur un soft qui possède beaucoup ce genre de code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#define DBMAPPING_NUM_NOT_NULL(FLD,OBJ,ATTR,TYPE) \
{ \
    TYPE V; \
		DBFieldSelector<RowStream::Row> &dbget_ref = FLD;\
    dbget_ref.fetchField(V); \
    if( !dbget_ref.isCursorValid()) { \
      RWXSTRStream str; str << dbget_ref.status().message(); \
      throw RWXMessage(RWXWARNING("Pb"), str); \
    } \
    if(V == TYPE::null) \
		V = TYPE(0); \
    OBJ.ATTR( V); \
}

Je voudrais savoir ce que vous en pensez. Au niveau de tout, de la rigueur, de l'efficacité, du C++. Estce que cela se fait encore ou c'est une vieille pratique. A quoi ca sert de faire ces Macro...
Merci

[Emmanuel Deloget]

C'est une très vieille pratique. Les macros, aujourd'hui, ne doivent être utilisées que lorsqu'elles fournissent un travail qu'aucune autre technique ne peut faire (par exemple, la stringification ou la concatenation de symboles).

Cette macro que tu présente ici a tout intérêt à être remplacé par une fonction inline placée dans un namespace anonyme (si elle n'est utilisée que dans un seul fichier), ou dans un namespace nommé dans un fichier header si plusieurs unités de compilation l'utilisent.