LIB de LOG et TRACE

Version imprimable

LIB de LOG et TRACE

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:

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:

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

24/01/2008, 13h43
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....
24/01/2008, 13h54
cmoibal

Citation:

Envoyé par vicenzo

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 ?
24/01/2008, 18h02
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:

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

Citation:

Envoyé par ram_0000

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:

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