surcharge opérateur erreur compilation

Version imprimable

Bonjour j'aimerais faire la surcharge des opérateur += etc... et j'ai un problème.
Voilà mon code :

Code:

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
 
#ifndef POINT_H
    #define POINT_H
 
    #ifndef POINT_DEBUG
        #define POINT_DEBUG 1
    #endif
 
#include <iostream>
using namespace std ;
 
class Point {
 
    public:
 
        // CONSTRUSTREUR
        Point();
        Point(Point&);
        Point(int,int);
 
         // DESTRUSTREUR
        ~Point();
 
        Point &operator+=(const Point&);
 
        void setX(int);
        void setY(int);
 
        int getX();
        int getY();
 
    private :
        int *x;
        int *y;
};
 
#endif

Code:

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
#include "point.h"
 
Point::Point() {
    x = new int; y = new int;
    *x = 0 ; *y = 0 ;
    if( POINT_DEBUG )
        cout << "Appel du constructeur Point(). x=" << *x << " y=" << *y
             << " &x=" << x << " &y=" << y << endl ;
}
 
Point::Point(int a, int b) {
    x = new int; y = new int ;
    *x = a ; *y = b;
    if( POINT_DEBUG )
        cout << "Appel du constructeur Point(int,int). x=" << *x << " y=" << *y
             << " &x= " << x << " &y=" << y << endl ;
}
 
Point::Point(Point &copy) {
    x = new int; y = new int ;
    *x = copy.getX(); ; *y = copy.getY() ;
    if( POINT_DEBUG )
        cout << "Appel du constructeur Point(Point&). x=" << *x << " y=" << *y
             << " &x= " << x << " &y=" << y << endl ;
}
 
Point::~Point() {
    if( POINT_DEBUG )
        cout << "Appel du destructeur ~Point()" << endl ;
    delete x,y ;
}
 
void Point::setX(int a) {
    *x = a;
    if( POINT_DEBUG )
        cout << "Appel de la fonction setX(int). x=" << *x << endl ;
}
 
void Point::setY(int b) {
    *y = b;
    if( POINT_DEBUG )
        cout << "Appel de la fonction setX(int). y=" << *y << endl ;
}
 
Point& Point::operator+=(const Point &point) {
    *x = point.getX();
    *y = point.getY();
    return *this;
}
 
int Point::getX() {
        return *x;
}
 
int Point::getY() {
        return *y;
}

Quand je compile, j'ai cette erreur :
point.cpp:46: error: passing `const Point' as `this' argument of `int Point::getX()' discards qualifiers
point.cpp:47: error: passing `const Point' as `this' argument of `int Point::getY()' discards qualifiers

merci de votre aide

21/02/2007, 12h19
David Fleury
Salut,

getX() et getY() ne sont pas const
Code:

1 2 3 int getX() const; int getY() const;
Le constructeur de copie n'a pas de raison d'être autre chose que
Code:

1 2 Point(const Point&);
et quelle étrange idée d'utiliser des new.... et ainsi introduire des bugs.
21/02/2007, 12h20
damien77

pourquoi ne faut-il pas utiliser de new ?
21/02/2007, 12h26
Ulmo
Parce tu sais à l'avance de combien tu en as besoin (un seul).
Donc plutôt déclarer ainsi :
Code:

1 2 3 4 5 6 7 class Point { public: [...] private : int x; int y; };
D'autre part, le

Code:

delete x, y;

ne fait probablement pas ce que tu crois...
21/02/2007, 14h45
damien77

new c'est bien pour réserver un espace mémoire,
et delete pour lui dire de viré de la mémoire les variables ?
21/02/2007, 14h53
Ulmo

Code:

delete x, delete y;

et

Code:

delete x, y;

sont très différents. Le second est parenthesé ainsi :

Code:

delete (x, y);

et (x, y) retourne juste y. Donc seul y sera libéré.
21/02/2007, 17h43
damien77

D'accord merci.
Donc new et delete servent quand nous ne savons pas combien d'instance vont exister en mémoire ?
Sinon, ca ne sert à rien..
21/02/2007, 17h57
Laurent Gomila

Ca sert aussi si on veut allouer une variable qui doit continuer d'exister après la portée courante.

Mais ne t'embête pas trop avec ça pour l'instant, tu n'as absolument pas besoin d'allocations dynamiques pour ta classe Point.
21/02/2007, 17h59
damien77

Ok merci...