[Débutant] erreur de l-value

Version imprimable

02/04/2006, 10h15
LaseLiep

[Débutant] erreur de l-value
Bonjour,

J'ai un petit problème avec comme message d'erreur : '&' nécessite une l-value

Explication du code :

J'ai une classe Point (dans un namespace dom).
Cette classe contient un x et un y.

J'ai maintenant une classe _non membre_ echange() qui doit échanger les valeurs de x et y.
Code:

1 2 3 4 5 void echange(int* x, int* y){ int temp = *x; *x = *y; *y = temp; } // end echange()
Dans mon main, lorsque que j'utilise ma fonction non membre echanger(), j'obtiens l'erreur !!!
Code:

1 2 point monPoint = point(); echange(&monPoint.get_x(),&monPoint.get_y()); // erreur: '&' nécessite une l-value
J'ai également essayer comme suit :
Code:

1 2 3 int* ptr_x = &monPoint.get_x(); // erreur: '&' nécessite une l-value int* ptr_y = &monPoint.get_y(); // erreur: '&' nécessite une l-value echange(ptr_x,ptr_y);
Mais je n'ai pas eu plus de chance ...

Pouvez-vous m'aidre?
02/04/2006, 10h32
reggae

Pourrais-tu nous montrer ta classe et le message d'erreur?
Les as-tu déclaré en public, private,etc?

Ma classe est la suivante :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
namespace dom {
	class point {
	private :
		int _x;
		int _y;
 
	public :
		void set_x(int);
		void set_y(int);
		int get_x();
		int get_y();
	}; // end class
} // end namespace

Ensuite j'ai déclaré mes fonctions membres et ma fonction non membre (qui est ni en privée, ni en publique, ni en statique car elle est non membre) et ensuite mon main() ou j'utilise mes fonctions ...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
int main(){
	using namespace dom;
	int* ptr_x;
	int* ptr_y;
 
	point monPoint = point();
	monPoint.set_x(2);
	monPoint.set_y(7);
	std::cout << "x=" << monPoint.get_x() << " , y=" << monPoint.get_y() << std::endl;
 
	ptr_x = &monPoint.get_x(); // erreur: '&' nécessite une l-value 
	ptr_y = &monPoint.get_y(); // erreur: '&' nécessite une l-value 
	echange(ptr_x,ptr_y);
	//echange(&monPoint.get_x(),&monPoint.get_y()); // ou alors erreur: '&' nécessite une l-value 
 
	std::cout << "x=" << monPoint.get_x() << " , y=" << monPoint.get_y() << std::endl;
	return 0;
} // end main()

