deux classes et liste initialisation

Version imprimable

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

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

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

07/04/2015, 00h42
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 ?
07/04/2015, 10h37
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
09/04/2015, 18h57
JolyLoic

Citation:

Envoyé par darkman19320

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 :)
09/04/2015, 20h17
darkman19320
Citation:

Envoyé par JolyLoic

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:

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:

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)
09/04/2015, 21h36
JolyLoic
Citation:
Envoyé par darkman19320

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

Code:

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:

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.

Citation:

Envoyé par darkman19320

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).
11/04/2015, 20h02
stracoma
Bonsoir.
jo_link_noir:

Citation:

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

Citation:

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:

Citation:

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:

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

Citation:

Envoyé par stracoma

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:

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:

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

11/04/2015, 22h25
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 :lol: