error: no match for 'operator='

Version imprimable

05/04/2011, 20h54
MastahPommeh

error: no match for 'operator='
Bonjour,
voici l'erreur exacte :

Code:

error: no match for 'operator=' in 'l2 = ListeObjet::operator+(ListeObjet&)(((ListeObjet&)(& l2)))

dans mon main j'ai trois ListeObjet l1 et l2
et je fais

Code:

l2=l1+l2

la surcharge de l'opérateur= fonctionne très bien.

pour la surcharge de l'opérateur+ c'est une autre histoire
Code:

1 2 3 4 5 6 7 8 ListeObjet ListeObjet::operator+(ListeObjet &l) { ListeObjet l3; l3=*this; l3+=l; std::cout<<l3; return l3; }
le soucis semble être le type de retour.
Quand je retourne une référence l'erreur disparait mais la liste retourné est vide alors qu'elle est bien construite, ce que je vérifie en l'affichant avant de la retourner.
Si je retourne un pointeur ça marche parfaitement bien
En revanche si je retourne un ListeObjet j'ai cette erreur. et je n'arrive pas a comprendre ce qui ne va pas
06/04/2011, 14h11
r0d

Bonjour,

Citation:

Envoyé par MastahPommeh

le soucis semble être le type de retour.

Non le soucis, c'est ce que te dis explicitement le compilateur: il n'y a pas d'opérateur d'affectation ( operator = ) pour ta classe ListeObjet.

Citation:

Envoyé par MastahPommeh

Quand je retourne une référence l'erreur disparait mais la liste retourné est vide alors qu'elle est bien construite, ce que je vérifie en l'affichant avant de la retourner.
Si je retourne un pointeur ça marche parfaitement bien

Tu ne nous dis pas tout. Car selon le code que tu nous montre, si tu retourne l3, par pointeur ou par référence, ça ne peut pas fonctionner. Car l3 est construit sur la pile et sera donc détruit à la sortie de l'operateur +. Donc le pointeur ou la référence ne seront plus valides.
06/04/2011, 14h43
MastahPommeh
si j'ai bien un operator= dans ma classe je l'ai testé de toutes les façon possible il marche parfaitement.

Par référence, en effet l'objet est détruit a la sortie de la fonction en revanche si l3 est un pointeur comme ceci :
Code:

1 2 3 4 5 6 7 8 ListeObjet* ListeObjet::operator+(ListeObjet &l) { ListeObjet* l3=new ListeObjet(); *l3=*this; *l3+=l; return l3; }
c'est le pointeur l3 qui est détruit et non pas l'objet pointé
06/04/2011, 15h45
r0d

Ah d'accord, si tu fais un new oui ça va marcher. Mais c'est très dangereux, car il faudra ensuite le détruire.

L'erreur dont tu parles dans le premier message, elle s'applique à quelle ligne?
Peux-tu nous montrer le code de l'opérateur d'affectation et du constructeur par copie?

Et puis une remarque, la signature de ton operateur + n'est pas bonne. Ça doit être:

Code:

ListeObjet ListeObjet::operator+(const ListeObjet &l)

alors

dans mon header j'ai :

Code:

1
2
ListeObjet & operator=(ListeObjet &);
ListeObjet operator+(const ListeObjet &); // j'ai rajouter le const comme tu me l'as fait remarqué

ensuite mon constructeur par copie :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
ListeObjet::ListeObjet(const ListeObjet &l){
	CelluleObjet* temp=l.debut;
	debut=new CelluleObjet(*temp);
	courant=0;
	temp=temp->getSuivant();
	avancer();
	while(temp->getSuivant()!=0)
	{
		courant->setSuivant(new CelluleObjet(*temp));
		temp=temp->getSuivant();
		avancer();
	}
	courant->setSuivant(new CelluleObjet(*temp));
}

mon opérateur d'affectation :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
ListeObjet & ListeObjet::operator=(ListeObjet &l)
{
	this->allerDebut();
	while(!estVide())
	{
		this->supprimer();
	}
	l.allerDebut();
	Objet p;
	while(!l.estFin())
	{
	    l.avancer();
		p=l.courant->getObjet();
		inserer(p);
	}
	return *this;
}

et enfin mon main :

Code:

1
2
3
4
5
6
7
 ListeObjet l=ListeObjet();
    ListeObjet l2=ListeObjet();
    Objet o=Objet();
    l.inserer(o);
    l.inserer(o);
    l2=l+l;                                    // c'est la ligne qui pose soucis
    cout<<l2;

06/04/2011, 16h25
r0d

Ton opérateur d'affectation n'est pas bon non plus. Ça doit être:

Code:

ListeObjet & operator=(const ListeObjet &);

De plus, ça c'est pas bon:

Code:

ListeObjet l=ListeObjet();

il faut faire:

Code:

ListeObjet l; // tout court

Une fois modifié, si ça ne marche toujours pas, donnes-nous le message d'erreur et à quelle ligne il s'applique.
06/04/2011, 17h22
MastahPommeh

c'était les const qui n'allait pas, merci beaucoup maintenant je dois tout modifier xD