Discussion :

[it45200]

Bonjour,

si j'ai 2 classes: disons pour simplifier une classe Moteur et une classe Voiture dont un des membre est de type Moteur, comment initialiser le membre de type Moteur dans le constructeur de la class Voiture:

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
class Moteur
{
	private:
	string type_moteur_;
	int nb_chevaux_;
 
	public:
	Moteur(string type_moteur,int nb_chevaux)
	: type_moteur_(type_moteur), nb_chevaux_(nb_chevaux)
	{
	}
 
	string getTypeMoteur()
	{
		return type_moteur_;
	}
 
	int getNbChevaux()
	{
		return nb_chevaux_;
	}
 
 
};
 
class Voiture
{
 
	private:
	string type_;
	string marque_;
	string model_;
	string couleur_;
	Moteur monMoteur_(string s,int i);
 
 
 
 
	public:
	Voiture(string type, string marque, string model, string couleur, Moteur monMoteur)
	: type_(type), marque_(marque), model_(model), couleur_(couleur)//, monMoteur_(Moteur(monMoteur.getTypeMoteur(),monMoteur.getNbChevaux()))
	{
 
	}
 
    string getTypeVoiture()
    {
        return type_;
    }
 
     string getMarqueVoiture()
    {
        return marque_;
    }
 
     string getModelVoiture()
    {
        return model_;
    }
 
     string getCouleurVoiture()
    {
        return couleur_;
    }
 
 
 
};

J'ai essayé avec le bout commenté dans la liste d'initialisation du constructeur voiture

Code :

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

 monMoteur_(Moteur(monMoteur.getTypeMoteur(),monMoteur.getNbChevaux()))}

mais j'obtiens un message d'erreur à la compilation:

Code :

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

error: class 'Voiture' does not have any field named 'monMoteur_'

Merci pour votre aide.

Bien cordialement,

[ternel]

Pour appeler le constructeur de std::string, tu n'as pas écrit marque_(std::string(marque)), mais marque_(marque).
Pour le moteur, c'est pareil: monMoteur_(monMoteur.getTypeMoteur(),monMoteur.getNbChevaux()).

Deuxièment, Moteur monMoteur_(string s,int i); est une déclaration de fonction, pas de variable.

Attention au nom des parametres, ce n'est pas le "monMoteur" de l'objet que tu donne, juste celui que tu vas copier, j'opterais plutot pour "moteur"
Quant à la variable membre, je t'invite à choisir entre monBidule et Bidule_, car monBidule_ est redondant.

Je rajoute les références constantes et surtout les fonctions constantes.

Une déclaration de classe étant en général dans un header, il ne faut pas y mettre de using. Ni using std::string;, ni using namespace std;.

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
class Moteur {
private:
	std::string type_moteur_;
	int nb_chevaux_;
 
public:
//un constructeur par copie est automatiquement généré
	Moteur(std::string const& type_moteur, int nb_chevaux) : type_moteur_(type_moteur), nb_chevaux_(nb_chevaux) {}
 
	std::string getTypeMoteur() const {return type_moteur_;}
 
	int getNbChevaux() const {return nb_chevaux_;}
};
 
class Voiture {
private:
	std::string type_;
	std::string marque_;
	std::string model_;
	std::string couleur_;
	Moteur monMoteur_;
public:
	Voiture(std::string const& type, std::string const& marque, std::string const& model, std::string const& couleur, Moteur const& moteur) :
		type_(type), marque_(marque), model_(model), couleur_(couleur),
		monMoteur_(moteur.getTypeMoteur(),moteur.getNbChevaux()) // ou monMoteur_(moteur)
	{}
 
	std::string getTypeVoiture() const {return type_;}
 
	std::string getMarqueVoiture() const {return marque_;}
 
 	std::string getModelVoiture() const {return model_;}
 
	std::string getCouleurVoiture() const {return couleur_;}
};

[LittleWhite]

Bonjour,

Dans la classe Voiture, ce n'est pas :

Code :

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

Moteur monMoteur_(string s,int i);

mais simplement :

Code :

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

Moteur monMoteur_;

Qu'il faut écrire, car c'est une instance de variable comme une autre (comme string, par exemple).

Ensuite, coté constructeur de la Voiture, il faut écrire :

Code :

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

1
2
3
4
5
Voiture(string type, string marque, string model, string couleur, Moteur monMoteur)
	: type_(type), marque_(marque), model_(model), couleur_(couleur), monMoteur_(monMoteur.getTypeMoteur(),monMoteur.getNbChevaux())
	{
 
	}

Mais je pense que vous pourriez juste faire une copie par le biais du constructeur par copie :

Code :

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

1
2
3
4
5
Voiture(string type, string marque, string model, string couleur, Moteur monMoteur)
	: type_(type), marque_(marque), model_(model), couleur_(couleur)//, monMoteur_(monMoteur)
	{
 
	}

Ensuite, dernier conseil : les string et le moteur devrait être des références constante afin d'éviter la copie de ces variables à chaque appel de fonction et une copie supplémentaire qui ne sert pas.

[it45200]

Merci à tous pour vos retours et vos conseils.
Bonne journée.

[koala01]

Salut,

En outre, il existe une loi nommé loi de déméter qui dit

Si un objet A (la voiture) utilise en interne un objet B (le moteur), l'utilisateur de A ne devrais pas avoir à connaitre B pour pouvoir utiliser son A

Autrement dit, tout ce que l'utilisateur (en ce, y compris celui qui va la créer effectivement) d'un objet de type voiture ne doit jamais avoir à s'inquiéter du fait qu'un réservoir est effectivement utilisé.

Qu'à la limite, il pense que le carburant est maintenu en place par l'action du saint esprit si ça l'amuse, grand bien lui fasse : tout ce qu'il a besoin de savoir à ce sujet, c'est:

• Où faire rentrer le carburant dans la voiture (la trappe à carburant)
• De quelle quantité / ratio de carburant il dispose (jauge)
• (éventuellement) la quantité de carburant utilisée sur une distance donnée (consommation moyene / instantanée / autre ?)
• (éventuellement) l'autonomie que lui laisse la quantité de carburant dont il dispose
• ET C'EST TOUT (ou peu s'en faut)

Du coup, au lieu de passer un objet de type réservoir au constructeur de ta voiture, tu devrais transmettre une chaine de caractères "typeMoteur" et un entier "puissanceMoteur" qui seront transmis au constructeur du moteur