Probleme de desalocation memoire

Version imprimable

Comment fait on pour desalouer ca correctement :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    float ***data ;
 
    data = new float**[depth+1];// i et j commence a 1
    for(i=1;i<=depth;i++)
    {
        data[i] = new float*[deltaY+nb_zero +1 ];
        for(j=1;j<=(deltaX + nb_zero );j++)
        {
            data[i][j] = new float[2*(deltaX+nb_zero) +2 ];
 
        }
    }

30/06/2006, 17h17
xxiemeciel

Salut,

tu dois refaire tes boucles et faire un delete [] pour chaque new [].

pense a utiliser la balise [code /code] SVP

Eh bien, généralement en C++, on utilise des conteneurs de conteneurs de conteneurs, mais rien que pour s'amuser, on va essayer quand même:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 void FreeTripleArray(float ***data, size_t depth, size_t tailleTotaleX) { // i et j commencent a 1 for(i=1 ; i<=depth ; i++) { for(j=1 ; j<=tailleTotaleX ; j++) { delete[] data[i][j]; data[i][j] = NULL; //inutile puisqu'on va effacer, //mais plus clair. } delete[] data[i]; data[i] = NULL; //même remarque. } delete[] data; }

30/06/2006, 17h24
le mauvais

j'ai fais ca mais ca plante:

for(i=1;i<(depth+1);i++)
{
for(j=0;j<(deltaY+nb_zero);j++)
{
delete[] data[i][j];
}
delete[] data[i]
}
30/06/2006, 17h26
Médinoc

Citation:

Envoyé par le mauvais

// i et j commencent a 1

Résultat, ta boucle commençant à zéro fait un delete sur un pointeur non-initialisé...
30/06/2006, 17h29
le mauvais

j ai pris ton code mais la j ai l droit a un acces violation

Essaye ca alors, c'est exactement le code de Medinoc mais avec les indices de ton code a toi :

Code:

1
2
3
4
5
6
7
8
9
10
11
 
    for(i=1;i<=depth;i++)
    {
        for(j=1;j<=(deltaX + nb_zero );j++)
        {
            delete [] data[i][j];
 
        }
        delete [] data[i];
    }
    delete [] data;

Perso, je ferai un truc de ce style.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
long sizeData = depth+1;
long sizeIdata = deltaY+nb_zero +1;
for( long i1 =1; i1<sizeData; i1++)
{
	float ** tmpData = data[i1];
	for(long i2 =1; i2<sizeIdata; i2++)
	{
		float * tmpIdata = tmpData[i2];
		if(tmpIdata !=NULL) delete [] tmpIdata;
	}
}
 
	if(data !=NULL) delete [] data;

30/06/2006, 18h20
harsh

A quoi te sert de commencer a 1 :?

Du coup tu as des zones de ton tableau 3D (les indices 0) qui pointent vers... l'inconnu et au dela. Je suis pret a parier que ta violation intervient alors que tu essayes de manipuler ces zones ;)

Code:

1
2
3
4
        data[i] = new float*[deltaY+nb_zero +1 ];
        for(j=1;j<=(deltaX + nb_zero );j++)
        {
            data[i][j] = new float[2*(deltaX+nb_zero) +2 ];

Ne devrait-on pas avoir plutôt

Code:

for(j=1;j<=(deltaY + nb_zero );j++)

30/06/2006, 19h21
xxiemeciel
Citation:
Envoyé par diogene

Code:

1 2 3 4 data[i] = new float*[deltaY+nb_zero +1 ]; for(j=1;j<=(deltaX + nb_zero );j++) { data[i][j] = new float[2*(deltaX+nb_zero) +2 ];

Ne devrait-on pas avoir plutôt

Code:

for(j=1;j<=(deltaY + nb_zero );j++)

??
bien vu diogene, effectivement si il y a des erreurs dans les indices ca risque de poser probleme et sa pourrait expliquer le acces violation.
04/07/2006, 14h29
le mauvais

ta raison harsh, j'ai des problemes de memory leak, le probleme c'est que j'utilise une fonction de fft3D de Numerical Recipes et quelle commence a un indice 1 et non 0...