[debutant]probleme avec le copy constructor

Version imprimable

06/11/2005, 17h50
Battosaiii

[debutant]probleme avec le copy constructor
Citation:

Error E2312 plane.cpp 20: 'FlyingMachine::name' is not an unambiguous base class
of 'plane' in function plane::plane(const plane &)
Warning W8057 plane.cpp 20: Parameter 'sourceplane' is never used in function pl
ane::plane(const plane &)
*** 1 errors in Compile ***

J'obtiens cette erreur. Mon copy construcotr ets de la formew :
Code:

1 2 3 plane::plane(const plane &sourceplane): landingGearOn(false), name(sourceplane.name){}
name n'appartient pas a la classe plane mais a sa classe mere. On peut appeller des state de la classe mere lorsque on definit une classe avec le copy constructor ? Comment regler ce probleme ?

Code type pour la copie de classes issues d'une hiérarchie polymorphique.

Code:

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
struct Mere
{
    // destructeur virtuel
    virtual ~Mere() {}
 
    // accesseur au seul attribut
    std::string const& nom() const { return nom_; }
 
    // La seule façon d'obtenir une copie polymorphique
    virtual Mere * clone() const = 0;
 
protected:
    // Aucun intérêt à être public, la classe n'est pas instanciable
    // Mais c'est requis par les descendants => protégé
    Mere(std::string const& un_nom) : nom_(un_nom) {}
 
    // Constructeur de recopie. Etre public n'a aucun sens dans
    // une hiérarchie car le polymorphisme se perdrait.
    // De plus, c'est requis par les constructeurs de copie de descendants,
    // => protégé
    Mere(Mere const& rhs) : nom_(rhs.nom()) {}
 
 
private:
    // Constructeur par défaut interdit jusqu'à preuve de la 
    // nécessité contraire
    Mere();
 
    // Avoir un opérateur d'affection n'a aucun sens dans une 
    // hiérarchie polymorphique!!!
    // => déclaré privé, non défini (<=> interdit)
    Mere& operator=(Mere const&)
 
    // implémentation de la propriété nom de la classe. C'est un
    // détail d'implémentation, on encapsule donc => privé
    std::string nom_;
};
 
// héritage public (<=> polymorphisme d'héritage/inclusion/substitution,
//  les objets Fille sont sustituables là où l'on attend des objets Mere; 
// google => LSP)
struct Fille : Mere 
{
    virtual ~Fille() {}
    Attrb attrb() const { return attrb_; }
 
    // On implémente le clonage là où cela a du sens
    // Le type de retour change ; google => retour covariant
    virtual Fille * clone() const {return Fille(*this); }
 
    // Constructeur initialisant. Il est public pour pouvoir générer 
    // de tels objets depuis n'importe où
    // Il profite du constructeur initialisant de la classe mère, qui est
    // la seule à savoir comment est implémentée la propriété nom
    Fille(std::string const& un_nom, Attrb a)
    : Mere(un_nom)
    , attrb_(a)
    {}
 
protected:
    // Constructeur de recopie utilisé par la fonction clone
    // l'utiliser depuis ailleurs n'a aucun sens => protégé, au cas où la 
    // classe devrait encore être dérivée, privé suffirait sinon
    // Il profite du constructeur de recopie (protégé!) de la classe mère,
    // toujours la seule classe à savoir pour la propriété nom
    Fille(Fille const& rhs)
    : Mere(rhs)
    , attrb_(rhs.attrb())
    {}
 
private:
    Fille(); // interdit jusqu'à preuve du besoin contraire
    Fille& operator=(Fille const&); // interdit à tout jamais
 
    Attrb attrb_; // implémentation d'un nouvel attribut
};

07/11/2005, 10h21
Laurent Gomila

Ne pas oublier que ça se trouve dans la FAQ :wink:

http://c.developpez.com/faq/cpp/?pag...es#CLASS_clone
07/11/2005, 12h19
Battosaiii

euh ca m'explique pas pourquoi je peux utiliser le state name dans ma fonction. Pourquoi je peux pas ?
07/11/2005, 12h22
diogene

Montre la déclaration des classes, sinon, on ne peut rien dire

Voila ce qu'il y a dans FlyingMachine.h c'est la classe mere
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 #ifndef FlyingMachine_h //It checked it has been defined #define FlyingMachine_h #include<iostream> #include<string> using namespace std; class FlyingMachine { protected : // I want derived class to use them that's why it is protected bool Engine; bool OnAir; string name; //
Je veux utiliser string name dans la definition du copy constructor
Code:

1 2 3 4 plane::plane(const plane &sourceplane): landingGearOn(false){} //name(sourceplane.name){}
Mais il m'indique une erreur comme indique dans le premier message.
Pourquoi ?

07/11/2005, 15h29
Laurent Gomila

Tu ne peux pas initialiser une donnée membre appartenant à une classe de base, dans la liste d'initialisation d'une classe dérivée. Passe plutôt par le constructeur par copie de ta classe mère.
07/11/2005, 22h26
Luc Hermitte

Citation:

Envoyé par Loulou24

Ne pas oublier que ça se trouve dans la FAQ :wink:
http://c.developpez.com/faq/cpp/?pag...es#CLASS_clone

Mais la FAQ ne traite pas du problème qu'il a eu. Bon d'accord, je n'ai pas expliqué, mais juste donné le code type complet pour la copie polymorphique.
09/11/2005, 10h03
Battosaiii

ma classe mere est abstraite donc je ne peux pas utiliser le copy constructeur.

L'un n'empêche pas l'autre. Je te renvoie au code que Luc a posté précédemment :

Code:

1
2
3
4
5
// Constructeur de recopie. Etre public n'a aucun sens dans
// une hiérarchie car le polymorphisme se perdrait.
// De plus, c'est requis par les constructeurs de copie de descendants,
// => protégé
Mere(Mere const& rhs) : nom_(rhs.nom()) {}

09/11/2005, 10h33
Battosaiii
pour une fois j'ai trouve la solution tout seul en ajoutant comme tu viens de le faire dans la class mere puis :
Code:

1 2 3 plane::plane(const plane &sourceplane): FlyingMachine(sourceplane),landingGearOn(false){}
Dans la classe fille .