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