redefinition operateur

Version imprimable

07/05/2006, 14h00
crischprolch

redefinition operateur

bonjour,
je fais une classe vecteur et je dois redefinir des opérateurs. Jusque là pas de probleme mais on me demande de redefinir l'operateur * par une fonction donc je fais la déclaration dans le point h suivante : Vecteur& operateur* (const Vecteur&, const Vecteur&); Mais il n'aime pas du tout cette déclaration. Pourquoi ? telle est la question...

merci par avance.
07/05/2006, 14h05
Pragmateek

L'opérateur doit prendre un paramètre je pense, le second étant l'objet "this".
07/05/2006, 14h27
Swoög

Citation:

Envoyé par seriousme

L'opérateur doit prendre un paramètre je pense, le second étant l'objet "this".

Il y a deux façons de redéfinir un opérateur (binaire ou unaire) :
par une fonction membre, dans ce cas, l'instance en cours (*this) est une des opérande (la seule opérande pour un opérateur unaire, l'opérande gauche pour un opérateur binaire) et l'autre opérande et le paramètre de la fonction.

une fonction externe à la fonction (le plus souvent amie), dans ce cas, le premier paramètre est l'opérande gauche, et le second l'opérande droite (dans le cas d'un opérateur binaire bien sûr)

ensuite, on pourrait certainement t'aider plus si on avait le message d'erreur...
07/05/2006, 15h20
b Oo

Salut,
je dirais que l'erreur vient peut-etre du fait que tu renvoies une référence sur un objet qui est détruit à la fin de ta fonction d'où un message d'erreur.

Bonne journée.

b Oo
07/05/2006, 15h22
Swoög

Citation:

Envoyé par b Oo

Salut,
je dirais que l'erreur vient peut-etre du fait que tu renvoies une référence sur un objet qui est détruit à la fin de ta fonction d'où un message d'erreur.

Bonne journée.

b Oo

exacte, voilà un point qu'on avait pas remarqué...

En fait, sans autre informations que "il aime pas" ni le contexte on peut te trouver toutes les erreurs du monde...
07/05/2006, 17h06
Lucien63
Code:

1 2 Vecteur operateur* (const Vecteur&);
Déclaré comme ceci serait peut-être suffisant.
07/05/2006, 17h15
Laurent Gomila

Ca n'a rien à voir avec le problème, mais si cette surcharge désigne le produit scalaire ou vectoriel alors c'est une très mauvaise idée. Pour ça il vaudra toujours mieux utiliser des fonctions classiques.
07/05/2006, 18h37
Matthieu Brucher
Citation:
Envoyé par Lucien63

Code:

1 2 Vecteur operateur* (const Vecteur&);

Déclaré comme ceci serait peut-être suffisant.
Il doit d'abord nous indiquer :
- où est le prototype de cette fonction, dan une classe ou pas
- le vrai message d'erreur du compilateur.
08/05/2006, 02h02
loufoque

C'est operator, pas operateur.
08/05/2006, 17h00
crischprolch

merci pour ces commentaires,
j'ai essayé de modifier Vecteur& en Vecteur mais ça ne change rien. Il m'est demandé de pouvoir calculer le produit scalaire de deux vecteurs en surchargeant l'operateur * et cette surcharge doit être effectuée par l'intermédiaire d'une fonction et non d'une méthode de la clase Vecteur. Avec ce sujet, j'en déduis que la déclaration de cette fonction doit être faite comme ceci (dans le .h) :

class Vecteur{

...

};

Vecteur operator* (const Vecteur&, const Vecteur&);

et l'implémentation doit être faite dans le .cpp.
Qu'en pensez-vous?
08/05/2006, 17h12
crischprolch

voici les erreurs a la compilation (j'utilise Dev-C++) :

expected init-declarator before '*' token

expected `,' or `;' before '*' token

Ces erreurs font référence à la déclaration de la fonction (ds le .h)

merci d'avance
08/05/2006, 17h16
Swoög

dans le code que tu donnes (:tagcode:) la syntaxe est correcte, ensuite quand tu dis, qu'il aime pas, il serait bon d'avoir l'erreur sans ça on ne POURRA PAS t'aider !
08/05/2006, 17h26
Laurent Gomila

Ta déclaration est correcte. Juste pour être sûr, tu la places bien après la déclaration de la classe ?

ourVoici le .h et la déclaration est bien en dehors de la classe

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
 
class Vecteur{
      protected :
      //attributs
      int dimension;
      float* coord;
 
      static const float val_defaut = 0.0;
      static const int dim_defaut = 3;
 
      public :
      //constructeurs
      Vecteur(int, float);
      Vecteur(int);
      Vecteur();
 
      //destructeur
      ~Vecteur();
 
      //methodes
      float get(int) const ;
 
      float set(int,float);
 
      int dimensions() const;
 
      //affecte le contenu de v dans this
      Vecteur& operator= (const Vecteur&);
 
      Vecteur& operator+ (const Vecteur&);
 
};
      void afficherVecteur(const Vecteur*);
 
      //surcharge de l'opérateur *
      Vecteur operator* (const Vecteur&, const Vecteur&);
 
#endif

08/05/2006, 17h48
Swoög
plutôt que d'utiliser ça :
Code:

1 2 int dimension; float* coord;
tu ferais mieux d'avoir recours à un vector<float> de la STL... tuto sur la STL : http://www.developpez.biz/download/stl.pdf
08/05/2006, 17h58
crischprolch

en fait c'est un tp donc j'ai des étapes à respecter et je ne peux pas utiliser la stl sinon le tp ne sert à rien vu que c'est mon premier tp de c++ ...
08/05/2006, 20h07
b Oo

Bonsoir,
les lignes d'erreurs indiquées par le compilateur font bien référence à ton fichier.h ? Normalement tu dois avoir le nom du fichier et la ligne où il y a une erreur.
Je pense qu'il y a peut-être une erreur de syntaxe dans ton fichier.cpp, tu pourrais montrer le code de celui-ci (les lignes que le compilateur t'indique).

b Oo