Discussion :

[odysseeh]

Bonsoir à tous

je voudrais quelques explications concernant les codes de programmes suivant, il s'agit de définitions de constructeur (surchargé)à vrai dire je suis quelque peu dérouté en voyant apparaitre des objet et pointeur dans la définition du constructeur.
Il s'agit de 2 classes, la classe Point et la classe Graphe, N étant le nombre de point du graphe

Code :

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

1
2
3
4
5
Graphe::Graphe(Point const *const Plist, const int N)
{
    iNbPoint=N;
    std::cout << "appel du constructeur graphe" << std::endl;
};

de même j'ai une autre surchage de constructeur avec en paramètre un vector et une référence

Code :

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

1
2
3
4
5
6
7
Graphe::Graphe(std::vector<Point> &Plist )
{
    for (unsigned int i=0;i<ArrayPoint.size();i++)
        {
            ArrayPoint[i]=Plist[i];
            std::cout << "appel du constructeur graphe par transmission" << std::endl;
        };

il s'agit d'extrait de mon tp si vous avez besoin de plus d'infos je pourrais éventuellement vous envoyer la totalité du tp

Merci

[imperio]

En général on apprécie que les gens lisent les règles du forum avant de poster...

Sinon pour ton problème, sois un peu plus précis : qu'est-ce que tu ne comprends pas ?

[ternel]

Bonjour,
juste parce que mes yeux brûlent:

Code :

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

1
2
3
4
5
Graphe::Graphe(Point const *const Plist, const int N)
: iNbPoint(N)
{
    std::cout << "appel du constructeur graphe" << std::endl;
};

[odysseeh]

j'ai du mal à assimiler la présence d'un objet et de pointeur dans le corp du destructeur.
Mea culpa pour la retranscription du code je ne me suis pas encore familiariser avec les outils proposé sur le ruban.

[Flodelarab]

j'ai du mal à assimiler la présence d'un objet et de pointeur dans le corp du destructeur.

Je ne vois pas de destructeur. Un destructeur de la classe Graphe serait

Code :

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

Graphe::~Graphe

Comme il n'y a pas de tilde, il n'y a pas de destructeur.

Quel est ton problème?

PS: Pour faire un objet Salade, on peut utiliser Endives ou Laitue ou Tomates. Pas étonnant qu'il existe plusieurs constructeurs.

[Bousk]

Bonsoir,

qu'est-on sensé trouvé surprenant dans ce code ?
Un pointeur dans un constructeur ? Et ?
Pourquoi serait-il interdit pour un objet d'instancier un autre objet ? As-tu déjà entendu parler de composition ? Un design pattern porte même son nom.

Dans le destructeur ? Idem.
Sans compter que tu ne montres aucun destructeur.

Tes constructeurs sont de simples constructeurs avec paramètres. Ni constructeur par défaut, ni par copie.

[Astraya]

Bonjour,

Il n'y a rien de très éffrayant la dedans.
Comme l'a dit leternel, utilise ce que l'on appelle les listes d'initialisations (FAQ) autant que possible. Cela te permets de créer les variables membres de l'objet "avant" l'objet, ce qui évite une affectation de chaque paramètre après l'appelle du constructeur. Tu auras un gain de performance considérable sur le code.

Code :

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

1
2
3
4
5
Graphe::Graphe(Point const *const Plist, const int N)
: iNbPoint(N)
{
    std::cout << "appel du constructeur graphe" << std::endl;
};

Code :

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

1
2
3
Graphe::Graphe(std::vector<Point> &Plist )
:ArrayPoint(Plist)
{}

PS: tu as mis un ";" à la fin de ta boucle for, tu n'en as jamais sauf à la fin de la déclaration d'une classe et pour un do...while;

j'ai du mal à assimiler la présence d'un objet et de pointeur dans le corp du destructeur.

Il n'y a rien de magique ou de compliqué. Fais juste très attention au pointeur, car tu dois exactement savoir pourquoi tu passe un pointeur car c'est très dangereux. Imagine ceci:

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
class A
{
    CClasse * m_ptr;
 
public:
    A()
     :ptr( new CClasse)
    {
    }
 
    ~A()
    {
       delete m_ptr;
    }
};
class B
{
    CClasse * m_ptr;
 
public:
    B(CClasse * ptr)
     :ptr( ptr)
    {
    }
 
    ~B()
    {
       delete m_ptr;
    }
};

Avec ces classes tu auras un gros problème si :

Code :

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

1
2
3
4
5
 
A * a = new A();
B * b = new B(A);
delete A;
b->m_ptr->foo(); //CRASH car le pointeur à été delete avec A!!

Une solution pour ça est le shared_ptr.


Pour la copie du vecteur, le faire avec la liste d'initialisation appellera le constructeur par copie de la classe std::vector qui s'occupe toute seule de faire appelle à la copie des objets dans le même ordre. http://www.cplusplus.com/reference/v...vector/vector/

EDIT:

Attention, ligne 7: :ptr( new CClasse)

Effectivement, Désolé et merci Medinoc

[Médinoc]

Attention, ligne 7: :ptr( new CClasse)