Discussion :

[k3ops]

Salut,
je souhaiterais savoir comment vous désallouez vos structures car je me suis aperçu d'une fuite mémoire dans mon programme :

Code :

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

1
2
3
4
5
 
IplImage* img       => cvReleaseImage(&img)
CvSeq* contour      => cvReleaseMemStorage(&(contour->storage))
IplConvKernel* rect => cvReleaseStructuringElement(&rect)
CvMat* mat          => cvReleaseMat(&mat);

Voila comment je les utilise est-ce que je me trompe ?
Quand faut-il utiliser cvFree ? à quoi sert-il ?

Vaut-il mieux utiliser chaque cvRelease spécifique (cvReleaseImage, cvReleasememStorage,...) où toujours utiliser : cvRelease((void**)pointeur) ???

Merci de vos conseils a+

[gassi64]

Le cvRealeaseImage est parfait...fais attention à bien tous les supprimer. Le reste je ne sais pas...

[Vinsss84]

Es tu bien sur que la fuite memoire viennent d'un élément d'openCv? (pas d'utilisation de tableaux ou d'objets non detruits par exemple)
Sinon le cvRelease dedié est en général bien suffisant

[k3ops]

non justement je ne suis pas sur du tout que ca vienne de OpenCV (j'utilise dans une classe des tableau à trois dimensions alors je me perd un peu pour libérer ensuite l'espace...) mais je voulais déjà savoir si du côté de OpenCV je pouvais être sûr de n'avoir rien raté (je pense que c'est le cas).
Je vais devoir éplucher les tableaux C++ alors
Vous avez une technique efficace pour libérer un tableau qui a été initialisé ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
double*** data;
data = new double**[nbData];
for(int i=0; i<nbData; i++) {
	data[i] = new double*[heightResult];
	for(int j=0; j<heightResult; j++) {
		data[i][j] = new double[widthResult];
		for(int k=0; k<widthResult; k++) {
			data[i][j][k] = val; 
		}
	}
}

Merci!!!

[Vinsss84]

delete data ;
delete [] data;
delete [][] data;

[k3ops]

merci de ta réponse en effet je ne faisait pas comme ca, ca vient surement de la...
par contre j'ai une erreur de compilation lorsque je fait :

Code :

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

delete[][] data;

cette notation existe t-elle ???

[k3ops]

j'ai finalement trouvé mieux, ca règle ce problème de fuite mémoire mais il m'en reste encore un peu (l'utilisation mémoire monte mais bien moins vite) il doit en rester quelque part, voici comment détruire un tableau à 3dimensions :

Code :

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

1
2
3
4
5
6
7
8
 
for(int k=0; k<dim1; k++) {
	for(int j=0; j<dim2; j++) {
		delete[] data[k][j];
	}
	delete[] data[k];
}
delete[] data;

et voila, merci des conseils.

[snoupi72]

Moi je sais que j'avais des fuites de mémoire du l'utilisation d'une séquence déclaré en global, je ne pouvais release mon MenStorage. Et j'ai donc du coup utilisé "cvClearSeq". Tu as peut le mettre même genre de fuite ...

[snoupi72]

j'ai finalement trouvé mieux, ca règle ce problème de fuite mémoire mais il m'en reste encore un peu (l'utilisation mémoire monte mais bien moins vite) il doit en rester quelque part, voici comment détruire un tableau à 3dimensions :

Code :

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

1
2
3
4
5
6
7
8
 
for(int k=0; k<dim1; k++) {
	for(int j=0; j<dim2; j++) {
		delete[] data[k][j];
	}
	delete[] data[k];
}
delete[] data;

et voila, merci des conseils.

Et pourquoi ne pas utilisé des vector ??? C'est plus simple d'utilisation.

[k3ops]

ben en fait je trouve ca plus compliqué par vecteurs pour accéder à une valeur précise, d'autant plus que un vecteur c'est bien pour ajouter des éléments mais pour y accéder le temps d'accès est décuplé (je perds plus de 5sec par exécution !!!)
Un vecteur à t-il besoin d'être détruit si c'est une vecteur d'entier par exemple ?

Code :

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

~vector()

[k3ops]

en fait j'avais fais une mauvaise manip et avec le vector c'est aussi rapide et je suis sur de bien le détruire avec ~vector() donc je vais garder cette solution, merci