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

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

[bolhrak]

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

Cad ? (quelle erreur est obtenue ?)

[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... :-/