C++

Design d'une classe collection

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Design d'une classe collection

cgo2

Dum spiro spero

Hello

Ma question porte sur le design de deux classes que nous appellerons Truc et TrucCollection.

Ce que je voudrais faire : TrucCollection conserve toujours la liste de toutes les instances des "Truc", sans que "l'utilisateur" n'est de méthode spécifique à appeller.

classe TrucCollection :

Code :

class TrucCollection
{
public:
static void addInstance(Truc *); // ajoute cette instance
static void removeInstance(Truc *); // supprime cette instance
private:
static un_conteneur_STL<Truc *> liste;
}

classe Truc :

Code :

class Truc
{
public:
Truc() { TrucCollection::addInstance(this); }
~Truc() { TrucCollection::removeInstance(this); }
}

Jusque là, ça fonctionne. Maintenant je veux ajouter une méthode "mrproper()" à la classe TrucCollection qui supprime toutes les instances (en fin de programme par exemple). Le problème est le suivant :

Code :

void mrproper()
{
for (...) // parcours de la liste
delete element; // appel le destructeur, qui va le retirer de la liste qu'on est en train de parcourir !
}

Retirer l'element d'une liste alors qu'on est en train de la parcourir, est-ce bien moral ? Est-ce mon design qui n'est pas correct ? Y a-t-il d'autres solutions ?

Merci d'avance pour vos conseils !

Publicité

benji_100

Je ne suis pas un pro mais je penses pas que ca soit moche ce que tu fais.

Tu ne "retire" pas d'éléments dans ta liste ! Elle conserve ses éléments qui sont des pointeurs. Tu vide juste les objets Trucs qui sont à leurs addresses. A la limite tu peux mettre les pointeurs à NULL apres le delete pour plus de cohérence mais si tu fais ca en fin de programme, alors cela n'a pas d'importance.

haazheel

J'ai peur de me tromper, mais est-ce que le pattern Factory ne sert pas à ça justement?

cgo2

Dum spiro spero

Ce qui me gène aussi c'est au niveau de la logique du truc :
- dans mrproper() je suis dans une boucle, j'ai l'itérateur sur l'objet à supprimer
- j'appelle le destructeur de l'objet
- le destructeur appelle removeInstance() qui _reparcours_ la liste pour retrouver l'objet à supprimer !

haazheel a écrit :

J'ai peur de me tromper, mais est-ce que le pattern Factory ne sert pas à ça justement?

Tu veux dire par exemple des fonctions Truc::create() et Truc::destroy() pour l'utilisateur (et ducoup mettre les constructeurs/destructeurs normaux en protected/private) ?

Taz

bisounours-codeur

tu connais les opérateurs new / delete

cgo2

Dum spiro spero

Taz a écrit :

tu connais les opérateurs new / delete

bah oui, mais je ne comprends pas le rapport avec ma question :heink:

skelter

moi je comprends pas a quoi ca peu servir ?

cgo2

Dum spiro spero

skelter a écrit :

moi je comprends pas a quoi ca peu servir ?

De quoi tu parles ?

skelter

de TrucCollection

cgo2

Dum spiro spero

Ah ok, ben en fait je voudrais conserver un pointeur vers toutes les instances de ma classe Truc, d'où qu'elles soient créées dans le programme, et sans que "l'utilisateur" ai besoin de les "enregistrer" manuellement. Par exemple il fait un Truc * toto = new Truc() et hop cette instance est automatiquement enregistrée dans TrucCollection.

Publicité

Taz

bisounours-codeur

et tu vois toujours pas le rapport là ?

cela dit, pourquoi tu ne fous pas un std:queue<> en static dans ta classe ?

cgo2

Dum spiro spero

Taz a écrit :

et tu vois toujours pas le rapport là ?

euh non désolé.... mais ma question ne porte pas sur l'utilisation de new/delete (ça, ça fonctionne, regarde mon tout premier post), alors je ne vois pas trop où tu veux en venir...

Taz a écrit :

cela dit, pourquoi tu ne fous pas un std:queue<> en static dans ta classe ?

Hum ça apporterait quoi par rapport à ce que j'ai déjà mis dans la classe TrucCollection ? :heink:

Taz

bisounours-codeur

cgo2 a écrit :

euh non désolé.... mais ma question ne porte pas sur l'utilisation de new/delete (ça, ça fonctionne, regarde mon tout premier post), alors je ne vois pas trop où tu veux en venir...

que t'as le droit de surcharger ces opérateurs.

Taz

bisounours-codeur

/me pas comprendre pourquoi toi pas faire une factory (et singleton la factory si tu y tiens)

cgo2

Dum spiro spero

ben parcequ'au départ j'étais parti sur l'idée de pouvoir écrire "new Truc" et pas TrucFactory::create()...

je suis en train de regarder si la surchage de l'opérateur delete peut m'aider.

skelter

ce que suggere taz c'est que au lieu d'avoir une classe TrucCollection tu met dans Truc la collection de pointeurs en static et tu surcharge new et delete de Truc pour qu'ils fassent eux meme l'ajout et la suppression de this dans la collection, c'est nettement mieux

cgo2

Dum spiro spero

oui c'est ce que j'avais enfin fini par comprendre (expliqué comme ça dès le départ ça aurait été beaucoup plus rapide ;) ducoup je peux résoudre mon problème initial en utilisant ::operator delete qui supprimera l'objet sans s'occuper de le retirer de la liste. merci beaucoup pour cette idée !

mais je me demande quel est l'avantage de mettre la liste en static dans la classe Truc ? par exemple si je décide d'implémenter des fonctions de traitements (afficher l'état de toutes les classes Trucs, faire une action, etc.) n'est-ce pas mieux de séparer :
- d'un côté les méthodes qui traitent une seule classe (dans Truc)
- et les méthodes qui traitent toutes les classes (dans TrucCollection) ?

Message édité par cgo2 le 15-06-2005 à 21:54:27

haazheel

Je crois bien que si, et c'est là justement qu'une factory entre en jeu il me semble...

http://www.dofactory.com/Patterns/ [...] spx#_self2

cgo2

Dum spiro spero

Ok ok, merci pour ces conseils, j'ai maintenant deux pistes pour résoudre mon problème :
- soit surcharger new et delete au lieu de mettre le code dans les constructeurs/destructeur
- soit faire une factory, à priori plus adaptée, mais qui empecherait après d'utiliser la syntaxe new/delete
:jap:

FORUM HardWare.fr

Programmation

C++

Design d'une classe collection

Sujets relatifs
[C++] Lecture d'une variable static d'une autre classe??	[C#] Création de packageensemble de fonction sans utiliser une classe?
Changer dynamiquement des propriétés d'une classe CSS	classe + lien vers une DB prob incompréhenssible (resolu)
Accéder à request dans une classe	Intérêt d'une classe mysql
AS2.0 : portée dans gestionnaire écrit dans une classe	Utilisation d'un objet dans une classe ?
Hériter d'une classe dans un même package	Besoin d'aide sur méthode contains de la class Collection
Plus de sujets relatifs à : Design d'une classe collection

Page générée en 0.049 secondes