Discussion :

[vincenet]

Bonjour,

Jusque là pour suivre le déroulement d'un code, j'utilise un affichage avec des printf avec la macro msg() :

Code :

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

1
2
3
4
 
int pmesg(int level, int service, char* format, ...);
 
    #define msg(...) do{if(pmesg(__VA_ARGS__)) fprintf(stderr, " (%s, line %d, %s)\n", __FILE__, __LINE__, __func__);} while(0)

Maintenant, je souhaiterai ne plus utiliser les fonctions printf mais syslog de linux. J'ai réussi à utiliser syslog et vsyslog dans pmesg(), mais je ne vois pas comment récupérer __VA_ARGS__ et les macros __FILE__, __LINE__ pour générer un affichage syslog sur une même ligne.

En deux fois :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
int pmesg(const char *file, int line, int level, int service, char* format, ...);
 
#define msg(...) do{pmesg(__FILE__, __LINE__, __VA_ARGS__);} while(0)
 
 
int pmesg(const char *file, int line, int level, int service, char* format, ...) {
        //...
        va_start(args, format);
        syslog(LOG_LOCAL0|LOG_DEBUG, "message de test %s %d", file, line);
        vsyslog(LOG_LOCAL0|LOG_NOTICE, format, args);
        va_end(args);
        //...
}

Auriez-vous une idée pour concaténer les deux parties du message ?
Merci

[Sve@r]

Auriez-vous une idée pour concaténer les deux parties du message ?

Bonjour

A tout hasard, sprintf() pour "construire" un nouveau format en utilisant les deux premiers

Code c :

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

1
2
3
char reformat[1024];
sprintf(reformat, "message de test %%s %%d %s", format)
syslog(LOG_LOCAL0|LOG_DEBUG, reformat, file, line, args);

Et si sprintf() n'accepte pas les "%" situés dans le champ "format" alors utiliser strcpy() + strcat()...

[vincenet]

J'aurai souhaité ne pas créer une variable locale pour la chaîne et devoir contrôler les dépassements, mais dans le principe ça marche :

Code :

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

1
2
3
4
5
6
7
8
        char reformat[1024];
        char tailformat[256];
        sprintf(tailformat, " (%s, line %d)", file, line);
        va_start(args, format);
        vsprintf(reformat, format, args);
        strcat(reformat, tailformat);
        syslog(LOG_LOCAL0|LOG_DEBUG, reformat);
        va_end(args);

Plus qu'à utiliser les versions limitant le nombre de caractères pour les fonctions printf (vsnprintf() au lieu de vsprintf() ...).
Ce qui donne :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#define MSG_SIZE 10
#define TAIL_SIZE 10
        va_start(args, format);
        char reformat[MSG_SIZE+TAIL_SIZE];
        char tailformat[TAIL_SIZE];
        snprintf(tailformat, TAIL_SIZE, " (%s, line %d)", file, line);
        if(vsnprintf(reformat, MSG_SIZE+TAIL_SIZE, format, args) < MSG_SIZE+TAIL_SIZE)
        {
                strncat(reformat, tailformat, MSG_SIZE+TAIL_SIZE);
        }
        syslog(LOG_LOCAL0|LOG_DEBUG, reformat);
        va_end(args);

Je vous laisse ajuster la taille des tableaux ;-)

[dinobogan]

Tu pourrais faire directement :

Code :

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

1
2
 
vsyslog(LOG_LOCAL0|LOG_DEBUG, "message de test %s %d : " format, file, line, args);

[vincenet]

Tu pourrais faire directement :

Code :

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

1
2
 
vsyslog(LOG_LOCAL0|LOG_DEBUG, "message de test %s %d : " format, file, line, args);

L'idée me séduit, as-tu bien vu ça dans un code C ?

Code :

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

1
2
3
4
5
6
7
8
9
10
myLib.c: In function 'pmesg':
myLib.c:173:66: error: expected ')' before 'format'
         vsyslog(LOG_LOCAL0|LOG_DEBUG, "message de test %s %d : " format, file, line, args);
                                                                  ^
myLib.c:173:66: error: too few arguments to function 'vsyslog'
In file included from /home/vincent/oe_daisy/oecore/sysroots/armv5te-oe-linux-gnueabi/usr/include/syslog.h:1:0,
                 from myLib.h:16,
                 from myLib.c:4:
/home/vincent/oe_daisy/oecore/sysroots/armv5te-oe-linux-gnueabi/usr/include/sys/syslog.h:200:13: note: declared here
 extern void vsyslog (int __pri, const char *__fmt, __gnuc_va_list __ap)

[dinobogan]

L'idée me séduit, as-tu bien vu ça dans un code C ?

arf.... j'ai écrit trop vite sans vérifier et en lisant la question en diagonale...
J'ai développé une librairie de log utilisée sur plusieurs projets. Je passe par ma fonction intermédiaire et ensuite une chaine de 1 Ko pour construire la ligne de log.
C'est ma fonction de log qui est utilisable comme ça donc c'est assez éloigné de ta question initiale

Mais le principe de base dans ma librairie est d'utiliser les MACRO. Exemple :

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
#include <stdlib.h>
#include <stdio.h>
#include <stdarg.h>
 
#define CHAINE_GEANTE 2048
 
