Discussion :

[oodini]

Bonjour,

J'ai une méthode assez courte dans laquelle j'alloue dynamiquement de la mémoire pour le pointeur nommé dataArray :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
int geometry::updateDynamicData(int current_key)
{
	MMatrix *matricesArray = new MMatrix[mMotionKeys];
	AtArray *dataArray;
	dataArray = new AtArray;
	dataArray = nodeGetArray(handle, "matrix");
	if (dataArray->data != NULL)
		matricesArray = (MMatrix*)dataArray->data;
 
	int offset = current_key-1;
 
	AtMatrix tmp;
	MMatrix matrix(mDagPath.inclusiveMatrix());
	matrix.get(tmp);
	matricesArray[offset] = tmp;
 
	setMatrixArray("matrix", &matricesArray, 1, mMotionKeys, false);
 
	delete dataArray;
	delete [] matricesArray;
}

Quand je laisse le delete en fin de bloc, ça me fait planter mon prgramme.
Si je l'enlève, c'est OK, même si je suppose ça provoque quelques fuites de mémoire...

[Laurent Gomila]

Ton appel à new pour allouer de la mémoire ne sert strictement à rien, puisqu'à la ligne d'après tu réaffectes dataArray pour le faire pointer ailleurs.

J'imagine que si le delete plante c'est que nodeGetArray ne renvoie pas un objet qui a été alloué correctement avec new. On peut voir le code de cette fonction ?

[screetch]

Code :

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

1
2
3
4
 
	AtArray *dataArray;
	dataArray = new AtArray;
	dataArray = nodeGetArray(handle, "matrix");

je pense que tu n'as pas compris la fonctionnalité de new et delete.

Ici new sert a allouer un objet AtArray et place l'adresse ou il a ete créé dans la variable dataArray.
A la ligne du dessous, tu affectes dans la variable contenant l'adresse de ce nouvel objet l'adresse renvoyee par nodeGetArray(handle, "matrix");
cette fonction te renvoie ladresse d'un objet deja existant.

ensuite, delete va essayer de liberer l'objet qui est pointé par dataArray, qui n'est pas l'objet créé au debut puisque son adresse a ete remplacee par l'adresse d'un autre objet.

En gros supprime le new, tu cherches a obtenir l'adresse d'un objet deja existant. Et supprime delete, puisque tu n'as pas alloué toi meme cet objet.

[oodini]

J'ai supprimé le new et le delete pour dataArray, et cela fonctionne à merveille.
Je vous remercie donc pour vos explications.

En revanche, j'ai le problème équivalent pour matricesArray (dont j'avais oublié la déclaration, rajoutée en première ligne).

Là, si j'enlève le new, ça ne fonctionne pas.
Et si je rajoute le delete [], ça ne fonctionne pas non plus.

[screetch]

une possibilite pour laquelle ca ne marche pas sans le new est que tu affectes matricesArray seulement dans certaines conditions et pas dans d'autre.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
{
	MMatrix *matricesArray;
	AtArray *dataArray;
	// non! dataArray = new AtArray;
	dataArray = nodeGetArray(handle, "matrix");
	if (dataArray->data != NULL)
		matricesArray = (MMatrix*)dataArray->data;
	else
		matricesArray = 0; // c'est a toi de savoir ce que l'on fait si
			// il n y a pas de data mais tu ne peux pas laisser
			// matricesArray non initialise
	...
	// non! delete dataArray;
}

[bolhrak]

Là, si j'enlève le new, ça ne fonctionne pas.

Cad ? (quelle erreur est obtenue ?)

[oodini]

une possibilite pour laquelle ca ne marche pas sans le new est que tu affectes matricesArray seulement dans certaines conditions et pas dans d'autre.

Ben en fait, si le if échoue, je dois créer le tableau, et je devrais donc utiliser un new....

Ce que j'ai fait : j'ai déporté le new dans le else.

Ca fonctionne, mais quid de la désallocation de la mémoire ?

Si je fais un delete[] en fin de bloc, ça couine.

