Discussion :

[b Oo]

Bonjour à tous,
j'ai :

Code :

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

1
2
3
4
5
6
7
8
 
class Garage
{
     // des trucs
   public:
     Voiture * creeVoiture();
     // des trucs
};

Code :

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

1
2
3
4
5
6
7
 
class Voiture
{
     // des trucs
   public:
     // des trucs
};

Code :

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

1
2
3
4
5
6
7
 
class Ferrari : public Voiture
{
     // des trucs
   public:
     // des trucs
};

Je voudrais pouvoir rajouter des voitures dans mon garage juste en créeant une classe qui hérite de Voiture (comme Ferrari).
Comment puis-je faire sachant que je veux juste que mon garage contienne toutes les voitures différentes (ie l'ensemble des voitures que je peux créer) et que garage n'est pas obligé de construire toutes les voitures ?
Par exemple je peux avoir Ferrari et Porsche qui peuvent être construites par mon garage, mais garage ne contiendra qu'une Ferrari par exemple au début.
Si je fais un conteneur de voitures * en static dans garage et une méthode static dans Voiture (que je dois redéfinir dans les classes filles), cela est-il correct ? Il y a mieux ?
J'aimerais aussi savoir si cela se rapproche d'un design pattern, si oui lequel.

Merci d'avance.

[JolyLoic]

Je ne comprend pas trop où tu veux en venir. Déjà, j'ai du mal à voir pourquoi Ferrari serait une classe dérivée de voiture. Qu'est-ce qu'une Ferrari a qui fait que ce n'est pas simplement une voiture ?

A part ça, le design pattern qui me vient en tête quand on parle de création d'objet est avant tout le pattern Factory. Mais n'ayant pas compris ton besoin, je ne sais s'il s'applique ici.

[Matthieu Brucher]

Ce n'est pas ton garage qui construiot des voitures, tu devrais bien séparer le garage - stockage - de la construction - pattern Builder ou Factory -

[b Oo]

Je suis d'accord, j'ai besoin d'une classe qui construit mes voitures.
Pour le garage, je dois suivre une interface qui comprend créerVoiture().
Donc si je suis obligé d'avoir creerVoiture dans ma classe garage, il serait mieux que je crée une classe Factory qui se charge d'implémenter la création d'une voiture et creerVoiture() de ma classe garage devra appeler la méthode de constrution de ma Factory.

Ce que je voudrais c'est que mon garage contiennent différentes voitures et que la méthode de ma Factory construise aléatoirement des voitures.
Pour rajouter une voiture, je voudrais simplement ajouter quelques classes.
Par exemple pour Ferrari, admettons que j'ai créer une Enzo, pour peugeot une 206.
Ma factory devra contruire par exemple 10 voitures : 7 Enzo, et 3 206 (les chiffres seront tirés aléatoirement).
Maintenant je veux rajouter une Porsche 911, je crée une classe et là je voudrais que ma classe Factory puisse la créer sans changer la méthode de création.
Si c'est toujours pas clair dites le moi.

Déjà, j'ai du mal à voir pourquoi Ferrari serait une classe dérivée de voiture. Qu'est-ce qu'une Ferrari a qui fait que ce n'est pas simplement une voiture ?

Tu ferais une instance de voiture que tu appelerais Ferrari, c'est bien ça ?
Je vois, en fait j'avais dans l'idée de faire dérivée chaque nouvelle voiture de la classe voiture. Mais apparemment ça n'a pas l'air très bien, si ?


Merci d'avance.

[JolyLoic]

Tu ferais une instance de voiture que tu appelerais Ferrari, c'est bien ça ?
Je vois, en fait j'avais dans l'idée de faire dérivée chaque nouvelle voiture de la classe voiture. Mais apparemment ça n'a pas l'air très bien, si ?

Ca dépend ce que tu veux en faire... Mais dans ce que j'ai vu actuellement de ton code, rien ne semble le justifier. J'ai bien aimé l'article suivant sur le sujet :
http://archive.eiffel.com/doc/manual...T/inh-m6fs.pdf
(paragraphe 24.4).

[b Oo]

Merci pour l'article JolyLoic.

Sinon pour problème.
Il faudrait que je fasse un membre static qui contient des Voitures * dans ma fabrique et que chaque nouvelle classe (qui hérite de Voiture) soit rajoutée dans la fabrique.

Code :

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

1
2
3
4
5
6
7
 
class Fabrique
{
      static vector<Voiture *> v;
   public :
      Voiture * creeVoiture(); // cree une voiture aleatoirement
};

Code :

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

1
2
3
4
5
6
 
class Fille : public Voiture
{
   public:
    static ajouterVoiture(); // la voiture Fille peut etre cosntruite par ma fabrique
};

Si je procede de cette maniere, cela est-il correct ?
Je rappelle que je ne veux pas modifier le code de ma fabrique.

Merci.

[loufoque]

Commence par de ne pas retourner des pointeurs bruts comme ça.

[JolyLoic]

Tu as un vecteur d'objets (d'ailleurs, pourquoi statique ? Tu ne peux pas avoir des garages différents ? Si tel est le cas, ne serait-ce pas à garage d'être un singleton ?) dans ta classe, et j'ai l'impression que ce que tu voudrais, conceptuellement, c'est une vecteur de classes, ce qui n'existe pas directement en C++.

Les deux solutions qui me viennent en tête sont :
- Avoir un vecteur de pointeur de fonctions qui pointent sur des fonctions de création d'objets (c'est assez classique pour implémenter une factory)
- Avoir un vecteur d'objets qui servent de prototype (comme le design pattern du même nom) et qui se clonent à le demande pour créer un nouvel objet (j'ai moins l'habitude, mais certains langages objets n'ayant pas la notion de classe fonctionnent uniquement comme ça).

Dans le premier cas, je verrais bien pour ton garage une interface genre :

Code :

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

1
2
3
4
5
6
7
8
9
 
class Garage
{
  typedef Vehicle *(*VehicleCreator)();
  void registerVehicle(string const &name, VehicleCreator creator);
  Vehicle *create(string const &name);
private:
  map<string, VehicleCreator> myCreators;
};

Charge aux véhicules de s'enregistrer auprès du garage. Evidemment, pour du vrai code, j'utiliserais probablement des pointeurs intelligents et des boost::function<Vehicle*()>, mais l'idée est là.

[b Oo]

Ok, merci JolyLoic, je vais essayer la deuxième solution.
Mon Garage n'est pas un singleton, mais les 2 garages qui heritent de Garage oui. (Garage me sert juste d'interface).

Commence par de ne pas retourner des pointeurs bruts comme ça.

C'est pour eviter d'oublier une désallocation ?
Si tu as le temps, j'aimerais que tu développes un peu plus.

Merci encore.

[Matthieu Brucher]

Oui, c'est pour éviter les fuites mémoires, utilise des pointeurs intelligents, par exemple -> http://miles.developpez.com/tutoriel...ost/smartptrs/