Discussion :

[Battosaiii]

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 :

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

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 ?

[Luc Hermitte]

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

Code :

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

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
};

[Laurent Gomila]

Ne pas oublier que ça se trouve dans la FAQ

http://c.developpez.com/faq/cpp/?pag...es#CLASS_clone

[Battosaiii]

euh ca m'explique pas pourquoi je peux utiliser le state name dans ma fonction. Pourquoi je peux pas ?

[diogene]

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

[Battosaiii]

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

Code :

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

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 :

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

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 ?

[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.

[Luc Hermitte]

Ne pas oublier que ça se trouve dans la FAQ
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.

[Battosaiii]

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

[Laurent Gomila]

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

Code :

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

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()) {}

[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 :

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

1
2
3
 
plane::plane(const plane &sourceplane):
FlyingMachine(sourceplane),landingGearOn(false){}

Dans la classe fille .