Discussion :

[sfarc]

Bonjour,

Pour pouvoir garder des traces dans mon application, j'ai mis au point un singleton TraceManager. La logique tourne autour de la ligne suivante :

Code :

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

#define TRACE(fmt, ...) TraceManager::GetInstance()->Trace(__FUNCTION__, __FILE__, __LINE__, fmt, __VA_ARGS__)

Ainsi, je peux faire :

Code :

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

TRACE("valeur de i : %d", i); // i variable connue

Sous Visual Studio 2010, j'arrive également à tracer sans argument comme :

Code :

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

TRACE("hello")

Par contre, lorsque j'essaie de tracer sans argument sous QtCreator, j'obtiens l'erreur suivante (la ligne considérée correspond à celle où TRACE est appelée) :

erreur : expected primary-expression before ')' token

L'erreur peut être évitée avec

Code :

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

TRACE("test", "");

mais c'est pas très pratique..

Une idée ?

Merci

[therwald]

Mais pourquoi le ; te paraît-il "superflu dans un cas et pas dans l'autre? Sinon, pourquoi ne pas l'intégrer à la macro?
Oops, lu trop vite, répondu trop vite, grillé par leternel dans ma réponse-rectif

[ternel]

Peut-être un gros #define TRACELINE(msg) quelque choseAvec éventuellement une fonction dédiée, telle que

Code :

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

TraceManager::Trace(__FUNCTION__, __FILE__, __LINE__, msg)

autre solution, une fonction inline ou dans une lib.

Code :

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

1
2
3
4
5
6
inline void trace(const char* fmt, ...) {
    va_list args;
    va_start(args);
    TraceManager::GetInstance()->Trace(__FUNCTION__, __FILE__, __LINE__, fmt, args);
    va_end(args);
}

[Astraya]

Simple supposition mais rajouter un ";" a la fin ça ferais pas l'affaire?

Code :

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

#define TRACE(fmt, ...) TraceManager::GetInstance()->Trace(__FUNCTION__, __FILE__, __LINE__, fmt, __VA_ARGS__);

[koala01]

Salut,

Simple supposition mais rajouter un ";" a la fin ça ferais pas l'affaire?

Code :

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

#define TRACE(fmt, ...) TraceManager::GetInstance()->Trace(__FUNCTION__, __FILE__, __LINE__, fmt, __VA_ARGS__);

Oui, mais non...

L'habitude quand on travaille avec des macros, consiste à ne justement pas rajouter de ";" à la fin, pour laisser l'utilisateur seul juge du moment où il le place.

Le fait est qu'une macro va simplement définir un symbole comme l'équivalent du code qui le défini et que ce symbole sera purement et simplement remplacé par son contenu lorsque le préprocesseur va passer par là.

Seulement, on n'a jamais la moindre certitude de la manière (peut etre erronée, mais pas forcément) dont la macro pourrait être utilisée, et on pourrait très bien avoir un code (tout à fait juste au demeurant) proche de

Code :

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

foo( TRACE("valeur de i: %d",i),somethingElse);

parce que foo prendrait, en l'occurrence, un pointeur de fonction comme premier argument

Si tu places le point virgule directement dans la macro, un tel appel devient totalement illégal, et le compilateur se plaindra d'avoir un "somethingElse" qui n'a pas à se trouver là (en plus de se plaindre de ne pas avoir de correspondance avec foo(/* pointeur de fonction*/) )

Bref, non: on ne place pas de ";" en fin de macro

[jo_link_noir]

Il y a eu toute une histoire de point virgule, mais n'est-ce pas juste un oublie du PO ?

Sinon, j'imagine que la macro avec un seul paramètre donne:

Code :

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

 TraceManager::GetInstance()->Trace(__FUNCTION__, __FILE__, __LINE__, fmt, )

Et comme la parenthèse suit la virgule, il y a une erreur de syntaxe.

Un moyen serait de tout mettre en variadic (#define TRACE(...)), mais on perd le nom du paramètre.

Une autre solution serait de compter le nombre de paramètre dans __VA_ARGS__ et ajouter une virgule si différent de 0.

Code :

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

1
2
3
4
5
6
7
#define TRACE(fmt, ...) TraceManager::GetInstance()->Trace(__FUNCTION__, __FILE__, __LINE__, fmt \
  BOOST_PP_IF( \
    PP_VARIADIC_SIZE(__VA_ARGS__),  \
    BOOST_PP_COMMA, \
    BOOST_PP_EMPTY \
  )() \
  __VA_ARGS__)

La macro PP_VARIADIC_SIZE retournant le nombre de paramètre.
Je n'ai pas réussi à le faire avec seulement boost preprocessor, c'est peut-être possible mais j'ai une macro qui se comporte comme tel ici (C'est juste monstrueux )

[Astraya]

@ Koala:
Voila pourquoi je n'aime pas, je hais, j’exècre les macros et essaye d'en supprimer le plus possible :p

Ce que je recommanderais pour limiter la casse: une macro pour juste get ton singleton

Code :

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

#define TRACEMANAGER TraceManager::GetInstance()

et ensuite tu utilises ta fonction comme-t-elle

[jo_link_noir]

@Astraya: Il faut quand même faire appel au 'macro magique' sinon __FUNCTION__, __LINE__ et __FILE__ feront référence à la fonction de formatage et non à l'emplacement de la macro.

Code :

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

#define TRACE TraceManager::GetInstance()->trace(__FUNCTION__, __FILE__, __LINE__).format

Par contre, j'ai trouver un truc sympa pour ajouter une virgule si __VA_ARGS__ n'est pas vide: concaténation (##)

Code :

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

#define TRACE(fmt, ...) TraceManager::GetInstance()->Trace(__FUNCTION__, __FILE__, __LINE__, fmt,## __VA_ARGS__)

[sfarc]

Merci à tous pour vos réponses

Bref, non: on ne place pas de ";" en fin de macro

C'est ce que je me disais. Après effectivement, dans le main pour mes tests, quand j'appelle la macro TRACE("Test"), j'ai oublié de mettre clairement le ";" mais ce n'est pas le problème.

Par contre, j'ai trouver un truc sympa pour ajouter une virgule si __VA_ARGS__ n'est pas vide: concaténation (##)

Pas mal comme solution, ça semble fonctionner!

Merci encore