Discussion :

[Belenos]

Bonjour,

Voila j'ai un probleme avec les classes et je ne sais pas si ca marche pas parce que je suis pas logique ou je ne connais pas certaines commandes.

Voiture.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class voiture
{
     public:
          //methodes
          std::string get_marque() const;
     protected:
          //attributs
          std::string m_marque;
          Moteur m_moteur;
};

Moteur.h

Code :

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

1
2
3
4
5
6
7
8
9
10
 
class Moteur
{
     public:
          Moteur(Voiture)
          //methodes
     protected:
          //attributs
          int m_num_serie;
};

Voiture.cpp

Code :

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

1
2
3
4
5
 
std::string Voiture::get_marque() const
{
      return m_marque;
}

Moteur.cpp:

Code :

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

1
2
3
4
5
6
7
 
Moteur::Moteur(Voiture voiture)
{
     if(voiture.get_marque()=="AB")
           m_num_serie=45;
     //else if.....etc pour les autres
}

J'aimerai récupérer m_marque de Voiture pour initialiser m_num_serie dans le constructeur de Moteur.

Supposons que:
si m_marque = AB alors m_num_serie = 45
si m_marque = JG alors n_num_serie = 8966
si m_marque = SF alors m_num_serie = 239
etc..

J'espère que je suis clair.
Et donc ca ne fonctionne pas le compilateur me dis qui connai pas Voiture dans Moteur.h
-> error: field 'Voiture' as incomplete type

[3DArchi]

Tu vas avoir un problème de conception.
Pour rappel, l'ordre d'appel des constructeurs est :
1/ D'abord ceux des membres
2/ Ensuite celui de la classe.
Donc Moteur::Moteur est appelée avant Voiture::Voiture puisque m_moteur est membre de Voiture. Donc tu ne peux lui passer en argument le voiture en train de construire.
En d'autres termes, si tu as ça :

Code :

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

1
2
 
Voiture::Voiture():m_moteur(*this){...}

C'est une erreur. Ton compilateur doit probablement râler...

[EDIT] : pourquoi ne pas passer la marque directement :

Code :

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

1
2
3
4
5
 
Voiture::Voiture(std::string const &marque_)
   :m_marque(marque_)
  ,m_moteur(marque_)// c'est bien marque_ et non m_marque !
{...}