Discussion :

[tnarol]

Salut,

J'ai récupéré la bibliothèque Poco qui me permet de logger des messages compatibles avec le type std::string.

Grosso modo il y a une methode comme ça :

Code :

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

1
2
 
void debug(std::string& s);

Mon idée est de permettre d'utiliser cette methode avec une std::stringstream
Donc je peux créer un adaptateur qui fait :

Code :

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

1
2
3
4
5
 
void stream_debug(std::stringstream& stream)
{
      instanceLogger.debug(stream.str());
}

Pour que le logger soit pratique à utiliser je voudrais pouvoir passer des streams "composées" directement, par exemple :

Code :

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

1
2
3
 
double d = 1.2345;
stream_debug("la valeur de d est " << d);

Mais ça ne marche pas car la chaine constante ne supporte pas "<<".
Donc j'ai imaginé le contournement suivant :

Code :

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

1
2
3
4
 
#define LOG(a) stringstream s; s << a << endl; stream_debug(s);
double d = 1.2345;
LOG("la valeur de d est " << d);

Ca marche mais est ce qu'il y a mieux à faire ???
Peut-on se passer de la macro ?

Merci

[Laurent Gomila]

Un logger qui ne permet pas de faire des sorties façon flux C++, ça me paraît déjà pas terrible. Tu es sûr que Poco ne le propose pas ?

Sinon au pire tu peux te fabriquer ton propre flux / streambuf :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
struct MyLogStreambuf : public std::stringbuf
{
    virtual int sync()
    {
        instanceLogger.debug(str());
        str("");
        return std::stringbuf::sync();
    }
};
 
MyLogStreambuf buf;
std::ostream Log(&buf);
 
Log << "la valeur de d est " << d << std::endl;

[Patriarch24]

Un logger qui ne permet pas de faire des sorties façon flux C++, ça me paraît déjà pas terrible. Tu es sûr que Poco ne le propose pas ?

Non, Poco ne le propose pas. Son design est assez proche bien que différent de log4cxx ou log4cpp (donc de log4j). Concernant les sorties C++, je ne connaît pas de logger qui permettait de faire de telles sorties (si tu en connais, je suis preneur ).

[JolyLoic]

Concernant les sorties C++, je ne connaît pas de logger qui permettait de faire de telles sorties (si tu en connais, je suis preneur ).

boost::log le permet, mais il utilise aussi une macro, afin en particulier d'éliminer tout calcul si les logs sont statiquement désactivés.

[Laurent Gomila]

Mais on dirait que sync() n'est pas appelé à l'exécution de "Log << "..."" , du coup ça n'affiche rien... sync() c'est bien une méthode virtuelle de la classe de base ??? Je crois que le polymorphisme de cette méthode ne marche pas...

sync() est appelé à chaque fois que le buffer du flux doit être vidé vers sa destination. En clair, si tu veux forcer la sortie mets un std::endl à la fin, ou un std::flush si tu ne veux pas passer de ligne.

[zais_ethael]

Pendant mon temps libre, j'essaie de concevoir une petite biblio de log. Le mieux auquel je sois arrivé concernant les flux est un truc du style:

Code :

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

logger.trace().stream() << "salut " << "le monde";

Théoriquement on devrait pouvoir le raccourcir en

Code :

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

logger.trace() << "salut " << "le monde";

mais j'ai eu des bugs et j'ai laissé tomber pour un temps.
C'est un problème plus compliqué qu'il n'y parait, il ne suffit pas de dériver d'une classe stream quelconque, qui plus est il faut penser à l'intégrité d'un message de log. La syntaxe suivante est purement illogique:

Code :

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

logger << "salut " << "le monde";

Du point de vue du compilo ça correspond à deux appels de fonction, donc deux messages alors que ça n'en forme qu'un seul.

[tnarol]

Un logger qui ne permet pas de faire des sorties façon flux C++, ça me paraît déjà pas terrible. Tu es sûr que Poco ne le propose pas ?

Sinon au pire tu peux te fabriquer ton propre flux / streambuf :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
struct MyLogStreambuf : public std::stringbuf
{
    virtual int sync()
    {
        instanceLogger.debug(str());
        str("");
        return std::stringbuf::sync();
    }
};
 
MyLogStreambuf buf;
std::ostream Log(&buf);
 
Log << "la valeur de d est " << d << std::endl;

J'ai essayé en créant une classe comme ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class LogStreamBuf : public std::stringbuf
{
public:
	LogStreamBuf(Logger* logger = NULL);
	virtual ~LogStreamBuf();
	void setLogger(Logger* logger);
        virtual int sync();
protected:
	Logger* _logger;
};

Mais on dirait que sync() n'est pas appelé à l'exécution de "Log << "..."" , du coup ça n'affiche rien... sync() c'est bien une méthode virtuelle de la classe de base ??? Je crois que le polymorphisme de cette méthode ne marche pas...