Discussion :

[perpau07]

Bonjour,

Je vais d'abord vous mettre le code o� est le probl�me. Le programme g�re des parties et des joueurs d'�checs.

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
#include <cstdlib>
#include <iostream>
#include "Joueur.h"
#include "partie.h"
using namespace std;
 
int main(void)
{
    Joueur Albert("albert");
    Joueur Bertrand("bertrand");
    Partie noob(&Albert,&Bertrand);
    Partie pro;
    pro.setJ1(&Albert);
    pro.setJ2(&Bertrand); 
    cout<<(int)Albert.getpartie()[0]<<endl;   
    cout<<(int)&noob<<endl;   
    cout<<"bla"<<endl;
    pro.~Partie();
    noob.~Partie();
    system("PAUSE");
    cout<<"bla"<<endl;
    return EXIT_SUCCESS;   
}

Voil� les variables des classes :

Extrait de Joueur.h :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
// variables de la classe :
	char * nom; // nom du joueur
	Partie * partie[5]; // tableau de pointeur sur des parties en cours

Extrait de Partie.h :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
// variables de la classe :
 
              char * date; // Pas de classe date pour l'instant
              Joueur * J1; //2 joueurs distincts
              Joueur * J2;

Voici �galement le code des destructeurs de chacune des 2 classes :

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
Joueur :: ~Joueur (){
       delete [] nom;
       for (int i = 0 ; i < 5 ; i++ ){
             if (partie[i]!=NULL){
                    partie[i]->~Partie();
             }
       }
}
 
Partie :: ~Partie (){ // le destructeur de partie qui détruit le tableau alloué dynamiquement et défait la liaison réciproque des objets
       delete [] date;
       for (int i = 0 ; i < 5 ; i++){ //on va ici parcourir le tableau des parties du joueur 1 a la recherche de la partie en cours          
           if ( J1->getpartie()[i] == this ){
                J1->minisetpartie(NULL,i); // une fois qu'on l'a trouvé on le supprime                                   
                }
           }
       for (int i = 0 ; i < 5 ; i++){ // on fait de même pour le deuxième joueur        
           if ( J2->getpartie()[i] == this ){
                J2->minisetpartie(NULL,i);
                }
       }
       J1=NULL; // enfin on suprime les liens de la partie vers les joueurs 1
       J2=NULL; // et 2
 
}

Probl�me : le programme affiche les 2 "bla" et plante apr�s : "Ce programme a cess� de fonctionner. Windows recherche une solution au probl�me"

Ce probl�me se produit donc apr�s l'appel aux destructeurs, pourtant il ne se produit pas si je ne les appelle pas... Pourquoi ?

[Invit�(e)]

Bonjour

C'est normal, dans ton code, tu cr�es deux objets Partie statiques : le programme se chargera de les cr�er ET de les supprimer.

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 Toto;
int f1(){ // Cas statique
   Toto obj_1; // OK on ne s'occupe ni de la contruction, ni de la destruction
}
int f2(){ // Cas dynamique
   Toto * pObj_2;
   pObj_2 = new Toto();
   delete pObj_2; // OK on s'occupe des deux
}
int f3(){
   Toto obj_3; 
   delete & obj_3; // PAS OK : c'est pas notre rôle de libérer l'objet
}
int f4(){
   Toto * pObj_4; 
   pObj_4 = new Toto(); // Pas OK, on crée l'objet 4, mais on ne le détruit pas -> fuite mémoire...
}

Dans ton cas, tu passes donc deux fois par le destructeur... tu d�truis donc deux fois date et les appels � J1->... J2->... se font en fait avec un pointeur NULL en guise de Joueur.

[perpau07]

Merci, mais la m�moire allou�e dans chaque objet (nom du joueur, date de la partie) n'est pas lib�r�e, si ?

[Invit�(e)]

Merci, mais la m�moire allou�e dans chaque objet (nom du joueur, date de la partie) n'est pas lib�r�e, si ?

Disons que le destructeur est appel�.
Apr�s la m�moire est lib�r�e si on demande � la lib�rer. (Ce qui pose souvent probl�me lorsqu'on utilise des pointeurs � l'int�rieur d'une classe)
Au passage : page de la FAQ sur les destructeurs
Pour s'assurer du passage dans les diff�rents constructeurs/destructeurs, il est souvent utile d'ajouter une trace :

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
#ifdef _DEBUG
#define TRACE_CONS cout << "Construction de " << this << endl;
#define TRACE_DEST cout << "Destruction de " << this << endl;
#else
#define TRACE_CONS 
#define TRACE_DEST 
#endif
 
#include <iostream>
using namespace std;
class Toto{
public:
   Toto(){
      TRACE_CONS;
   }
   ~Toto(){
      TRACE_DEST;
   }
};

Apr�s, pour �viter tout probl�me de fuite m�moire avec les chaines de caract�res, il est fortement conseilll� en c++ d'utiliser la classe string.

[perpau07]

Merci bien !