Discussion :

[LaseLiep]

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 :

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

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 :

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

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 :

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

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?

[reggae]

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

[LaseLiep]

Ma classe est la suivante :

Code :

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

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 :

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
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()

[bigboomshakala]

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

[bigboomshakala]

hmm comme c'est pas forcément évident à voir, un exemple

toi tu fais l'équivalent de

Code :

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

ptr = &36;

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

pour y remédier

Code :

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

1
2
3
int & get() { return a; }
 
ptr = &toto.get(); // équivaut à ptr = &toto.a; (quand a est public)

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

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

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

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.

[bigboomshakala]

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

non

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

[titoine1978]

Merci pour la précision. Je vais changer cette mauvaise habitude...

[bigboomshakala]

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 :

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

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)
pas la même signification (voir FAQ : références)

[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

[bigboomshakala]

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 :

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

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

Code :

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

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 &"

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

[reggae]

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