Discussion :

[vincho]

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
 
#include <iostream>
using namespace std;
 
class Point
{
protected:
   int taille;
  int *dat;
public :
  Point(){}
 
  Point(int n)
    {
     taille = n;
      dat =new int[n];
    }
  int size(){return this->taille;}
  inline int & operator []( int i){return this->dat[i];}
  friend Point operator +(const  Point&  ,const  Point&  );
};
 
inline Point  operator+(const Point&  x1,const Point&  x2)
{
  Point x(x1.size());
  for(int i=0;i<x1.size();i++)
  {
    x[i]= x1[i]+x2[i];
  }
  return x;
}
 
int main()
{}

La j'ai un message d'erreur

_rm3.cpp: In function «Point operator+(const Point&, const Point&)»:
_rm3.cpp:24: erreur: passing «const Point» as «this» argument of «int Point::size()» discards qualifiers

Quelqu'un peut m'aider à résoudre ce problème???
(hormis la solution d'enlever le const)

[David Fleury]

Code :

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

1
2
 
int size() const {return this->taille;}

probablement.

peut être qu'implémenter operator +, par += serait le bienvenue.
ne devrait être amie de Point (pas besoin)
attention au problème de new dans le constructeur sans operateur = et de constructeur de copie.
(sans compter la fuite à cause du destructeur manquant)

[vincho]

Merci ca à l'air de marcher.

J'ai contruit cette classe point pour illustrer mon probleme donc sans les précautions que tu mentionnes.

[vincho]

En revanche si je remplace le type int * par un vector<int> alors ca plante de nouveau:

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
 
#include <iostream>
#include <vector>
using namespace std;
 
class Point
{
protected:
  vector<int> dat;
 
public :
  Point(){}
 
  Point(int n)
    {
     dat.resize(n);
    }
  int size()const {return (this->dat).size();}
  inline int & operator []( int i)const {return (this->dat)[i];}
};
 
inline Point  operator+(const Point&  x1,const Point&  x2)
{
  Point x(x1.size());
  for(int i=0;i<x1.size();i++)
  {
    x[i]= x1[i]+x2[i];
  }
  return x;
}
 
int main()
{
}

Code :

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

1
2
3
 
_rm3.cpp: In member function «int& Point::operator[](int) const»:
_rm3.cpp:20: erreur: invalid initialisation of reference of type «int&» from expression of type «const int»

[zais_ethael]

Normal:

Code :

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

inline int & operator []( int i)const {return (this->dat)[i];}

Dans cette expression, ton vector est constant, et l'opérateur [] d'un vector constant renvoie une réfèrence constante. Il serait préfèrable de déclarer cet opérateur en double:

Code :

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

1
2
inline int & operator []( int i){return (this->dat)[i];}
inline const int & operator []( int i)const {return (this->dat)[i];}

[vincho]

Merci amigo,
Ca marche!!!!!!

A retenir l'pérateur [] d'un vecteur constant renvoie une référence constante.

[Médinoc]

Une référence constante ou une valeur ?

[Loack-]

Bonjour,

j'ai jamais eu ce problème car je met toujours en const des fonctions qui ne modifient pas les variables de ma classe, mais j'ai une question sur:

Code :

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

1
2
_rm3.cpp: In function «Point operator+(const Point&, const Point&)»:
_rm3.cpp:24: erreur: passing «const Point» as «this» argument of «int Point::size()» discards qualifiers

Pourquoi si on ne met pas de const ça ne va pas ?

Edit:
Après reflexion, les instances déclarées comme const, elles ne peuvent appeler que des fonctions membres const, c'est ça ?

[Laurent Gomila]

C'est ça. C'est tout l'intérêt d'avoir des fonctions membre const.

[zais_ethael]

Une référence constante ou une valeur ?

Normalement une réfèrence constante. Sinon la double déclaration que j'ai mis afficherait au moins un warning (retour de réfèrence sur une variable locale/temporaire).