Discussion :

[meryeminfo]

salut tout le monde
je suis débutante en c++;je compile sous devc++
j'ai un problème dans ce programme quand je fais la valeur de retour de la fonction "sym" par reference et je ne sais pas d'pi l'erreur aidez moi s'il vous plais

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 
#include<iostream>
using namespace std;
class point
{
       int x,y;
       public:
              point(int a=0,int b=0)
              {
                        x=a;
                        y=b;
                        cout<<"la construction de l'objet "<<x<<" "<<y<<endl;
              }
              void affiche()
              {
                   cout<<"l'objet courant est "<<x<<" "<<y<<endl;
              } 
              point& sym()
              {
                  point res;
                  res.x=-x;
                  res.y=-y;
                  return res;
              }
              ~point()
              {
                     cout<<"la destruction de l'objet "<<x<<" "<<y<<endl;
              }
};
int main()
{
    point no(9,9),lo;
    lo=no.sym();
    lo.affiche();
    system("PAUSE");
    return 0;
}

et merci d'avance

[Lucien63]

Tu retournes une reference sur un objet local donc un objet qui n'existe plus à la sortie de ta fonction.

[ram-0000]

En fait, la question est que doit faire la fonction sym()

En regardant ton code, la réponse est : elle retourne un objet symétrique de l'instance de la classe courante.

donc la fonction est :

Code :

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

1
2
3
4
5
6
7
              point sym()
              {
                  point res;
                  res.x=-x;
                  res.y=-y;
                  return res;
              }

Il faudrait jouer aussi avec les const qualifier.

comme sym() ne modifie pas la classe courante, il peut devenir :

Code :

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

              point& sym() const

et c'est pareil pour affiche(). Avec les const, le compilateur peut faire plus de controle et t'aider à détecter plus vite les bugs

PS : j'aime pas l'écriture fonction(), je préfère fonction(void) mais il parait que c'est pareil ...

NOTA: Cette intervention a dénéré une digression sur l'utilité d'expliciter le type void comme parametre de fonction.

Vous pourrez suivre cette digression ==>ici<==.

[meryeminfo]

merci pour votre réponse
mais je n'ai pas encore bien saisi le problème s'il vous plais Lucien63 vous pouvez me détailler un peu ce que vous avez dit:

Tu retournes une reference sur un objet local donc un objet qui n'existe plus à la sortie de ta fonction.

merci ram-0000 pour l'information
merci encore une fois

[Lucien63]

Code :

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

1
2
3
4
5
6
7
              point& sym()
              {
                  point res;  // l'objet est créé ici.
                  res.x=-x;
                  res.y=-y;
                  return res;
              }   // ici il est détruit.

Comme ta reference fait reference à l'objet local res de la fonction sym et que cette objet est automatiquement détruit à la sortie de la fonction. la reference fourni en retour, reference un objet qui à été détruit qui par consequent n'existe plus.

[Goten]

merci pour votre réponse
mais je n'ai pas encore bien saisi le problème s'il vous plais Lucien63 vous pouvez me détailler un peu ce que vous avez dit:

Code :

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

1
2
 
Tu retournes une reference sur un objet local donc un objet qui n'existe plus à la sortie de ta fonction.

merci ram-0000 pour l'information
merci encore une fois

La référence c'est un alias sur une variable. D'autre part quand tu crées une variable, à la sortie du scope où a été crée la variable alors elle est détruite. Ici la portée c'est la fonction donc tu crées la variable et elle est détruite a la sortie de la dite fonction donc tu peux pas la retourner..

edit : grillée...

[meryeminfo]

merci beaucoup
si je ne fais pas un retour par reference l’objet local res de la fonction sym() est détruit AVANT que la transmission de valeur ait été faite.Ainsi, la libération de place mémoire a lieu trop tôt

[ram-0000]

Exemple :

Code :

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

1
2
3
point a(10, 12);
 
point p = a.sym();

l'objet point retourné par la méthode sym() aura une durée de vie suffisante pour permettre l'affectation dans p. En fait, on pourrait dire que l'objet retourné par la fonction sym() est effacé après le ';'

Au fait, une classe qui s'appelle point, cela doit exister en vrai (donc pas utile de la réinventer) ou alors si c'est pour apprendre/exercice, lui donner un autre nom (MaClassePoint) permettrait d'éviter une collision de nom.