Discussion :

[damien77]

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

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

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

[David Fleury]

Salut,

getX() et getY() ne sont pas const

Code :

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

1
2
3
 
        int getX() const;
        int getY() const;

Le constructeur de copie n'a pas de raison d'être autre chose que

Code :

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

1
2
 
Point(const Point&);

et quelle étrange idée d'utiliser des new.... et ainsi introduire des bugs.

[damien77]

pourquoi ne faut-il pas utiliser de new ?

[Ulmo]

Parce tu sais à l'avance de combien tu en as besoin (un seul).
Donc plutôt déclarer ainsi :

Code :

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

1
2
3
4
5
6
7
class Point {
     public:
[...]
    private :
        int x;
        int y;
};

D'autre part, le

Code :

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

delete x, y;

ne fait probablement pas ce que tu crois...

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

[Ulmo]

Code :

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

delete x, delete y;

et

Code :

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

delete x, y;

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

Code :

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

delete (x, y);

et (x, y) retourne juste y. Donc seul y sera libéré.