Trace arguments erreur

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:

---

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

---

Ainsi, je peux faire :

Code:

---

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

---

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

Code:

---

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) :

Citation:

---

erreur : expected primary-expression before ')' token

---

L'erreur peut être évitée avec

Code:

---

TRACE("test", "");

---

mais c'est pas très pratique..

Une idée ?

Merci ;)

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 :aie:

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

Code:

---

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

---

autre solution, une fonction inline ou dans une lib.

Code:

---

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); }

---

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

Code:

---

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

---

Salut,

Citation:

---

Envoyé par Astraya

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

Code:

---

#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:

---

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

---

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

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 ;) :D

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:

---

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:

---

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 :mouarf:)

@ 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:

---

#define TRACEMANAGER TraceManager::GetInstance()

---

et ensuite tu utilises ta fonction comme-t-elle :)

@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:

---

#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:

---

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

---

Merci à tous pour vos réponses :)

Citation:

---

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.

Citation:

---

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 :)