Est-ce que mes destructeurs sont corrects ?Résolu

Question

Bonjour, voila, j'ai les classes suivantes, et le programme suivant :
class st2
{
public:
int nb;            // Le nombre de briques de chacun des joueurs     
int *liste;        // La liste des briques (de dimension nb);


~st2();  // Le destructeur de la classe
};




class st1
{
st2 *st;    
~st1();              // Le destructeur de la classe
};

// Les destructeurs
st2::~st2(){
delete liste;
}
st1::~st1(){
delete st;
}

// Le programme qui l'utilise
main(){
st1 liste_st[5];
for(int i=0;i<5;i++){
st=new st2[2];
for(int j=0;j<5;j++){
liste=new int[4];
}
}

J'aimerais tout simplement que vous me disiez si les destructeurs sont corrects, merci  :)

luhtor · Accepted Answer

Pour ta classe st2, c'est "delete[]" ( [ ] ) et non delete qu'il faut utiliser puisque tu détruits un tableau.


Par contre, ca c'est n'importe quoi:

main(){

st1 liste_st[5];

for(int i=0;i<5;i++){

st=new st2[2];

for(int j=0;j<5;j++){

liste=new int[4];

}

}

nisaloncaje · Answer

pourquoi ? (c'est du code raccourci que j'ai utilisé pour l'exemple)

luhtor · Answer

Pourquoi, bas qu'est ce que tu essais de faire avec ca ?

st1 liste_st[5];

 for(int i=0;i<5;i++)

{

 st=new st2[2];

 for(int j=0;j<5;j++)

{

 liste=new int[4];

 }

}

cs_ronaldo1 · Answer

En effet avec ce programe tu va allouer de la mémoire pour st 5 fois et pour liste (qui n'ai même pas créé) 25 fois

cs_ronaldo1 · Answer

Je te conseil pour le liste d'allouer la mémoire dans le construteur de la class st2 style:

st2::st2(int i){ 
    nb = i;
    liste = new int[i]; 
}

nisaloncaje · Answer

ok. merci.

Est-ce que mes destructeurs sont corrects ?

6 réponses

Votre réponse

Discussions similaires