Discussion :

[amateurc]

Bonjour, pour m'entrainer avec les pointeurs et classes j'ai fait une classe liste, tout fonctionne très bien jusqu'à la destruction de l'objet liste qui contient un pointeur vers la tête de ma liste. Ce qui me conduis à penser que mon constructeur par recopie et la surcharge de l'opérateur = sont fausse.
voici ce que j'ai fais : le problème est que ca ne marche trop (la surcharge de l'opérateur + contient la même opération, au début je voulais utiliser memcpy mais je ne sais pas trop comment fonctionne la fonction avec ma liste.)

Code :

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

1
2
3
4
5
6
7
8
 
Liste::Liste (const Liste &l)
{
//avec memcpy : memcpy(this->tete, l.tete, sizeof(Liste))
//avec un parcours manuel, la classe IteratorR permet le déplacement dans la liste chaque élément de la liste l est copié dans la liste de l'objet pointé par this
	for(IteratorR i(l); i.hasNext(); i.getNext())
		this->AjoutEnTete(i.getCurrent());
}

Voici le contenu de la messagebox que m'affiche visual C++

Exception non gérée à 0x00413fa6 dans Liste.exe*: 0xC0000005: Violation d'accès lors de la lecture de l'emplacement 0xcccccccc.

[LittleWhite]

Bonjour,

Je crois que c'est un problème d'allocation mémoire.

Ou alors, je viens peut être de tilter ... comme vous êtes dans le constructeur ( par recopie certes ) de la Liste, celle ci ne peux pas faire d'appel à this ( car l'objet n'est pas encore entièrement construit ).

Donc va falloir se pencher sur le memcpy() je crois

[screetch]

ca ressemble a quoi le destructeur de ta liste ?

[JolyLoic]

Déjà, oublie memcpy, c'est une mauvaise idée pour copier des objets.

Ensuite, tu ne nous en donne pas assez pour comprendre où est l'erreur. Que fait AjouteEnTete, que fait GetCurrent ? Est-ce que ces fonctions travaillent avec les données contenues dans la liste, ou bien avec les pointeurs qui te permettent d'en assurer l'infrastructure ?

[amateurc]

le destructeur est le suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
Liste::~Liste()
{
	IteratorRW it (*this);
 
	while (it.hasNext())
	{
		it.suppr();
	}
}

ici mon iteratorrw permet de parcourir la liste, la méthode suppr supprime l'élément courant.

getCurrent() retourne l'entier contenue dans le maillon de la liste, AjoutEnTete comme son nom le dit ajoute en tete de liste une valeur de type int

[screetch]

est ce que suppr appelle delete sur les objets pointés ?

[amateurc]

oui dans suppr j'appel delete