Discussion :

[TraPpeur]

Bonjour,

Voila mon probleme :

J'ai une classe avec comme attributs :

Code :

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

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....

J'ai essayé dans mon destructeur de mettre:

Code :

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

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...

[bigboomshakala]

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

Code :

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

1
2
3
4
5
6
7
if (tab)
{
   for (int i=0;i<ordre;i++)
      delete [] tab[i]; 
 
   delete [] tab; 
}

[TraPpeur]

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 :

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

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 :

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

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....

[bigboomshakala]

Code :

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

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é.

[TraPpeur]

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...

[bigboomshakala]

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

Code :

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

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;
}

[TraPpeur]

Navré mais ça ne marche pas non plus :

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





....
??


Encore merci de ton aide...

[bigboomshakala]

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

Code :

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

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++)

[Luc Hermitte]

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.

[Matthieu Brucher]

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

[TraPpeur]

Pour le destructeur, utiliser :

Code :

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

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 :

Code :

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

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.

++