C++

[c++] class singleton

Recherche :

Mot : Pseudo : Filtrer
Page : 1 2 Page Suivante Page Précédente Bas de page
Auteur	Sujet : [c++] class singleton

xterminhate

Si vis pacem, para bellum.

Je decouvre qu'il existait une classe de base pour faire un singleton, je croyais que c'etait pas possible... à cause en particulier du static instance.

Code :

template< typename X >
struct singleton
{
inline static X* instancier();
protected:
singleton() {};
virtual ~singleton() {};
singleton( const singleton& ) {};
singleton& operator=( const singleton& ) {};
private:
static X* instance;
};
template< typename X >
X* singleton<X>::instance( 0 );
template< typename X >
X* singleton<X>::instancier()
{
if( instance == 0 )
instance = new X;
return instance;
}
// template de classe derivée
class X : public singleton<X> ...

La classe est semble t il incomplete, mais ca donne une idée.

Message édité par xterminhate le 04-08-2004 à 12:16:23

---------------
Cordialement, Xterm-in'Hate...

Publicité

blackgoddess

vive le troll !

Bien :jap:

Aucune lib n'en fournit une ?

---------------
-( BlackGoddess )-

Taz

bisounours-codeur

si. merde j'ai oublié de répondre hier soir. utilise un truc genre smart pointer pour éviter tout fuite.

blackgoddess

vive le troll !

boost::scoped_ptr serait approprié ?

---------------
-( BlackGoddess )-

Taz

bisounours-codeur

non. un simple std::auto_ptr suffit je pense

xterminhate

Si vis pacem, para bellum.

Ca permettrait de ne pas appeler explicitement un methode de destruction, c'est bien ca ?

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

oui. fais le test, tu verras

xterminhate

Si vis pacem, para bellum.

En ré-ecrivant le membre "static auto_ptr<X> instance", est ce vraiment compatible de la foncion membre instancier() ( copie de autoptr à la ligne return instance ? ).

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

heink ?

xterminhate

Si vis pacem, para bellum.

Au temps pour moi, j'ai forcé sur le rouge à midi! :pt1cable:

Code :

template< typename X >
struct singleton
{
inline static X* instancier();
protected:
singleton() {};
virtual ~singleton() {};
singleton( const singleton& ) {};
singleton& operator=( const singleton& ) {};
private:
static std::auto_ptr<X> instance;
};
template< typename X >
std::auto_ptr<X> singleton<X>::instance( 0 );
template< typename X >
X* singleton<X>::instancier()
{
if( instance.get() == 0 )
instance.reset( new X );
return instance.get();
}

Je suis pas trop sur, je maitrise pas l'auto_ptr.

Message édité par xterminhate le 04-08-2004 à 12:41:08

---------------
Cordialement, Xterm-in'Hate...

Publicité

Taz

bisounours-codeur

vous avez vraiment le chic pour faire compliquer. ni membre statique ni rien, juste un static dans une fonction et directement initialisé.|

blackgoddess

vive le troll !

Code :

#include <memory>
template< typename X >
struct singleton
{
inline static X& get()
{
static std::auto_ptr<X> obj;
if(!obj.get())
obj.reset(new X);
return *obj;
}
};
class X : public singleton<X> ...

comme ca alors ?

---------------
-( BlackGoddess )-

Taz

bisounours-codeur

pourquoi tu veux pas initialiser ton obj ?

mynab

non comme ca:

Code :

template<typename X> class singleton
{
public:
static X& get()
{
static X obj;
return obj;
}
protected:
singleton() {}
};
class X : public singleton<X>
{
protected:
X() {}
};

Tu n'as pas besoin de mettre inline a partir du moment ou la methode est implemetee dans la definition de la classe...

Message édité par mynab le 04-08-2004 à 15:00:02

SoWhatIn22

note que je ne sais pas dans quelle mesure c'est vrai, mais le fait d'utiliser une instance statique et no pas un pointeur peut poser de sérieux problèmes:

1. tu dois avoir une initialisation indépendante de toutes les autres données de ton système puisque tu ne peux pas présupposer de l'ordre de l'initialisation des varaiables statiques et des varaiables globales

