Discussion :

[babar63]

Bonjour à tous

Voila ma situation je possède une classe outilDessin classe abstraite et une classe dérivé classe Point(d'autre à venir), je possede également une liste chainé permettant de gérer les pointeurs sur outilDessin.
Enfin je possede une classe composite, elle contient une liste d'objet, et permet de gérer tous ces objets (sauvegarde,restauration,ajout d'objet....)
Maintenant mon probleme vient de la restauration d'une scene à partir d'un fichier texte
voila un bout du code de la restauration :

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
int taille;
entree>>taille;
 
//je recupere les objets :
 
int type;
for (int i=1 ; i<=taille ; i++)
{
	entree>>type; //je recupere le type de mon objet (0 pour le point)
	switch(type)
	{
		case 0 :
		{
			Point * P = new Point;
			P->ecritureEntree(entree); //fonction retournant dans P le
                                                            //point récupéré du fichier
		 	addObject(*P);
			delete P;
		}
 
                ......
	}
}

Voila j'ai déjà tenter beaucoup de variantes mais le problème récurrent vient du fait que la mémoire de l'objet P est désallouée donc je ne plus accéder aux valeur du point dans ma liste chainée par la suite !!!

je vous remercie d'avance ; )
@bientot

[Laurent Gomila]

Il faut que tu stockes des pointeurs dans ta liste ; par valeur comme tu le fais actuellement cela ne va garder que la partie outilDessin de chaque instance, et virer toute donnée spécifique à la classe dérivée.

[zais_ethael]

Pourquoi fais tu un delete sur P? A partir du moment où tu l'as transféré dans ta classe conteneur, son cycle de vie est supposé être correctement géré, alors pourquoi le détruire? Surtout qu'à moins que ton objet ne soit clonable, ta classe conteneur n'a aucun moyen de le dupliquer.

Edit: ha oui, j'avais pas vu la copie par valeur

[babar63]

merci a vous, cependant ma liste chainée contient des pointeurs de la classe outilDessin et je possède également une focntion qui permet de cloner mon objet point mais j'ai déja tenter de le cloner :

Code :

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

1
2
3
4
 
Point  P;
P.ecritureEntree(entree);
addObject(P.getCopy());

mais toujours aucun résultat, la copie est perdu et je n'ai plus d'acces aux élements de ma liste : (

[Laurent Gomila]

ma liste chainée contient des pointeurs

Ton code semble affirmer le contraire.

[babar63]

En effet je passe l'objet lui meme en paramètre mais je recupere son adresse de toute facon la classe outilDessin étant abstraite je ne peux que travailler avec des pointeurs sur ma liste chainée.
Le probleme vient du fait que la mémoire attribuée à l'objet instancié (d'une classe fille) et désalloué lorsque je sors de ma fonction et donc le programme plante si j'essai d'accéder à ses données qui ont était perdu.
J'ai pensé à créer une classe chargement qui permetrait de sauvegarder ces objets en mémoire mais j'esperais trouver un moyen plus "propre" ou efficace si possible.
Merci pour vos réponses si quelqu'un trouve une solution je suis tout ouïe ; )
@bientot

[Laurent Gomila]

On peut voir la fonction addObject ?