Citation:
Envoyé par LaseLiep
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 int main(){ using namespace dom; int* ptr_x; int* ptr_y; point monPoint = point(); monPoint.set_x(2); monPoint.set_y(7); std::cout << "x=" << monPoint.get_x() << " , y=" << monPoint.get_y() << std::endl; ptr_x = &monPoint.get_x(); // erreur: '&' nécessite une l-value ptr_y = &monPoint.get_y(); // erreur: '&' nécessite une l-value echange(ptr_x,ptr_y); //echange(&monPoint.get_x(),&monPoint.get_y()); // ou alors erreur: '&' nécessite une l-value std::cout << "x=" << monPoint.get_x() << " , y=" << monPoint.get_y() << std::endl; return 0; } // end main()
get_x et y renvoient une "valeur" (i.e. une Constante). tu ne peux pas faire de référence vers des Constantes, mais uniquement vers des Variables (i.e. int Z est une variable) qu'elles soient constantes ou non.

pour faire ce que tu essaies là, il faudrait que get_x et y renvoie une référence vers les données de ton objet.

02/04/2006, 10h53
bigboomshakala
:roll: hmm comme c'est pas forcément évident à voir, un exemple

toi tu fais l'équivalent de

Code:

ptr = &36;

or une constante n'a pas d'adresse (il n'y a pas d'espace alloué)

pour y remédier
Code:

1 2 3 int & get() { return a; } ptr = &toto.get(); // équivaut à ptr = &toto.a; (quand a est public)
02/04/2006, 11h15
titoine1978

bigboomshakala :
Est-ce que ca change quelquechose si a est protected ou private ?
J'ai l'habitude de renvoyer des pointeurs sur des membres protected ou private, c'est mal ?
02/04/2006, 11h17
LaseLiep

J'te remercie, en effet ma fonction marche maintenant, mais je ne comprends po pourquoi.

Je n'ai jamais vu de fonction qui renvoyer Type & fonction(). Cela signifie que si par exemple je veux que ma fonction retourne une adresse sur un int, je lui fait retourner juste son int, mais qu'ensuite dans le type de retour, je lui demande l'adresse de ce int? Et c'est en mettant int& que je peux avoir cela ???

Et dans ce bout de code :

Code:

ptr = &toto.get();

Mon get() me renvoie déjà une adresse, donc pourquoi si j'ai l'adresse de mon x (appartenant à l'objet toto) je redemande encore une adresse ???

Merci pour ton aide.
02/04/2006, 11h19
bigboomshakala

Citation:

Envoyé par titoine1978

bigboomshakala :
Est-ce que ca change quelquechose si a est protected ou private ?

non

Citation:

Envoyé par titoine1978

J'ai l'habitude de renvoyer des pointeurs sur des membres protected ou private, c'est mal ?

oui.
car le principe de la POO, c'est l'encapsulation. en renvoyant un pointeur vers une donnée membre protégée ou privée, on viole ce principe.
et si vraiment il est nécessaire de pouvoir accéder directement à une donnée membre, alors autant la mettre en public (et donc pas besoin de méthodes pour y accéder).
02/04/2006, 11h21
titoine1978

Merci pour la précision. Je vais changer cette mauvaise habitude...
02/04/2006, 11h21
bigboomshakala

Citation:

Envoyé par LaseLiep

J'te remercie, en effet ma fonction marche maintenant, mais je ne comprends po pourquoi.

Je n'ai jamais vu de fonction qui renvoyer Type & fonction(). Cela signifie que si par exemple je veux que ma fonction retourne une adresse sur un int, je lui fait retourner juste son int, mais qu'ensuite dans le type de retour, je lui demande l'adresse de ce int? Et c'est en mettant int& que je peux avoir cela ???

Et dans ce bout de code :

Code:

ptr = &toto.get();

Mon get() me renvoie déjà une adresse, donc pourquoi si j'ai l'adresse de mon x (appartenant à l'objet toto) je redemande encore une adresse ???

Merci pour ton aide.

type & toto = var; (référence sur une variable, alias)
type * ptoto = &var; (pointeur)
:arrow: pas la même signification (voir FAQ : références)
02/04/2006, 11h36
LaseLiep

J'te remercie bigboomshakala.
J'avoue qu'après avoir fait du C, c'est vraiment pas facile de reconnaître une utilité à la référence. Quand faut-il l'utiliser, quand il ne faut po, quand utiliser les pointeurs ... C'est assez vague dans ma tête pour le moment ;)
02/04/2006, 11h44
bigboomshakala
:D

en C on n'a pas le choix, il n'y a que les pointeurs.
en C++, on privilégie généralement les références (pour faire simple, c'est un pointeur déguisé que l'utilisateur n'a pas a géré). c'est plus sûr, et l'écriture est plus naturel puisqu'on manipule la référence comme la variable elle-même (pas de -> ou d'*). c'est vrai qu'au départ c'est une notion qui peut-être difficile à intégrer.

la référence est avantageuse aussi en cela qu'elle permet d'éviter la duplication d'objet. si on a besoin de passer en argument un objet à une fonction et qu'on sait qu'il ne sera pas modifié, on peut passer une référence constante, plutôt que de recopier l'objet
Code:

1 2 3 4 void func (const type obj) // recopie { ... // obj pas modifié }
Code:

1 2 3 4 5 // référence, pas de recopie de l'objet passé en argument void func (const type & obj) { ... // obj pas modifié }
note : désolé pour le "const type &" :lol:
02/04/2006, 11h58
LaseLiep

Ok, faut que ça fasse son petit bonhomme de chemin dans ma tête, et que je pratique, ça viendra peut être plus facilement après (je l'espère tout du moins).

J'te remercie pour le temps que tu as passé sur mon problème et pour tes explications.
02/04/2006, 12h10
reggae

Désolé pour le retard :?
Si tu as un problème, consulte la FAQ:
http://c.developpez.com/faq/cpp/?page=references
voilavoila