Discussion :

[parano]

Hello,

j'ai écrit une classe Vacances (je ne met pas tout le code car trop long) :


Je ne comprends pas pourquoi car j'ai bien pris soin de mettre les adresses dans le constructeur de Vacances.

Est-ce que quelqu'un pourrait m'indiquer ce que je dois faire pour que mon code fonctionne dans le main.

Merci

[Laurent Gomila]

Essaye de remplacer tes "\n" par des std::endl pour forcer l'affichage.

[Melem]

Tes classes sont très mal construites, je me demande même comment se fait-il que ca se compile. Ta classe Vacances dérive donc si j'ai vraiment compris des classes Plage et Glacier? On doit donc avoir quelque chose du genre :

Code :

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

1
2
3
4
5
Vacances::Vacances(int argument_de_Plage, int argument_de_Glacier):
Plage(argument_de_Plage), Glacier(int argument_de_Glacier)
{
    return; //ou rien
}

Mais dans ton cas, même si je ne sais pas vraiment ce que t'as réellement envi de faire, je crois que Vacances ne doit pas dériver de Plage et Glacier. On doit seulement avoir

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
class Plage;
class Glacier;
 
class Vacances
{
protected:
    Plage& plage;
    Glacier& glacier1;
    Glacier& glacier2;
public:
    Vacances(Plage& plage, Glacier& glacier1, Glacier& glacier2);
};

Et ca:

Code :

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

Vacances::~Vacances(){}

C'est inutile.

[Laurent Gomila]

Tes classes sont très mal construites, je me demande même comment se fait-il que ca se compile. Ta classe Vacances dérive donc si j'ai vraiment compris des classes Plage et Glacier?

Non, d'après ses sujets précédents, plage, glacier1 et glacier2 sont des variables membres de Vacances.

Et ca:

Code :

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

Vacances::~Vacances(){}

C'est inutile.

Pas s'il est virtuel, ce qui semble probable vu qu'on est en plein dans une hiérarchie de classes.

[Melem]

Et ca:

Code :

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

Vacances::~Vacances(){}

C'est inutile.

Pas s'il est virtuel, ce qui semble probable vu qu'on est en plein dans une hiérarchie de classes.

Bien sur, un déstructeur d'une classe est la plupart du temps une fonction virtuelle mais dans ce cas il doit explicitement appeler le déstructeur de la classe de laquelle elle dérive.

[Laurent Gomila]

Bien sur, un déstructeur d'une classe est la plupart du temps une fonction virtuelle mais dans ce cas il doit explicitement appeler le déstructeur de la classe de laquelle elle dérive.

Absolument pas

[Melem]

Cooooo...ool!!!

[Laurent Gomila]

Ca marche, mais il faut savoir pourquoi.

En gardant dans ta classe des références vers des plages et glaciers externes, ce sont bien ceux que tu vas passer au constructeur qui seront modifiés par lancement_simulation(). Ce que tu faisais avant, c'était stocker des copies de ces paramètres, et donc les modifications engendrées par lancement_simulation() n'avaient aucune incidence sur les plages et glaciers originaux.

Après il faut voir si c'est réellement le comportement souhaité.