Discussion :

[stracoma]

Bonsoir à vous. J'ai besoin de votre aide. Regardons le code suivant:

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
#include <iostream>
#include <string>
using namespace std;
class Etudiant
{
private:
    string nom;
    string prenom;
    int age;
public:
    Etudiant(string n, string p, int a):
        nom(n),prenom(p),age(a){}
    void affiche()
    { cout<<nom<<" === "<<prenom<<" === "<<age<<endl;}
};
class Ecole
{
private:
    int nbr_de_classes;
    Etudiant membre;
public:
   Ecole(int ndc,string n,string p,int a):
        nbr_de_classes(ndc),membre(n,p,a){}
 
    void affiche()
    {
        cout<<nbr_de_classes<<endl;
        membre.affiche();
    }
 
};
int main()
{
   cout<<"Tout ce passe bien"<<endl;
    return 0;
}

Pourquoi ce nouveau constructeur d'initialisation de la classe Ecole ne marche pas?:

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
#include <iostream>
#include <string>
using namespace std;
class Etudiant
{
private:
    string nom;
    string prenom;
    int age;
public:
    Etudiant(string n, string p, int a):
        nom(n),prenom(p),age(a){}
    void affiche()
    { cout<<nom<<" === "<<prenom<<" === "<<age<<endl;}
};
class Ecole
{
private:
    int nbr_de_classes;
    Etudiant membre;
public:
    Ecole(int ndc,Etudiant e):
    nbr_de_classes(ndc),membre.nom(e.nom),membre.prenom(e.prenom),membre.age(e.age){}
 
 
    void affiche()
    {
        cout<<nbr_de_classes<<endl;
        membre.affiche();
    }
 
};
int main()
{
   cout<<"Tout ce passe bien"<<endl;
    return 0;
}

Merci à vous

[jo_link_noir]

Parce que la syntaxe n'est pas valide.
Le but de la liste d'initialisation est d'initialiser les membres, pas les membres des membres.
Et puis bon, avec cette forme, où se trouve l'appel du constructeur de Etudiant ?

[darkman19320]

Salut,

Comme le dit jo_link_noir, la syntaxe est incorrecte: tu ne peux pas faire appel aux fonctions d'un objet au niveau de l'instantiation de celui-ci (il n'existe pas encore).
Petit conseil, fait un constructeur par copie: regarde la FAQ

[JolyLoic]

Petit conseil, fait un constructeur par copie: regarde la FAQ

Pour quoi faire ? Ici, sauf si je n'ai vraiment pas les yeux en face des trous, ça ne servirait à rien. Et comme ce n'est pas si évident à écrire, autant ne rien faire, ça n'en marchera que mieux

[darkman19320]

Pour quoi faire ? Ici, sauf si je n'ai vraiment pas les yeux en face des trous, ça ne servirait à rien. Et comme ce n'est pas si évident à écrire, autant ne rien faire, ça n'en marchera que mieux

Vu que le constructeur de la classe Ecole est de cette forme :

Code :

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

Ecole(int ndc,Etudiant e);

Et que cette classe contient un étudiant, je voyais plus simple de faire un constructeur de copie pour la classe Etudiant au lieu de faire:

Code :

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

1
2
Ecole(int ndc, const Etudiant & e): nbr_de_classes(ndc),membre(e.nom(), e.prenom(), e.age()) //En supposant que les accesseurs soient implémentés...
{}

D'après la question, il s'agit d'un débutant donc autant lui apprendre à faire des constructeurs par copie, non? (bien que dans ce cas précis, le constructeur par copie généré automatiquement par le compilateur devrait faire l'affaire)

[JolyLoic]

Vu que le constructeur de la classe Ecole est de cette forme :

Code :

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

Ecole(int ndc,Etudiant e);

Et que cette classe contient un étudiant, je voyais plus simple de faire un constructeur de copie pour la classe Etudiant au lieu de faire:

Code :

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

1
2
Ecole(int ndc, const Etudiant & e): nbr_de_classes(ndc),membre(e.nom(), e.prenom(), e.age()) //En supposant que les accesseurs soient implémentés...
{}

Je suis d'accord sur le fait qu'il est plus simple et plus souple de copier un étudiant plutôt que d'en reconstruire un pièce à pièce. Là où je ne suis pas d'accord, c'est d'écrire un constructeur de copie pour faire ça.

D'après la question, il s'agit d'un débutant donc autant lui apprendre à faire des constructeurs par copie, non? (bien que dans ce cas précis, le constructeur par copie généré automatiquement par le compilateur devrait faire l'affaire)

