Discussion :

[deathness]

Bonjour!

J'ai une erreur sur une méthode que je ne comprend pas trop.

La voici:
error C4172: retourne l'adresse d'une variable locale ou temporaire


Et la methode:

Code :

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

1
2
3
4
5
 
Coord& Table::pixelToTable (const Coord& c) const
{
    return Coord(1, 1);
}

Je cree un objet non? Donc je peux bien le retourner... Je ne comprends pas pourquoi le compilo me sort ça.

Que devrais-je faire? Merci d'avance!

[Invité]

Ton compilateur a très bien fait son boulot : l'objet Coord est bien créé, mais sur la pile...c'est donc effectivement un objet local à la fonction (il sera détruit à la sortie de la fonction).

Autrement dit tu vas le créer pour le détruire aussitôt.

Il faut que tu le créer sur le tas avec un "new" (attention, penser à libérer la mémoire après !), ou que tu le créés avant d'appeler ta fonction puis que tu le passes par référence.

[koala01]

Salut,

Ton compilateur a très bien fait son boulot : l'objet Coord est bien créé, mais sur la pile...c'est donc effectivement un objet local à la fonction (il sera détruit à la sortie de la fonction).

Autrement dit tu vas le créer pour le détruire aussitôt.

Jusque là, je suis d'accord

Il faut que tu le créer sur le tas avec un "new" (attention, penser à libérer la mémoire après !), ou que tu le créés avant d'appeler ta fonction puis que tu le passes par référence.

Malheureux, non, surtout pas...

Si on crées l'objet avec new, il faut le renvoyer par pointeur

En effet, si on a l'habitude de réfléchir à l'opportunité de libérer la mémoire pointée par un pointeur, il est beaucoup moins naturel d'y réfléchir lorsque l'on a affaire à un objet ou à une référence.

Il y a deux solutions:

• Soit, tu veux, effectivement, renvoyer ta coordonnée par référence, et il faut alors que la dite coordonnée soit membre de la classe Table, ce qui, je le crains, n'aurait sans doute pas énormément de sens.
• Soit, tu crées effectivement la coordonnée sur la pile, mais tu renvoie... une coordonnée par valeur et non par référence.

La solution dans ce cas étant, très vraisemblablement, de renvoyer ta coordonnée par valeur et non par référence

[Invité]

Effectivement, je me suis mal exprimé : je voulais bien dire créer l'objet et renvoyer le pointeur, OU le créer à l'extérieur de la fonction et travailler sur une référence.

[koala01]

Effectivement, je me suis mal exprimé : je voulais bien dire créer l'objet et renvoyer le pointeur, OU le créer à l'extérieur de la fonction et travailler sur une référence.

Et encore...

A vrai dire, le code dont on dispose m'émeut au plus haut point, car, tel que présenté ici, je ne vois vraiment pas l'intérêt :

On a une fonction membre qui s'engage à ne pas modifier l'objet au départ duquel elle est invoquée et qui reçoit... une coordonnée, qui ne peut pas être modifiée.

Cette fonction membre renvoie une nouvelle coordonnée définie en dur (Coord(1,1)).

Partant de là:

1. la nouvelle coordonnée doit donc bien être renvoyée... par valeur
2. à quoi ca sert de passer une coordonnée en paramètre si c'est... pour ne pas l'utiliser