Discussion :

[r0d]

Bonjour tout le monde,

je viens d'implémenter un petit logger super simple (et trés spécifique à mon appli, c'est pourquoi je ne peux pas utiliser des logger existants). Etant donné que je ne dois en avoir qu'un dans toute mon appli, j'ai estimé qu'il était légitime d'en faire un singleton.

Voici, en gros, à quoi ressemble mon logger:

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
32
33
34
35
36
/// Logger.h
// includes et autres
 
class Logger : public std::ofstream
{
public:
	static void Init(std::string filename)
	{
		if ( m_pInstance == NULL )
		{
			m_pInstance = new Logger(filename);
			// ici, du code relatif à l'initialisation du logger			
		}
	}
 
	static Logger& GetInstance()
	{
		if ( m_pInstance == NULL )
		{
			m_pInstance = new Logger();
		}
		return *m_pInstance;
	}
 
	// destructor (bon, en réalité, le destructeur est privé)
	~CLogger(){close();}
 
private:
	// constructor
	Logger(std::string filename = "c:/temp/log.txt") 
		: std::ofstream( filename.c_str(), std::ios::app  ) 
	{}
 
	// unique Logger instance
	static CLogger* m_pInstance;
};

Je déclare et j'initialise le singleton dans Logger.cpp

Code :

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

1
2
3
4
///Logger.cpp
#include "CLogger.h"
 
CLogger* CLogger::m_pInstance = NULL;

Bon, mon code fonctionne, mais j'ai 2 problèmes:
1/ J'aimerais forcer l'utilisateur de cette classe à initialiser le logger. En gros, j'aurais aimé faire en sorte que si Logger::Init(monfichier); n'est pas appelé, ça génère une erreur à la compilation.
Mais je ne vois vraiment pas comment faire. Est-ce seulement possible?

2/ J'utilise un outil d'analyse de code qui me hurle dessus à la ligne:

Code :

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

CLogger* CLogger::m_pInstance = NULL;

En gros, il me dit: "This non-local object will be initialised at runtime."
Et il me dit d'aller voir l'item #47 du livre "Effective C++, 2nd edition" de Scott Meyers.
Ce que j'ai fais. Et je ne suis pas plus avancé, car j'ai l'impression que mon implémentation est fidèle aux conseils du livre.
Voyez-vous où est mon erreur?


note: afin de simplifier l'utilisation de ce logger, j'ai défini les macros suivantes:

Code :

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

1
2
#define LOG(X) CLogger::GetInstance() << X << std::endl
#define INIT_LOG(X) CLogger::Init(X)

Ce qui me permet de l'utiliser ainsi:

Code :

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

1
2
INIT_LOG("c:/temp/log.txt");
LOG( "test " << 1234 << UnObjet << std::endl );

[yan]

essai

avec des pragma, #define ... tu doit pouvoir faire ce que tu veut (pour vc2005)

J'ai peut etre pas compris ton problème

[nicroman]

Je suis peut etre trop parano....
Mais je vois un "new" et pas de "delete" associé.... en tant qu'utilisateur ca me ferait peur


Pourquoi ne pas faire simplement:

Code :

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

1
2
3
4
5
6
 
    static Logger& Logger::GetInstance()
    {
        static Logger myInstance;
        return myInstance; 
    }

??

Sinon, vérifier à la compilation qu'une fonction est bien appelée... je vois pas comment tu peux faire !

[yan]

pour moi se serait mieux de faire dans logger.cpp

Code :

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

1
2
3
#include "CLogger.h"
 
CLogger* CLogger::m_pInstance = new CLogger();

et dans la fonction Init :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
static void Init(std::string filename)
	{
	if ( m_pInstance != NULL )
		 {
		 delete 	m_pInstance;
                 m_pInstance = NULL;
		  }
       m_pInstance = new Logger(filename);
			// ici, du code relatif à l'initialisation du logger	
	}

[r0d]

Je suis peut etre trop parano....
Mais je vois un "new" et pas de "delete" associé.... en tant qu'utilisateur ca me ferait peur

Je n'ai pas mis tout le code ici. Je fais le delete dans le destructeur

[Laurent Gomila]

Je fais le delete dans le destructeur

Pas le destructeur de CLogger j'espère

Sinon, tu as quoi exactement dans la partie "initialisations" ?

[Mat007]

Salut,

Code :

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

1
2
 
static void Init(std::string filename)

C'est sans doute const std::string& non ?

MAT.

[r0d]

Hum, je m'excuse, j'ai fais quelques erreurs de copier/coller: CLogger et Logger c'est la même classe (c'est juste que j'ai renommé bêtement entre-temps).

Pas le destructeur de CLogger j'espère

Bah si pourquoi?

Code :

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

1
2
3
4
5
6
7
8
~CLogger()
{
	if ( m_pInstance != 0 )
	{
		delete m_pInstance;
	}
	close();
}

Comme ça, dés qu'on sort du scope (ici, c'est l'appli), le logger est automatiquement détruit, et le fichier log fermé. Non?

Sinon, tu as quoi exactement dans la partie "initialisations" ?

Des trucs
J'initialise des variables membre, des histoires de dates et tout.
Mais pourquoi cette question, ça ne me semble pas important?

C'est sans doute const std::string& non ?

Oui bien sûr, c'était pour voir si vous suiviez...
hum...

[Desboys]

Bonjour,

Bah si pourquoi?

Code :

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

1
2
3
4
5
6
7
8
~CLogger()
{
	if ( m_pInstance != 0 )
	{
		delete m_pInstance;
	}
	close();
}

Comme ça, dés qu'on sort du scope (ici, c'est l'appli), le logger est automatiquement détruit, et le fichier log fermé. Non?

En théorie, le destructeur ne sera jamais invoqué sur ton pointeur ( à moins de passer par un smart_ptr<> ).
Pour t'en convaincre, essaies ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <iostream>
#include <cstdlib>
class MaClasse
{
public:
 MaClasse() { std::cout << "Construction de ma classe\n"; }
 ~MaClasse() { std::cout << "Destruction de ma classe\n"; }
}
int main(int argc,char **argv)
{
 MaClasse* pMaClasse = new MaClasse;
 return EXIT_SUCCESS;
}

Ici le scope est la fonction main(), qui ne change rien au fait que ton destructeur ne sera pas appelé, quelque-soit le scope.

Moi j'aurai tendance à lancer une exception si mon instance n'a pas été initialisée ( avec un petit message précisant la condition d'utilisation valable ).

S. Desbois

[Laurent Gomila]

Bah si pourquoi?

Et qui appelle le destructeur de CLogger ? (réponse : personne)
De plus si tu es dans le destructeur, c'est que ton instance est en train d'être détruite, donc faire un delete dessus... c'est plutôt moyen.
L'instance ne sera détruite (et son destructeur appelé) que lorsque tu feras un delete explicite, puisqu'elle a été allouée avec new.

J'initialise des variables membre, des histoires de dates et tout.
Mais pourquoi cette question, ça ne me semble pas important?

Pour rien

[r0d]

Rha oui pardon, je ne suis pas réveillé. Je passe bien par un fonction statique pour détruire mon logger.

Sinon, je suis parvenu à améliorer mon code avec vos conseils, j'ai moins de warnings maintenant, mais il m'en reste quelques uns.

[Mat007]

J'utilise un outil d'analyse de code

C'est un peu hors-sujet mais c'est quoi cet outil, par curiosité ?

MAT.

[r0d]

Ca s'appelle QAC++
Je le découvre, et pour l'instant, je trouve ça plutôt pas mal.