Je pense que s'il est important d'apprendre assez tôt la notion de copie, il est déjà primordial d'apprendre dans quels cas on n'a pas besoin de faire quoi que ce soit pour que la copie marche.

Apprendre comment écrire un constructeur de copie pas par défaut me semble bien moins urgent. Je ne dis pas qu'il ne faut pas savoir le faire, seulement, je considère ça comme un sujet déjà avancé (et le nombre de classes où j'ai vu des constructeurs de copie mal implémentés me conforte dans mon opinion). C'est plein de subtilités sur la manière de l'implémenter correctement et sans pertes de performances, ça demande d'assimiler non pas 1 mais 5 fonctions spéciales d'un coup, c'est un risque de plus lors des modifications du code.

Et ce qui me gêne le plus, c'est que d'en montrer trop tôt, alors que ce n'est pas nécessaire, sous-entend que c'est une pratique normale d'en mettre (j'ai vu des gens qui pensaient qu'il fallait en mettre pour toutes les classes), alors que c'est l'exception (je n'arrive honnêtement pas à me rappeler la dernière fois que j'en ai mis un dans du vrai code).

[stracoma]

Bonsoir.
jo_link_noir:

Parce que la syntaxe n'est pas valide.
Le but de la liste d'initialisation est d'initialiser les membres, pas les membres des membres.

après ça je me suis dit je laisse tomber.
Mais j'ai suivi le conseil de: darkman19320

Petit conseil, fait un constructeur par copie: regarde la FAQ

J'y suis allé, mais mon problème (depuis mon enfance), je n'ai jamais compris un cours non illustré par un exemple, ou plutôt un exercice basique mais corrigé.
après ce message:

Vu que le constructeur de la classe Ecole est de cette forme :

Code :Sélectionner tout - Visualiser dans une fenêtre à part
Ecole(int ndc,Etudiant e);
Et que cette classe contient un étudiant, je voyais plus simple de faire un constructeur de copie pour la classe Etudiant au lieu de faire:

Code :Sélectionner tout - Visualiser dans une fenêtre à part
1
2
Ecole(int ndc, const Etudiant & e): nbr_de_classes(ndc),membre(e.nom(), e.prenom(), e.age()) //En supposant que les accesseurs soient implémentés...
{}
D'après la question, il s'agit d'un débutant donc autant lui apprendre à faire des constructeurs par copie

Effectivement, je suis un "pré-débutant", j'ai vainement essayé d'utiliser ce constructeur de copie mais sans succès (car je ne l'ai jamais fait auparavant);
si quelqu'un me le fasse pour que je puisse l'assimiler.
J'ai essayé :

Code :

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

1
2
public:
    Ecole(int ndc, const Etudiant & e): nbr_de_classes(ndc),membre(e.nom(), e.prenom(), e.age()) {}

mais le débogueur me signale que nom est privé.

[Iradrille]

Effectivement, je suis un "pré-débutant", j'ai vainement essayé d'utiliser ce constructeur de copie mais sans succès (car je ne l'ai jamais fait auparavant);
si quelqu'un me le fasse pour que je puisse l'assimiler.

Si, tu l'as déjà utilisé, et bien plus souvent que tu ne le penses.

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
class Etudiant
{
private:
    string nom;
    string prenom;
    int age;
public:
    // tu prends des string en parametres et non des refs constantes : ça implique une copie
    // et donc un appel au constructeur par copie
    Etudiant(string n, string p, int a):
        nom(n), // utilisation du ctor par copie de std::string (2eme copie donc)
        prenom(p), // pareil
        age(a)
    {}
    void affiche()
    { cout<<nom<<" === "<<prenom<<" === "<<age<<endl;}
};

(C'est probablement un constructeur par mouvement qui sera appelé, mais ça ne change pas grand chose ici).

Pour les std::string, tu les copies sans t'occuper de ce qu'ils contiennent réellement. C'est exactement la même chose pour Etudiant.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class Ecole
{
private:
    int nbr_de_classes;
    Etudiant membre;
public:
    Ecole(int ndc,Etudiant e):
       nbr_de_classes(ndc),
       membre(e) // appel du ctor par copie de Etudiant, celui par défaut suffit ici
    {}
};

[stracoma]

Iradrrille, Merciiiiiiiii
C'est ce que je cherchais depuis le début. Maintenant ça marche et surtout j'ai compris la logique.
C'est dommage q'on ne peut pas voter un message plus d'une fois