2. dans le cas où ce code se retrouve dans une bibliothèque dynamique contenant du code C++, les objets statiques sont instanciés sans que le constructeur soit appelé si le programme appelant a été écrit/compilé en C (pb bien connu par ceux qui écrivent des extensions pour Python en C++). Dans ce cas, il vaut mieux passer par un pointeur initialisé à 0 et créer l'instance dynamiquement au premier appel.

blackgoddess

vive le troll !

bin dans un cas l'objet sera créé au lancement du programme, dans l'autre cas à la 1ere demande de l'objet ... je ne sais pas si c'est important ...

---------------
-( BlackGoddess )-

Taz

bisounours-codeur

dans tous les cas, j'aime pas trop les singleton en C/C++, je préfère les implémentations dans les langages qui permettent les weakref, genre python, je me sens plus tranquille quand on me réclame un truc, je file une weakref et voilà

Taz

bisounours-codeur

d'ailleurs j'aime bien aussi le borg-shared-state pattern

SoWhatIn22

google(borg-shared-state pattern) => 'did not match any documents'. c'est quoi donc ce pattern?

SoWhatIn22

BlackGoddess a écrit :

bin dans un cas l'objet sera créé au lancement du programme, dans l'autre cas à la 1ere demande de l'objet ... je ne sais pas si c'est important ...

ça peut l'être si la construction de cet objet nécessite la collaboration d'autres objets, ou dans le cas (2) que j'ai indiqué; par contre pour l'utilisateur de cette méthode peut importe.
Mais quand on choisit une implémentation plutôt qu'une autre, c'est bien de savoir pourquoi.

Taz

bisounours-codeur

http://aspn.activestate.com/ASPN/C [...] cipe/66531

xterminhate

Si vis pacem, para bellum.

mynab a écrit :

non comme ca:

Code :

template<typename X> class singleton
{
public:
static X& get()
{
static X obj;
return obj;
}
protected:
singleton() {}
};
class X : public singleton<X>
{
protected:
X() {}
};

Tu n'as pas besoin de mettre inline a partir du moment ou la methode est implemetee dans la definition de la classe...

Quelle simplification ! Par contre, est il tjs possible de masquer le constructeur de la classe dérivée (protected) en utilisant ce template de singleton ?

---------------
Cordialement, Xterm-in'Hate...

mynab

Non tu es oblige de remettre le constructeur dans la classe derivee... Sinon quelqu'un pourra construire un objet de type X. Mais tu n'as pas besoin de deriver la classe si tu n'as rien a y ajouter. Tu peux direcement appeler: singleton<int>::get() par exemple. Ou pour faire plus simple tu peux faire un typedef sur singleton<int>.

Message édité par mynab le 05-08-2004 à 12:08:52

blackgoddess

vive le troll !

il y a un problème par contre si 2 classes utilisent singleton<int> indépendemment, en effet elles travailleraient sur la meme variable...

---------------
-( BlackGoddess )-

Taz

bisounours-codeur

faites une factory

blackgoddess

vive le troll !

tu saurais ou trouver de la doc expliquant ce genre de design ?

---------------
-( BlackGoddess )-

mynab

BlackGoddess a écrit :

il y a un problème par contre si 2 classes utilisent singleton<int> indépendemment, en effet elles travailleraient sur la meme variable...

ben c'est pas le but d'un singleton?

Taz

bisounours-codeur

cherche design pattern sur le forum et google
cherche factory singleton ensuite

Taz

bisounours-codeur

je crois que y a un exemple dans TIC++ de B Eckel

xterminhate

Si vis pacem, para bellum.

Ce que je voulais dire...

Code :

template<typename X> class singleton
{
public:
static X& get()
{
static X obj;
return obj;
}
protected:
singleton() {}
};
class X : public singleton<X>
{
protected:
X() {}
};
int main()
{
X& x( X::get() );
}

ne compile pas :

In static member function `static X& singleton<X>::get() [with X = X]' `X::X()' is protected within this context... blabla.

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

colle un friend

xterminhate

Si vis pacem, para bellum.

Ok. Mais, ajouter friend singleton<X> dans la déclaration de X est un peu contraignant.

---------------
Cordialement, Xterm-in'Hate...

schnapsmann

Zaford Beeblefect

xterminhate a écrit :

Ok. Mais, ajouter friend singleton<X> dans la déclaration de X est un peu contraignant.

non c'est le contraire

Code :

template<typename X> class singleton
{
friend class X;
public:
static X& get()
{
static X obj;
return obj;
}
protected:
singleton() {}
};

