Discussion :

[milomar]

Bonjour,

J'ai un problème en ce qui concerne la communication entre classes.

Je viens de commencer dans l'orienté objet et j'aimerais réaliser un programme qui comporte deux classes.

Mon problème est le 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
class Premier
{
     constructeurs;
     destructeur;
     méthodes;
     attribut 1;
     attribut 2;
}
 
class Seconde
{
    constructeurs;
    destructeur;
    méthodes;
    attribut 3;
    attribut 4;
}

Ce que j'aimerais faire c'est créer un objet de la classe "Premier" dans mon main, ce qui appelera mon constructeur. En fonction des valeurs passées en paramètre à ce constructeur, j'aimerais alors pouvoir agir sur un objet de la classe "Seconde" en modifiant l'attribut 3.

je ne sais pas comment faire cela...
j'ai essayé la chose suivante :

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
class Premier
{
     constructeurs;
     destructeur;
     méthodes;
     attribut 1;
     attribut 2;
     Seconde objet2;
}
 
dans mon main():
{
     Premier objet1(param.)
     cout<<objet1.objet2.attribut3;
}

Le problème est que ceci me renvoie vraissemblablement une adresse et non le contenu de mon attribut. Selon moi, cet attribut est effectivement sans valeur car j'ai vérifié qu'à aucun moment mon programme n'entre dans le constructeur de mon objet2.

Pour moi, c'est dans le constructeur de la classe Premier qu'il faudrait ajouter quelque chose pour forcer l'appel au constructeur de l'objet 2.


je suis conscient que c'est pas facilement compréhensible la façon dont je viens d'expliquer les choses, et au besoin, je me montrerais plus explicite.

Merci d'avance.

[JolyLoic]

A partir du moment où la classe Première comporte un membre de type Seconde, un des constructeurs de la classe seconde est automatiquement appelé avant que le corps du constructeur de Première ne soit exécuter.

Comment as-tu testé que ton objet de type seconde n'était pas construit ?

Pour voir ton problème, il faudrait que tu postes du vrai code qui le reproduise.

[babar63]

Le problème est que ceci me renvoie vraissemblablement une adresse et non le contenu de mon attribut

D'après le "pseudo-code" que tu as présenté il n'y as pas de raison pour que tu es une adresse affichée, quel est le type de attribut3?
Comme l'a dit JolyLoic, le constructeur de ta classe Seconde seconde sera appelé juste avant celui de Premier, par défaut c'est bien sur le constructeur par défaut qui est appelé. Si tu le souhaite utilises la liste d'initialisation pour appelé un constructeur spécifique exemple :

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
class Seconde
{
public:
 
        Seconde() {}
        Seconde(int _i) : m_Att3(_i) {}
 
        ~Seconde() {}
 
        int m_Att3;
};
 
 
class Premier
{
public:
 
        Premier() {}
        Premier(int ParamObj) : m_Obj(ParamObj) {} //appel du constructeur 1parametre de Seconde
        Premier(int _i1, int _i2) : m_Att1(_i1), m_Att2(_i2) {}
 
	~Premier() {}
 
	int m_Att1;
	int m_Att2;
	Seconde m_Obj;
};
 
 
void main()
{
	Premier monPremier(2);
	std::cout << monPremier.m_Obj.m_Att3 << std::endl;
}

Ce code appelera dans un premier temps le constructeur à 1paramètre de Seconde puis le constructeur de Premier... Cela règle-t-il ton problème?

[Mat007]

Salut,

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
class Seconde
{
public:
 
        Seconde() {}
        Seconde(int _i) : m_Att3(_i) {}
 
        ~Seconde() {}
 
        int m_Att3;
};
 
 
class Premier
{
public:
 
        Premier() {}
        Premier(int ParamObj) : m_Obj(ParamObj) {} //appel du constructeur 1parametre de Seconde
        Premier(int _i1, int _i2) : m_Att1(_i1), m_Att2(_i2) {}
 
	~Premier() {}
 
	int m_Att1;
	int m_Att2;
	Seconde m_Obj;
};
 
 
void main()
{
	Premier monPremier(2);
	std::cout << monPremier.m_Obj.m_Att3 << std::endl;
}

Il faut initialiser toutes les données membres dans les listes d'initialisation de tous les constructeurs, sinon certaines ne le seront pas dans certains cas et au final c'est le n'importe quoi assuré.
Mais peut-être que les constructeurs qui ne servent pas devraient être supprimés surtout.

MAT.

[babar63]

C'était un exemple écrit en vitesse pour montrer l'appel au constructeur dans la liste d'initialisation mais j'aurais du le préciser car c'est vrai que ce code n'a aucun sens

[milomar]

Bonjour,

Avant tout merci à tous pour vos réponses ...

Malheureusement c'est ce que j'ai fait il me semble ... Je vous envoie ici le code très simplifié de ce que je veux faire.

J'ai créé un projet divisé en plusieurs fichiers séparés .cpp et .hpp :
Malheureusement cela ne fonctionne pas ... j'ai une tonne d'erreurs à la compilation dont "Plateau is not a type".

Serait-ce possible que ce soit un problème d'édition des liens ?

De plus, le code ci-dessous, s'il compilait, appellerait le constructeur par défaut pour l'objet de la classe plateau mais comment devrais-je faire pour qu'il appelle le constructeur avec un paramètre. En effet, le compilateur m'interdit de mettre un paramètre lors de ma déclaration dans le fichier Jeu.hpp (du type : Plateau p(int)).

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
 
//Main.cpp
 
#include <iostream>
#include "Jeu.hpp"
#include "Plateau.hpp"
 
using namespace std;
 
int main ()
{
    int param;
 
    cout<<"Entrer parametre 1"<<endl;
    cin>> param;
 
    Jeu game (param);
    cout<< game.p.nombre<<endl;
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
//Jeu.hpp
 
class Jeu
{
  public
        Jeu(int);
        Jeu();
        ~Jeu();
        int attribut1;
        Plateau p;
 
 
};

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
 
//Jeu.cpp
 
#include <iostream>
#include "Jeu.hpp"
 
using namespace std;
 
Jeu::Jeu(int a)
{
      attribut1 = a;
}
 
Jeu::Jeu()
{}
 
Jeu::~Jeu()
{}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
// Plateau.hpp
class Plateau
{
  public
        Plateau(int);
        Plateau();
        ~Plateau();
        int nombre;
 
 
};

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
 
// Plateau.cpp
 
#include <iostream>
#include "Plateau.hpp"
 
using namespace std;
 
Plateau::Plateau(int b)
{
      nombre = b;
}
 
Plateau::Plateau()
{
      nombre = -1;
}
 
Plateau::~Plateau()
{}

Merci d'avance à ceux qui prendront le temps de me débloquer de cette situation.

[JolyLoic]

Dans Jeu.hpp, tu utilises Plateau, sans avoir défini de quoi il s'agit. Ajoute un #include "Plateau.hpp" dans ce fichier.
Ensuite, tes fichiers ne sont pas protégés contre l'inclusion multiple.

Pour ton problème, j'ai l'impression que tu ne connais pas l'existence des liste d'initialisation.

Quand le C++ construit un objet, il :
- Commence à construire la partie concernant les classes de base éventuelles (avec les informations dans la liste d'initialisation).
- Puis il initialise les données membre, en utilisant les informations de la liste d'initialisation
- Puis il appelle le corps du constructeur.

La liste d'initialisation se défini ainsi :

Code :

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

1
2
3
4
5
Jeu::Jeu(int a) :
    p(a*a),
    attribut1(a)
{
}

Lire d'ailleur à ce sujet :