Discussion :

[pyrrhon_]

Bonjour à tous,

voilà, je sais bien que c'est déjà fais, mais j'ai voulu faire un logger pour plusieurs raisons: pour apprendre certaines bases, parce que ce logger est destiné à une application qui, pour différentes raisons, doit utiliser un minimum de libs externes, etc.

Bref, je vous présentes ici mon header trés épuré (plus de pare-feu de compil, etc.), en espérant que vous aurez un peu de temps pour me dire ce que vous en pensez:

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
 
enum LOG_LEVEL
{
	LOG_DEBUG,	
	LOG_WARNING,
	LOG_ERROR,
	LOG_CRITICAL
};
 
class Logger
{
public:
	Logger();
	~Logger();
	static int	init(const LOG_LEVEL level, const std::string logFilePath, const bool consoleOut);
	void		log(const std::string log, const LOG_LEVEL level = LOG_DEBUG);
	void		log(const std::string log, const bool consoleOut, const LOG_LEVEL level = LOG_DEBUG);
	static LOG_LEVEL		logLevel;
	static std::string	logFile;
	static bool		consoleOutput;
	static bool		isInitialized;
 
private:
	static bool	writeLine(std::string str);
	std::string	logLevel2String(LOG_LEVEL level);
};
 
void LOG(const std::string log, const LOG_LEVEL level = LOG_DEBUG);
void LOG(const std::string log, const bool consoleOut, const LOG_LEVEL level = LOG_DEBUG);

Le principe consiste à initialiser une fois de logger puis n'utiliser que la fonction LOG(...).

Et voici un exemple d'utilisation:

Code :

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

1
2
3
Logger::init(logLevel, logFile, true);
LOG("test log"); //log simple
LOG("test log avec console output", true); //log avec outout sur la console

[JolyLoic]

Sans même regarder les possibiités de configuration du log, je n'aime pas du tout l'interface que cet objet va imposer à chaque fois qu'on veut logger quelquechose. Elle m'oblige à ne logger que des strings, alors que j'aimerais pouvoir logguer des strings, des nombres, mes propres classes, des combinaison...

Un truc genre :

Code :

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

LOG() << "Test " << 42 << monObjet;

Ou

Code :

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

LOG( "Test " << 42 << monObjet);

Ou autre écriture du genre.

[ZaaN]

JolyLoic a raison et pour rendre ton logger plus confortable, voici un prototype de fonction à nombre d 'argument variable pour logger tes info :

Code :

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

void AddToLogFile(LPCTSTR szFmt,...);

là. tu passes tes elements comme dans un printf standard

PS : rajoute y tes paramètres, comme ton loglevel...

[Laurent Gomila]

Code :

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

void AddToLogFile(LPCTSTR szFmt,...);

Pratique pour faire un truc moche et pas extensible

En C++ on a fait beaucoup mieux avec le chaînage d'appels et la surcharge, comme l'a montré Loïc. D'ailleurs un bon point de départ pour un logger peut être un flux standard (ostream), qu'on peut dériver. On peut aussi combiner ça à std::streambuf (qu'on peut dériver également). Il doit y avoir de bons articles sur le sujet.

[ZaaN]

merci, merci,

D'ailleurs un bon point de départ pour un logger peut être un flux standard (ostream), qu'on peut dériver.

En gros il faut redefinir les operateurs << pour chaque type d'objet ?

[pyrrhon_]

Merci pour vos remarques

J'avais hésité à utiliser les opérateurs de flux, mais étant donné que je ne connais pas grand chose sur le sujet, j'avais abandonné. Et puis je n'en voyais pas trop l'intérêt. Maintenant je le vois trés clairement.
Je vais le faire, ça me fera un bon excercice

[Laurent Gomila]

En gros il faut redefinir les operateurs << pour chaque type d'objet ?

Justement non, l'intérêt de dériver de std::ostream est entre autres d'avoir ces surcharges déjà disponibles.
Par contre si tu ne dérive pas de std::ostream, tu vas devoir toutes te les taper.