Message édité par schnapsmann le 05-08-2004 à 21:05:00

---------------
From now on, you will speak only when spoken to, and the first and last words out of your filthy sewers will be "Sir!"

Taz

bisounours-codeur

ben ça sert à rien ça, puisque X a déjà accès

friend class singleton<X> est ok

à toi de savoir ce que tu fais

xterminhate

Si vis pacem, para bellum.

schnapsmann a écrit :

non c'est le contraire

Code :

template<typename X> class singleton
{
friend class X;
public:
static X& get()
{
static X obj;
return obj;
}
protected:
singleton() {}
};

Etonnant, tu as quel compilateur ?

---------------
Cordialement, Xterm-in'Hate...

schnapsmann

Zaford Beeblefect

merde je voyais private et non protected là

---------------
From now on, you will speak only when spoken to, and the first and last words out of your filthy sewers will be "Sir!"

mynab

xterminhate a écrit :

Ce que je voulais dire...

Code :

...
int main()
{
X& x( X::get() );
}

ne compile pas :

In static member function `static X& singleton<X>::get() [with X = X]' `X::X()' is protected within this context... blabla.

Ben c'est normal... je ne comprends pas trop ce que tu veux faire avec l'appel que tu fais dans le main...

X derive de singleton<> et est donc un singleton. Tu essaies donc de creer une deuxieme instance x de ton singleton. C'est normal que ca ne compile pas puisque en faisant un singleton c'est justement de que tu demandes au compilo: t'interdire de creer plusieurs instances de ton singleton!

Si tu veux creer un singleton sur la classe X alors il faut faire:

Code :

class X
{
friend class singleton<X>;
protected:
X() { // ce que tu veux }
};

et apres tu accedes a singleton<X> et voila!

Message édité par mynab le 05-08-2004 à 22:36:38

xterminhate

Si vis pacem, para bellum.

mynab a écrit :

Ben c'est normal... je ne comprends pas trop ce que tu veux faire avec l'appel que tu fais dans le main...

Dans le main, je souhaite acceder à l'instance unique de mon singleton de X. C'est un cas d'utilisation parmi tant d'autres. Le but etait bien de faire un modèle de singleton utilisable.

A mon sens, le rôle de singleton<X>::get() est de retourner une référence unique. Donc l'appel dans le main, ne multiplie pas les instances de mon singleton de X! Corriges moi si je fais une erreur.

D'ailleurs, appeler x::get ou singleton<x>::get revient au même.

Code :

int main()
{
{
X& y1( X::get() );
}
X& y2( X::get() );
X& y3( X::get() );
X& y4( X::get() );
X& y5( X::get() );
X& x1( singleton<X>::get() );
X& x2( singleton<X>::get() );
{
X& x3( singleton<X>::get() );
}
X& x4( singleton<X>::get() );
// &y1 = &y2 = &y3 = ... = &x1 = & x2 = ....
}

mynab a écrit :

X derive de singleton<> et est donc un singleton. Tu essaies donc de creer une deuxieme instance x de ton singleton. C'est normal que ca ne compile pas puisque en faisant un singleton c'est justement de que tu demandes au compilo: t'interdire de creer plusieurs instances de ton singleton!

Non. Je ne vois pas ce qui est normal. Comment arriverais tu à utiliser le modèle que tu m as donné sans ajouter le friend ? [:airforceone]

Merci.

Message édité par xterminhate le 06-08-2004 à 07:30:11

---------------
Cordialement, Xterm-in'Hate...

tanguy

Voila mon singleton, dites moi ce que vous en pensez:

Code :

#ifndef SINGLETON_H
#define SINGLETON_H
#include <memory>
template<typename T>
class Singleton {
public:
static T & getInstance() {
static std::auto_ptr<T> instance;
if (!instance.get()) {
instance.reset(new T);
}
return *instance.get();
/* Sans auto_ptr donc fuite memoire.
static T * instance;
if (!instance) {
instance = new T;
}
return *instance;
*/
}
protected:
Singleton() { }
virtual ~Singleton() { }
Singleton(const Singleton & ) { }
Singleton & operator=(const Singleton & ) { }
};
#endif //SINGLETON_H

Et un exemple d'utilisation de la classe:

Code :

#include "Singleton.h"
#include <iostream>
using namespace std;
class Toto : public Singleton<Toto> {
friend class Singleton<Toto>;
public:
void sayHello() const {
cout << "Hello World!" << endl;
}
protected:
Toto() {
_counter += 1;
cout << "Initialization: " << _counter << endl;
}
private:
static int _counter;
};
int Toto::_counter = 0;
int main() {
Toto & toto1 = Toto::getInstance();
toto1.sayHello();
Toto & toto2 = Toto::getInstance();
toto2.sayHello();
}

J'ai compile avec la ligne suivante:
g++ -g -ansi -Wall -pedantic main.cpp

Et valgrind me dit:

valgrind --tool=memcheck --leak-check=yes --show-reachable=yes ./a.out
==7364== Memcheck, a memory error detector for x86-linux.
==7364== Copyright (C) 2002-2004, and GNU GPL'd, by Julian Seward et al.
==7364== Using valgrind-2.3.0.CVS, a program supervision framework for x86-linux.
==7364== Copyright (C) 2000-2004, and GNU GPL'd, by Julian Seward et al.
==7364== For more details, rerun with: -v
==7364==
Initialization: 1
Hello World!
Hello World!
==7364==
==7364== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 17 from 1)
==7364== malloc/free: in use at exit: 0 bytes in 0 blocks.
==7364== malloc/free: 1 allocs, 1 frees, 4 bytes allocated.
==7364== For counts of detected errors, rerun with: -v
==7364== No malloc'd blocks -- no leaks are possible.

