Constructeur par copie et classe abstraite

**Benlev** · 29/10/2009, 19h44

Bonjour,

Je me pose une question sur le sujet suivant :
J'ai une classe abstraite Account, dont héritent trois classes (différents types d'Account).

J'ai une autre classe, Client, qui possède un vecteur de pointeurs sur Account.

Ma question :
Comment puis-je implémenter le constructeur par copie de Client, alors que je ne connais pas le type d'Account à copier ?

Merci d'avance,

Benjamin

**Benlev** · 29/10/2009, 20h31

J'ai oublié de dire que j'avais un solution, mais que j'en cherchais une autre :

Je crée une fonction virtuelle pure dans Account:

Code :

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

virtual Account* copy() const =0;

que j'implémente de la manière suivante dans chacune des classes dérivées de Account :

Code :

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

1
2
3
4
5
 
DerivedAccount* DerivedAccount::copy() const
{
     return new DerivedAccount(*this);
}

**tomlev** · 29/10/2009, 20h40

A priori je dirais qu'il suffit que tu copies les pointeurs, tu n'as pas besoin de connaître leur type concret... par exemple :

Code :

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

1
2
3
4
5
6
7
Client::Client(Client* pModele)
{
    for(int i = 0; i < pModele->nbAccounts; i++)
    {
        this->accounts[i] = pModele->accounts[i];
    }
}

Evidemment, en faisant ça tu ne fais pas une copie des Accounts : les 2 clients (le modèle et la copie) pointent vers le mêmes Accounts. Si tu veux aussi en faire des copies, je dirais qu'il faut ajouter dans Account une méthode virtuelle "Copy" ou "Clone", qui renvoie un pointeur vers une copie du compte.

EDIT: j'avais pas vu ton deuxième post... Avec cette méthode copy, ton constructeur par copie de Client devient :

Code :

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

1
2
3
4
5
6
7
Client::Client(Client* pModele)
{
    for(int i = 0; i < pModele->nbAccounts; i++)
    {
        this->accounts[i] = pModele->accounts[i]->copy();
    }
}

**Luc Hermitte** · 29/10/2009, 21h35

J'interdirai tout simplement la copie.
Ces objets sont typiquement des entités.

**Benlev** · 29/10/2009, 21h39

Envoyé par tomlev

Evidemment, en faisant ça tu ne fais pas une copie des Accounts

Je souhaite faire la copie des Accounts

La deuxième solution que tu proposes est celle que j'avais choisi...
Je me demandais simplement si le C++ permettait une autre solution...

**Luc Hermitte** · 29/10/2009, 21h52

Code C++ :

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

1
2
3
class Account : boost:noncopyable {
   ...
}

**Benlev** · 29/10/2009, 22h05

Envoyé par Luc Hermitte

Code C++ :

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

1
2
3
class Account : boost:noncopyable {
   ...
}

merci mais je ne peux pas utiliser boost

**Mat007** · 29/10/2009, 22h51

Salut,

Tu peux regarde la FAQ et plus précisément comment effectuer la copie d'objets polymorphes et comment rendre une classe non copiable.

MAT.

**Benlev** · 29/10/2009, 23h16

Merci Mat, le premier lien répond parfaitement à ma question : pas d'autre solution qu'une fonction 'clone'.
Désolé de pas avoir vu ça plus tôt.

**Luc Hermitte** · 30/10/2009, 16h42

Mais es-tu vraiment sûr d'avoir besoin de les copier ? Pour 95% des entités, y implémenter la copie s'avère une perte de temps sêche : un truc compliqué qui ne sera jamais utilisé.

**3DArchi** · 30/10/2009, 17h28

Salut,
Profitons-en pour donner quelques lectures :
Quand est-ce qu'une classe a une sémantique de valeur ?
Quand est-ce qu'une classe a une sémantique de d'entité ?

Envoyé par Luc Hermitte

Mais es-tu vraiment sûr d'avoir besoin de les copier ? Pour 95% des entités, y implémenter la copie s'avère une perte de temps sêche : un truc compliqué qui ne sera jamais utilisé.

Si ça s'appelle souvent Clone, c'est bien parce qu'on ne veut pas copier un objet mais en créer un nouveau identique au précédent. Ce n'est pas qu'une question rhétorique.

**Luc Hermitte** · 30/10/2009, 17h30

Dupliquer, copier, cloner, c'est pareil.
Sur les entités, c'est 95% du temps une perte de temps.

**3DArchi** · 30/10/2009, 17h34

Envoyé par Luc Hermitte

Dupliquer, copier, cloner, c'est pareil.

Pas tout à fait. On copie une valeur pour en avoir une identique dans sa sémantique. Quand on clone une entité c'est pour avoir un second objet avec le même état initial.

Envoyé par Luc Hermitte

Sur les entités, c'est 95% du temps une perte de temps.

Oui, car on ne sait pas pourquoi on l'a cloné. On peut très bien par exemple dans un jeu cloner un personnage pour en avoir deux instances qui vont ensuite évoluer séparément.

**Goten** · 30/10/2009, 17h47

Envoyé par Benlev

merci mais je ne peux pas utiliser boost

C'est pas la mort... tu déclares (pas définie) ton copy ctor et ton op= en privée et rulez.

+1 pour éviter la copie quand même...

Et j'ai quand même du mal à voir la frontière entre clonage et copiage...

**Benlev** · 30/10/2009, 17h57

Je n'ai pas vraiment besoin de copier dans ce cas précis...

C'est plus qu'au moment d'implémenter le constructeur par copie, je me suis posé cette question...

Merci pour vos réponses