[Debutant] Destructeur

Bonjour,

Voila mon probleme :

J'ai une classe avec comme attributs :

Code:

---

1 2 3 4

int ordre; // tableau la matrice double long **tab;

---

Je suppose, vu que j'utilise de l'allocation dynamique, que je dois faire un destructeur mais j'avoue je sais pas trop comment.... :oops:

J'ai essayé dans mon destructeur de mettre:

Code:

---

1 2 3 4 5 6 7

for (int i=0;i<ordre;i++) { delete [] tab[i];   } delete [] tab;

---

Mais ça merde kan mon tab est vide...Je sais pas trop koi faire.

Merci d'avance pour votre aide...

salut,

il suffit de tester si le pointeur est valide ou non avant de tenter de détruire quelque chose qui n'existe peut-être pas :wink:

Code:

---

1 2 3 4 5 6 7

if (tab) { for (int i=0;i<ordre;i++) delete [] tab[i];   delete [] tab; }

---

Bonjour, d'abord merci pour ta réponse...

Je viens d'essayer ce que tu m'as donné et ça ne marche pas..... :(
J'ai droit a un beau message d'erreur du genre :


"Unhandled exception at 0x10252758 in MatriceBlooVisual.exe: 0xC0000005: Access violation reading location 0xccccccc0."

à la ligne "delete [] tab;"

quand mes tableaux ne sont pas alloués...et un truc pire quand mes tableaux sont alloués...

Voila le code de l'allocation dasn le constructeur si ca peut aider :

Code:

---

1 2 3 4 5 6

tab = new double long * [ordre]; // on détermine un tableau de 'ordre' pointeurs de type double long for (int i=0;i<=ordre;i++) { tab[i]= new double long[ordre+1]; // on détermine un tableau de 'ordre' double long pour chaque pointeur précédent }

---

Donc voila je suis un peu perplexe...

J'ai essayé ça :

Code:

---

1 2 3 4 5 6 7 8

// désallouer tout ce qui a été alloué for ( int i = 0; i < ordre; ++i ) { delete [] tab[ i ]; } tab = NULL; delete [] tab;

---

et ça a l'air de marcher mais je sais pas si ça libere correctement la memoire....

Citation:

---

Code:

---

1 2 3 4 5 6 7 8

// désallouer tout ce qui a été alloué for ( int i = 0; i < ordre; ++i ) { delete [] tab[ i ]; } tab = NULL; delete [] tab;

---

---

tab = NULL;
delete [] tab;

ça ne désalloue rien du tout là. en faisant tab = NULL, tu perds l'adresse vers la zone mémoire à désallouer (donc fuite de mémoire puisque tu ne pourras pas désallouer cette zone, n'ayant plus de pointeur vers elle).

sinon je ne peux rien tester là, n'ayant pas de compilo sous la main. désolé. :cry:

Oui c'est ce qui me semblait, que je supprimais qu'une partie....
Bon je vais continué de chercher. Pour info, j'utilise Visual C++ 2005.
Merci de ton aide...

ok, a priori delete[] n'intervient que dans le cas de tableaux d'objets. ici ce n'ets pas le cas, donc :

Code:

---

1 2 3 4 5 6 7 8 9 10

// désallouer tout ce qui a été alloué if (tab) { for ( int i = 0; i < ordre; ++i ) if (tab[i]) // pas nécessaire dans ton cas je pense delete tab[i];   delete tab; }

---

Navré mais ça ne marche pas non plus :

ca me met dans tous les cas et toujours sur le "delete tab":

http://up.mezimages.com/up/602670hump.jpg

:evil:

....
??


Encore merci de ton aide...

je viens de m'apercevoir qu'il y a une erreur lors de l'allocation

Code:

---

1 2 3 4 5 6

tab = new double long * [ordre]; // on détermine un tableau de 'ordre' pointeurs de type double long for (int i=0;i<=ordre;i++) { tab[i]= new double long[ordre+1]; // on détermine un tableau de 'ordre' double long pour chaque pointeur précédent }

---

for (int i=0;i<ordre;i++)

1- faq
2- attention aux copies qui si non blindées finiront sur des doubles delete
3- Ce qui existe déjà est ton ami (vector<vector<T>>, boost.multi_array, boost.ublas, blitz++, ...). Cela reste toutes fois un excellent exercice.

Il faut dire dans le constructeur que tab vaut NULL par défaut sans quoi il est initialisé à n'importe quoi.

Pour le destructeur, utiliser :

Code:

---

1 2 3 4 5 6 7 8 9

if (tab) { for ( int i = 0; i < ordre; ++i ) if (tab[i]) // pas nécessaire dans ton cas je pense delete [] tab[i];   delete [] tab; }

---

En modifiant le constructeur qui effectivement avait une erreur :oops: :

Code:

---

1 2 3 4 5 6

tab = new double long * [ordre]; // on détermine un tableau de 'ordre' pointeurs de type double long for (int i=0;i<ordre;i++) { tab[i]= new double long[ordre+1]; // on détermine un tableau de 'ordre' double long pour chaque pointeur précédent }

---

Cela semble marcher....Toutefois il faut que je redefinisse aussi le constructeur par recopie, mais bon je devrais ya arriver.
Merci pour votre aide.

++