Et si je n'utilise pas auto_ptr (le code en commentaire), valgrind me dit:

valgrind --tool=memcheck --leak-check=yes --show-reachable=yes ./a.out
==7371== Memcheck, a memory error detector for x86-linux.
==7371== Copyright (C) 2002-2004, and GNU GPL'd, by Julian Seward et al.
==7371== Using valgrind-2.3.0.CVS, a program supervision framework for x86-linux.
==7371== Copyright (C) 2000-2004, and GNU GPL'd, by Julian Seward et al.
==7371== For more details, rerun with: -v
==7371==
Initialization: 1
Hello World!
Hello World!
==7371==
==7371== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 17 from 1)
==7371== malloc/free: in use at exit: 4 bytes in 1 blocks.
==7371== malloc/free: 1 allocs, 0 frees, 4 bytes allocated.
==7371== For counts of detected errors, rerun with: -v
==7371== searching for pointers to 1 not-freed blocks.
==7371== checked 2227148 bytes.
==7371==
==7371== 4 bytes in 1 blocks are still reachable in loss record 1 of 1
==7371== at 0x1B904730: operator new(unsigned) (vg_replace_malloc.c:132)
==7371== by 0x80489D6: Singleton<Toto>::getInstance() (Singleton.h:24)
==7371== by 0x80488F0: main (main.cpp:29)
==7371==
==7371== LEAK SUMMARY:
==7371== definitely lost: 0 bytes in 0 blocks.
==7371== possibly lost: 0 bytes in 0 blocks.
==7371== still reachable: 4 bytes in 1 blocks.
==7371== suppressed: 0 bytes in 0 blocks.

Message édité par tanguy le 14-11-2004 à 18:33:19

xterminhate

Si vis pacem, para bellum.

Ce modèle ci ne convenait pas ?

Code :

template< typename X >
struct singleton
{
static X& instancier()
{
static X obj;
return obj;
}
virtual ~singleton() {}
protected:
singleton() {}
singleton( const singleton& ) {}
singleton& operator=( const singleton& ) {}
};

---------------
Cordialement, Xterm-in'Hate...

Publicité

Page : 1 2

Page Suivante

Page Précédente

Haut de page

FORUM HardWare.fr

Programmation

C++

[c++] class singleton

Sujets relatifs
[css] class, id et heritage	Récupérer via Javascript une propriété d'une class CSS non utilisé
[JAVA] Inner class et accès aux membres protected --> compiler bug ?	Propriétés 'class' et 'div'
Probleme de class avec GD	Comen séparer l'interface graphik et les méthod en diférentes class
[Applet] load : class not found	Infos sur les fichiers .class
[Java] Retrouver le code java a partir des .class ?	Eclipse et Export Jar : Failed to load Main-Class ....
Plus de sujets relatifs à : [c++] class singleton

Page générée en 0.145 secondes