Discussion :

[cmoibal]

Bonjour a tous,

alors voici, j'ai ecrie un fichier log.h qui contient des MACRO permettant de créer une lib de Trace, et la voila :

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
23
24
25
26
27
28
29
30
31
#ifndef LOG_H_INCLUDED
#define LOG_H_INCLUDED
#define LOG_FILE    "/tmp/apc.log"
#ifdef DEBUG_CONSOLE
#define LOG_START()
#define LOG(mesg, args...)   fprintf(stderr, "%s: (function %s line %d):",__FILE__,__FUNCTION__,__LINE__); \
                             fprintf(stderr, mesg "\n", ##args)
#define LOG_END()
#endif  // DEBUG_CONSOLE

#ifdef DEBUG_FILE
#define LOG_START()         FILE *log_file_decriptor; \
                            if((log_file_decriptor=fopen(LOG_FILE, "w"))==NULL) \
                            LOG("LOG can not start.")
#define LOG(mesg, args...)  fprintf(log_file_decriptor, "%s: (function %s line %d):",__FILE__,__FUNCTION__,__LINE__); \
                            fprintf(log_file_decriptor, mesg "\n", ##args)
#define LOG_END()           if(fclose(log_file_decriptor)!=0) \
                            LOG("LOG ERROR")
#endif  // DEBUF_FILE

 

#if !defined(DEBUG_FILE) || !defined(DEBUG_CONSOLE)
#define LOG_START()
#define LOG(mesg, args...)
#define LOG_END()
#endif
 
#endif // LOG_H_INCLUDED

pour le LOg sur la console sa marche avec DEBUG_CONSOLE
mais pour les fichier ça ne marche pas et je sais pas pourquoi ?

j'utilise le fichier comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include <log.h>
 
void function(type args, ...)
{
     LOG_START();
     ...
     LOG(Message);
     ...
     LOG_END();
}

[Vincent Rogier]

Petite remarque en passant :

Tout le monde n'a pas un compilo C99 sous la main !

Ce code ne peut pas être compilé avec les compilos Microsoft par exemple ( pour ne citer qu'eux...bon, c'est pas un grand mal mais quand même !)

Le C99 n'est pas impémenté par tout le monde (c'est pas forcément un grande perte ... pour que cela apporte !)

c'est dommage de ne pas être "portable" (au sein réalité du terrain) pour des fonctionnalités de base comme du log....

[cmoibal]

Petite remarque en passant :

Tout le monde n'a pas un compilo C99 sous la main !

Ce code ne peut pas être compilé avec les compilos Microsoft par exemple ( pour ne citer qu'eux...bon, c'est pas un grand mal mais quand même !)

Le C99 n'est pas impémenté par tout le monde (c'est pas forcément un grande perte ... pour que cela apporte !)

c'est dommage de ne pas être "portable" (au sein réalité du terrain) pour des fonctionnalités de base comme du log....

vraiment une trés trés bonne remarque... MERCI

Meis pour le moment est ce que il y a une erreur dans le code ?

[ram-0000]

Une autre remarque en passant, d'après ce que je comprend, il faut que LOG_START(), LOG() et LOG_END() soient dans la même fonction pour que le descripteur de fichier soit connu

Que se passe t'il si tu as des appels imbriqués, je pense que cela va poser un problème

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
 
void fct_1(void)
{
   LOG_START();
   LOG("mon message");
   fct_2();
   LOG_END();
}
 
void fct_2(void)
{
   LOG_START();
   LOG("mon autre message");
   LOG_END();
}

En plus, c'est un peu lourd d'appeler LOG_START(), LOG() et LOG_END() à chaque fois.
Une autre solution serait d'avoir LOG_START() et LOG_END() dans la fonction main() et ensuite tu mets tes LOG() où tu veux dans le code sans soucis d'appel imbriqué.

[Cvortex]

Une autre solution serait d'avoir LOG_START() et LOG_END() dans la fonction main() et ensuite tu mets tes LOG() où tu veux dans le code sans soucis d'appel imbriqué.

Ce n'est pas suffisant. Il aussi faut que FILE *log_file_decriptor soit une variable globale.
Fait comme ca dans ton .h:

Code :

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

1
2
3
4
5
 
extern FILE *log_file_decriptor;
#define LOG_START()          \
                            if((log_file_decriptor=fopen(LOG_FILE, "w"))==NULL) \
                            LOG("LOG can not start.")

et declare ta variable dans un .c du style global.c

Ainsi tu pourras faire un LOG_START en début de ton main, un LOG_END à la fin, et ensuite tu pourras utiliser LOG() n'importe où.