Discussion :

[LinuxUser]

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 :

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

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 :

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

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.

[Neckara]

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 :

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

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

[LinuxUser]

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

Code :

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

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

[Neckara]

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

[LinuxUser]

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 :

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

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.

[Neckara]

Je n'aime pas trop cette syntaxe

Pourquoi ne pas faire :

Code :

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

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

?

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

[LinuxUser]

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

Code :

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

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 :

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

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 :

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

1
2
3
#define DEBUG ON
ou
#define DEBUG OFF

J'ai essayé :

Code :

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

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

[Neckara]

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"

[LinuxUser]

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 :

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

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

[LinuxUser]

Au final, je vais obter pour cela :

Code :

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

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.

[Iradrille]

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 :

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

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

[Neckara]

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

[Bousk]

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

Code :

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

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