void mon_log( char * format, ... )
{
    va_list args;
    va_start( args, format);
    vprintf( format, args );
    va_end( args );
}
 
#define INFO_LOG( format, ar... ) mon_log( "ligne %d, fonction %s : "format"\n", __FILE__, __FUNCTION__, ##ar )
 
int main()
{
    INFO_LOG( "salut %d, ca roule ? %s", 56, "hello" );
    char chaine[] = "coucou";
    INFO_LOG( "%s", chaine );
    exit( 0 );
}

Il faut passer par une macro et une fonction.

[Médinoc]

Ça manque un peu de polyvalence, tu devrais toujours faire la version v:

Code C :

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
#include <stdlib.h>
#include <stdio.h>
#include <stdarg.h>
 
#define CHAINE_GEANTE 2048
 
void mon_logv( char const * format, va_list args )
{
    vprintf( format, args );
}
 
void mon_log( char const * format, ... )
{
    va_list args;
    va_start( args, format);
    mon_logv(format, args);
    va_end( args );
}
 
#define INFO_LOG( format, ar... ) mon_log( "ligne %d, fonction %s : "format"\n", __FILE__, __FUNCTION__, ##ar )
 
int main()
{
    INFO_LOG( "salut %d, ca roule ? %s", 56, "hello" );
    char chaine[] = "coucou";
    INFO_LOG( "%s", chaine );
    exit( 0 );
}

Aussi, c'est mieux d'accepter des pointeurs const quand on ne modifie pas le contenu.

[dinobogan]

Ça manque un peu de polyvalence, tu devrais toujours faire la version v:

Aussi, c'est mieux d'accepter des pointeurs const quand on ne modifie pas le contenu.

C'est ce que j'ai fait dans ma librairie
Le code posté était juste pour donner l'idée et exprimer à quel point mon intervention était à côté de la plaque sur la question initiale

[vincenet]

Bonjour et merci pour les réponses.

Cependant, cela ne m'apprend pas grand chose car il manque les paramètres dans vos exemples : le niveau d'importance du message "int level" et le service concerné "int service".

Donc voilà où j'en suis en essayant de coller à votre proposition :

Code :

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

msg(ERROR, MSG_JEU, "file path:%s does it exist ?", res); // exemple d'appel (que je ne souhaite pas changer)

macro et prototype :

Code :

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

1
2
3
4
#define msg(...) my_log(format" ligne %d, fonction %s", __VA_ARGS__, __FILE__, __FUNCTION__ )   
 
 
int my_log(int level, int service, char* format, ...);

Code :

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

1
2
3
4
5
6
myLib.h:58:31: error: too few arguments to function 'my_log'
 #define msg(...) my_log(format" ligne %d, fonction %s", __VA_ARGS__, __FILE__, __FUNCTION__ )
                               ^
jeu.c:1037:13: note: in expansion of macro 'msg'
             msg(ERROR, MSG_JEU, "file path:%s does not exist", res);
             ^

[Invité]

Bonsoir,

Pour commencer,

Code :

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

#define msg(...) my_log(format" ligne %d, fonction %s", __VA_ARGS__, __FILE__, __FUNCTION__ )

où est cette entrée format ?
Ensuite, tu définis ta fonction my_log comme suit :

Code :

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

int my_log(int level, int service, char* format, ...);

qu'as-tu fait des paramètres level et service ?

Les trois points dans msg(...) sont reportés à l'endroit où tu mets __VA_ARGS__, ce qui donne avec ton appel :

Code :

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

1
2
3
4
msg(ERROR, MSG_JEU, "file path:%s does it exist ?", res);
// =>
my_log(<non défini>" ligne %d, fonction %s", ERROR, MSG_JEU, "file path:%s does it exist ?", res, __FILE__, __FUNCTION__ )
//                                          [----------------- __VA_ARGS__ ---------------------]

ils ne vont pas miraculeusement s'éparpiller selon tes désirs.

Rajoute donc les paramètres level, service et format dans ta macro, comme ceci :

Code :

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

#define msg(level, systeme, format, ...) my_log(level, systeme, format" ligne %d, fonction %s", __VA_ARGS__, __FILE__, __FUNCTION__ )

Au passage un const sur le paramètre format ne ferait pas de mal.

[vincenet]

Merci.

J'ai simplement ajouté les ## et une virgule avant les ... dans la macro et je pense avoir compris le principe :

Code :

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

#define msg(level, service, format, ...) my_log(level, service, format" (ligne %d, file %s)", ##__VA_ARGS__, __LINE__, __FILE__ )

Ma question initiale est donc résolue.

Petite question bonus pour la gymnastique avec format et __VA_ARGS__

Dans la fonction mon_log(), l'affichage se fait finalement avec syslog :

Code :

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

        vsyslog(LOG_LOCAL0|LOG_DEBUG, format, args);

J'aimerai ajouter deux valeurs dans l'affichage (numéro du message et les millisecondes du temps courant). Je lai fait dans un premier temps en concaténant format avec une deuxième chaîne via les fonctions vsnprintf, snprintf et strncat et des buffers. Y aurait-il moyen d'éviter ces appels et buffers avec le principe de manipuler format et __VA_ARGS__ ?

Deuxième interrogation, si je fais :

Code :

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

	    msg(WARNING, MSG_XML, "une chaine : ", path); // %s manquant

Je n'ai aucun warning à la compilation :-(