[oodini]

Cad ? (quelle erreur est obtenue ?)

Mon programme mouline dans le vide (freeze ?), quand je fais ça.

[oodini]

Bon, avec le if, j'ai rajouté un else utilisant un booléen dédié appelé initialisation :

Code :

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

1
2
3
4
5
6
7
8
9
	if (dataArray->data != NULL)
	{
		matricesArray = (MMatrix*)dataArray->data;
	}
	else
	{
		matricesArray = new MMatrix[mMotionKeys];
		initialisation = true;
	}

Ensuite, en fin de bloc, je mets cela :

Code :

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

1
2
	if (initialisation)
		delete [] matricesArray;

Ca marche, mais je ne sais pas si c'est ce qu'il a de mieux à faire... :-/

[befalimpertinent]

Code :

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

1
2
3
4
5
6
7
8
9
	if (dataArray->data != NULL)
	{
		matricesArray = (MMatrix*)dataArray->data;
	}
	else
	{
		matricesArray = new MMatrix[mMotionKeys];
		initialisation = true;
	}

Ensuite, en fin de bloc, je mets cela :

Code :

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

1
2
	if (initialisation)
		delete [] matricesArray;

Ca marche, mais je ne sais pas si c'est ce qu'il a de mieux à faire... :-/

C'est correct si l'allocation / désalocation de cet objet

Code :

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

1
2
 
(MMatrix*)dataArray->data;

est fait correctement. Montres nous ça dans ton code

[oodini]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
int Geometry::updateDynamicData(int current_key)
{
	MMatrix *matricesArray;
 
	bool initialisation = false;
 
	AtArray *dataArray;
	dataArray = AiNodeGetArray(handle, "matrix");
	if (dataArray->data != NULL)
	{
		matricesArray = (MMatrix*)dataArray->data;
	}
	else
	{
		matricesArray = new MMatrix[mMotionKeys];
		initialisation = true;
	}
 
	int offset = current_key-1;
 
	AtMatrix tmp;
	MMatrix matrix(mDagPath.inclusiveMatrix());
	matrix.get(tmp);
	matricesArray[offset] = tmp;
 
	setMatrixArray("matrix", &matricesArray, 1, mMotionKeys, false);
 
	if (initialisation)
		delete [] matricesArray;
 
	return true;
}

[screetch]

Quand tu passes par des pointeurs l'important est de savoir QUI obtient la "possession" (l'ownership, la proprieté) de ton pointeur, et qui va le liberer donc.

Si tu recuperes un pointeur de quelquun, il n'est pas a toi, tu ne dois pas le liberer, sauf si cette personne/objet/machin te TRANSMET la proprieté (c'est en general precisé en gros)

Dans ton code essaye d'etablir qui a la propriete des objets alloues avec new, c'est a eux de faire la desallocation avec delete.

A la rigueur si tu as du mal a voir, nomme uen variable MYptr, MYmatrix, et desalloue la a la fin.

Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
	AtArray *dataArray; // pas a moi
	MMatrix *matricesArray; // pas a moi a priori
	MMatrix *MYmatricesArray = 0; //A MOI je le desallouerai 
	dataArray = AiNodeGetArray(handle, "matrix");
	if (dataArray->data != NULL)
	{
		matricesArray = (MMatrix*)dataArray->data;
	}
	else
	{
		MYmatricesArray = new MMatrix[mMotionKeys]; // c'est mon mien!
		matricesArray = MYmatricesArray ; // c'est sur matricesArray  qu'on travaille, je le "prete"
		initialisation = true;
	}
 
 
...
 
delete [] MYmatricesArray; // c'etait le mien je le nettoie. Si je n'avais rien créé, delete [] 0 est parfaitement legal!

[oodini]

Salut,

Désolé pour le délai de réponse, j'avais dû passer sur autre chose.

En fait, pour la désallocation, je dois utiliser

Code :

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

1
2
if (initialisation == TRUE)
    delete [] MYmatricesArray;

Si j'omets la condition, ça plante.