Activer/Désactiver l'affichage des messages

Bonjour,

Je cherche une solution simple et portable (le plus possible) pour définir une macro permettant d'activer et désactiver l'affichage des messages (typiquement std::cout<<.....).

J'ai trouvé quelques manière de faire et j'ai retenue celle-ci :

Code:

---

1 2 3 4 5

#ifdef DEBUG #define D(x) x #else #define D(x) #endif

---

Sinon, je vois souvent quelque chose de très ressemble avec un do{} while() en plus (ou parfois un for( ; 0 ; ) ) mais je comprends pas l'interêt:

Code:

---

1 2 3 4 5

#ifdef DEBUG #define D(x) do {x} while (0) #else #define D(x) do {} while (0) #endif

---

De manière général si quelqu'un peut me conseiller "la bonne" manière de faire, je suis toute ouïe.

Merci de votre aide.

Il me semble que le do {} while(0) permet de :
- limiter la portée des variables ;
- faire comme si la valeur est "void".

Sinon, en C++, on peut écrire une fonction template "debug" :

Code:

---

1 2 3 4 5 6

template<typename T> void DEBUG(const T & t) { #ifdef DEBUG std::cerr << t << std::endl; #endif }

---

Ce qui te permettra par la même occasion de pouvoir surcharger ta fonction DEBUG pour, par exemple, afficher de manière différentes certains types (surtout pour les pointeurs).

Pourquoi "templatiser" si tout mes messages ont la même forme :

Code:

---

std::cout<<"\n\t\t"<<__FILE__<<":"<<__LINE__<<Error: ...";

---

Je veux juste definir un moyen pour faire : message d'erreur ON ou OFF (pour que tous les massages d'erreurs s'affichent ou non).

Tu pourras par exemple donner des objets à ta fonctions pour avoir plus de renseignements sur leur état (?)

Citation:

---

Envoyé par Neckara

Tu pourras par exemple donner des objets à ta fonctions pour avoir plus de renseignements sur leur état (?)

---

OK, j'y songerai plus tard.

Par contre, j'ai testé les solutions que j'ai citées (ci-dessus) et dans les deux cas rien ne s'affiche.
Si je fais par exemple:

Code:

---

1 2 3 4 5 6 7 8 9

#include "debug.h" #define DEBUG 1 ... int main() { D(std::cout<<"\n Test ";);   return 0; }

---

Le message ne s'affiche pas.

Je n'aime pas trop cette syntaxe :aie:

Pourquoi ne pas faire :

Code:

---

1 2 3

#define D(T) std::cout << (T) << std::endl;   D("texte" << 42 << "foo")

---

?

Cela t'éviteras d'oublier les std::endl :aie:

Tu as raison, je pense faire un truc du genre:

Code:

---

1 2 3 4 5

#ifdef DEBUG #define D(message) do {std::cout << message << std::endl;} while(false) #else #define D(message) do {} while(false) #endif

---

Par contre, coment faire pour mettre une condition sur la defintion de DEBUG?
J'ai essayé cela mais ça ne marche pas:

Code:

---

1 2 3 4 5 6 7

#ifdef DEBUG #define D(message) do {std::cout << message << std::endl;} while(false); if(1) #else #define D(message) do {} while(false); if(0) #endif

---

Idéalement, je souhaiterais avtiver/désactiver de cette manière:

Code:

---

1 2 3

#define DEBUG ON ou #define DEBUG OFF

---

J'ai essayé :

Code:

---

1 2 3 4 5 6 7 8 9

#ifdef DEBUG #if DEBUG == ON #define D(message) do {std::cout << message << std::endl;} while(false); #endif   #if DEBUG == OFF #define D(message) do {} while(false); #endif #endif

---

Quelque soit la valeur que je donne à DEBUG, les messages ne s'affchent pas,comme si la macro était systématiquement redéfinie (dans le if DEBUG == OFF).

Soit une des bibliothèque défini la macro DEBUG, soit c'est ton IDE/Compilateur qui le fait.

Essaye d'utiliser une macro "MYDEBUGPERSO"

C'est déjà une macro perso, et pour cette exemple je n'utilise aucune lib ni IDE.

Par contre en activant certains flag de compilation, le compilateur me dit bien que je suis entrain de redéfinir DEBUG lorsque je suis ici :

Code:

---

#if DEBUG == OFF

---

EDIT: j'ai l'impression que la condition #if n'est pas testée (ou est toujours vrai?) et qu'on rentre systematiquement dedans.


EDIT2: en fait je viens de me rendre compte que dans mon #if je faisais une comparaison de chaines de caractère, donc soit j'utilise une fonction faite pour cela, soit j'attribut une valeur entier à ON et OFF pour pouvoir les tester ensuite. :arf:

Au final, je vais obter pour cela :

Code:

---

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

#ifndef DEBUG_H #define DEBUG_H   #include <iostream>   #define OFF 0 #define ON 1   #define FLAG ON // ou OFF   #if FLAG == ON #define DEBUG #endif     #ifdef DEBUG #define DEBUG(str) do { std::cout << str << std::endl; } while( false ); #else #define DEBUG(str) do { } while ( false ); #endif   #endif /* DEBUG_H */

---

Si personne y voit d'erreurs ou de d'améliorations, je considère comme résolu.

Merci de votre aide.

Citation:

---

Envoyé par Neckara

Il me semble que le do {} while(0) permet de :
- limiter la portée des variables ;
- faire comme si la valeur est "void".

---

Mais pourquoi ce do while ? Un simple bloc suffirait non ? (reliquat du C peut être ?)

Code:

---

1 2	// assez inutile ici, mais c'est pour l'exemple #define DBG(str) { std::cerr << str << std::endl; }

---

Je n'ai pas lu en détail, mais cela pourrait être différent si on a des "if" :
http://stackoverflow.com/questions/1...vs-scope-block

Citation:

---

Envoyé par Iradrille

Mais pourquoi ce do while ? Un simple bloc suffirait non ? (reliquat du C peut être ?)

Code:

---

1 2	// assez inutile ici, mais c'est pour l'exemple #define DBG(str) { std::cerr << str << std::endl; }

---

---

Le do while t'indiquera une erreur si tu oublies le ; de fin de ligne, ça peut servir