C++

Ou placer un virtual operator== ?

Recherche :

Mot : Pseudo : Filtrer
Page : 1 2 Page Suivante Page Précédente Bas de page
Auteur	Sujet : Ou placer un virtual operator== ?

Slayne

Yaouchhh

Bonjour,

Je suis novice en C++, et je suis confronté à un problème dont pour l'instant je ne connais pas la solution.

J'ai pour habitude de redefinir mes operator +, *, == à l'extérieur de ma classe, dans un autre fichier (operateurs.cpp par exemple) en mettant les fonctions operator en friend.

Mais voila, je souhaite qu'un operator== soit virtual, car je veux le redéfinir dans les sous classes. Mais il est impossible que mon friend bool operator==(...) devienne virtual, une fonction friend ne pouvant etre virtual.

Quelle est la solution la plus propre pour résoudre ça ?
Merci d'avance

Publicité

xterminhate

Si vis pacem, para bellum.

Fait voir ta classe de base qui contient un opérateur... C'est pas clair ton histoire.

Tu peux ecrire ca...

Code :

class deBase
{
virtual deBase operator* ( const deBase & );
};
class Heritee : public deBase
{
virtual Heritee operator* ( const Heritee & );
};

Message édité par xterminhate le 01-05-2004 à 16:10:59

---------------
Cordialement, Xterm-in'Hate...

Slayne

Yaouchhh

Pour reprendre ton exemple, le probleme c'est que je définis toujours mes operateurs externes comme ca :

Code :

class deBase
{
friend deBase operator* ( const deBase &, const deBase & );
};

pour ensuite les définir dans un fichier à part, operateurs.cpp par exemple.
Mais il devient impossible de les rendre virtual du coup, ce qui est génant vu que je redéfini ces opérateurs dans les classes Héritées.

(hum j'ai l'impression d'avoir réécris la meme chose, donc c'est peut etre pas plus clair [:guts] )

Mais bon effectivement je peux tout réécrire comme ça, mais est ce vraiment une bonne solution ? (c'est peut etre la seule ..)

Message édité par Slayne le 01-05-2004 à 16:21:05

Taz

bisounours-codeur

je pense pas que ça soit très sain de faire des opérateurs virtuels, encore moins les binaires

xterminhate

Si vis pacem, para bellum.

Désolé, mais je ne comprends pas ce que tu veux faire. Rendre virtuel des opérateurs n'est pas une finalité e nsoi. As tu vraiment les idées claires en terme de programmation objet ?

---------------
Cordialement, Xterm-in'Hate...

Slayne

Yaouchhh

Bon d'accord je prend un exemple qui j'espere sera plus clair.

Comme classe mère, je prend une classe abstraite Forme.

Code :

class Forme
{
public :
friend operator==(const Forme &,const Form & );
protected :
Point centre;
}

Son operator== retournera true si le centre des 2 Formes est le meme.

Les classes qui hériteront de Forme seront Carré,Cercle, Triangle, etc.
Par exemple Cercle :

Code :

class Cercle : public Forme
{
public :
friend bool operator==(const Carre &,const Carre & );
private :
double rayon;
}

Si je fais un :
Cercle c1(Point(0,0),2);
Cercle c2(Point(0,0),3);
c1 == c2;
Il n'y aura aucun soucis, ca retournera false, les 2 cercles n'ont pas le meme rayon.

en revanche, un :
Forme *c3 = new Cercle(c1);
Forme *c4 = new Cercle(c2);
(*c3) == (*c4);
Ca retournera vrai vu que ce sera operator== de Forme qui sera appelé.

J'espere avoir été plus clair cette fois ci

Message édité par Slayne le 01-05-2004 à 16:35:31

Taz

bisounours-codeur

les friend cai mal

Taz

bisounours-codeur

Code :

#include <iostream>
struct Base
{
virtual ~Base()
{ }
virtual bool less(const Base &other) const =0;
};
struct Foo
: public Base
{
bool less(const Foo &other) const
{
std::cout << "bool Foo::less(const Foo &other) const\n";
return this < &other;
}
virtual bool less(const Base &other) const
{
std::cout << "bool Foo::less(const Base &other) const\n";
return less(dynamic_cast<const Foo &>(other));
}
};
struct Bar
: public Base
{
bool less(const Bar &other) const
{
std::cout << "bool Bar::less(const Bar &other) const\n";
return this < &other;
}
virtual bool less(const Base &other) const
{
std::cout << "bool Bar::less(const Base &other) const\n";
return less(dynamic_cast<const Bar &>(other));
}
};
inline bool operator<(const Base &lhs, const Base &rhs)
{
std::cout << "bool operator<(const Base &lhs, const Base &rhs)\n";
return lhs.less(rhs);
}
inline bool operator<(const Foo &lhs, const Foo &rhs)
{
std::cout << "bool operator<(const Foo &lhs, const Foo &rhs)\n";
return lhs.less(rhs);
}
inline bool operator<(const Bar &lhs, const Bar &rhs)
{
std::cout << "bool operator<(const Bar &lhs, const Bar &rhs)\n";
return lhs.less(rhs);
}
int main()
{
Base *b1, *b2;
Foo f1, f2;
Bar g1, g2;
std::cout << "Foo < Foo\n";
f1 < f2;
std::cout << '\n';
std::cout << "Bar < Bar\n";
g1 < g2;
std::cout << '\n';
b1 = &f1;
b2 = &f2;
std::cout << "Base& < Base& -> Foo < Foo\n";
*b1 < *b2;
std::cout << '\n';
b1 = &g1;
b2 = &g2;
std::cout << "Base& < Base& -> Bar < Bar\n";
*b1 < *b2;
std::cout << '\n';
b1 = &f1;
b2 = &g1;
try
{
std::cout << "Base& < Base& -> Foo < Bar\n";
*b1 < *b2;
}
catch(std::bad_cast &ex)
{
std::cout.flush();
std::cerr << "Failed " << ex.what() << '\n';
}
}

xterminhate

Si vis pacem, para bellum.

Si tes operateurs prennent en argument des objets de la classe dans laquelle ils sont définis, je vois pas l'interet de friend.

Code :

class Forme
{
public :
virtual bool operator== ( const Forme & ) = 0;
};
class Cercle : public Forme
{
public :
virtual bool operator== ( const Forme & );
};
class Carre : public Forme
{
public :
virtual bool operator== ( const Forme & );
};

Message édité par xterminhate le 01-05-2004 à 17:15:24

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

xterminate > sauf que toi tu n'a absolument rien compris à rien ... alors avant de conseiller ...

Publicité

xterminhate

Si vis pacem, para bellum.

Quel est le problème, j'essais de comprendre en effet ?

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

--Message édité par xterminhate le 01-05-2004 à 16:48:45--

Taz

bisounours-codeur

toute façon essaie avec ta méthode, ça ne peut pas marcher ...

xterminhate

Si vis pacem, para bellum.

Forme *c3 = new Cercle(c1);
Forme *c4 = new Cercle(c2);
(*c3) == (*c4);

C'est bien le == de Forme qui est appelé... et apres ? C'est quoi l'objectif ?

---------------
Cordialement, Xterm-in'Hate...

Slayne

Yaouchhh

xtermin : Mes operateurs prennent des objets de ma classe c'est vrai, mais ils n'influent absolument pas sur elle. Ce ne sont pas des méthodes de classes, et en général je les mets justement à l'extérieur de mon fichier maclasse.cpp.

Taz : Ta solution peut etre pas mal en effet. Enfin du moins le résultat c'est ce que je veux. Mais il risque d'y avoir une duplication de code dans mon cas, vu que ce serait plutot à la classe mère de comparer ses paramètres, et pas à ses classes filles.
Tu vois ce que je veux dire ?

Slayne

Yaouchhh

xterminhate a écrit :

Forme *c3 = new Cercle(c1);
Forme *c4 = new Cercle(c2);
(*c3) == (*c4);

C'est bien le == de Forme qui est appelé... et apres ? C'est quoi l'objectif ?

Mon objectif, c'est que ce soit le == de Cercle qui soit appelé dans ce cas précis.

Taz

bisounours-codeur

Slayne a écrit :

Taz : Ta solution peut etre pas mal en effet. Enfin du moins le résultat c'est ce que je veux. Mais il risque d'y avoir une duplication de code dans mon cas, vu que ce serait plutot à la classe mère de comparer ses paramètres, et pas à ses classes filles.
Tu vois ce que je veux dire ?

non. ça veut rien dire ce que tu dis : si tu as une hiérarchie, chaque classe doit être capable de faire une comparaison entre 2 de ses instances. on a jamais vu quelqu'un rajouter du code à dans sa classe mère à chaque fois qu'on ajouter une classe fille.

(qui corresponds à mon bool less(const Class& ) const pour Bar et Foo). on map ces less sur des bool operator<(const Classe&, const Classe& ). là quand on manipule 2 instances de même type (statiquement) on a le comportement classique sans surcout.

après on a pas de multimethodes, alors on se débrouille pour rajouter quelque chose sans surcout pour le cas normal: chaque classe dérivant de Base doit fournir un bool less(const Base& ) const.

y a un peu de duplication mais pas tant que ça, puisqu'on écrit operator<(const Base&, const Base& ) une fois pour toute, la liaison virtuelle assurant l'appel correct et le cast de la seconde opérande

xterminhate

Si vis pacem, para bellum.

Ah je commence à comprendre, mais auras tu le cas suivant :

Code :

Carre Car;
Cercle Cer;
Forme *F1,*F2;
F1 = Car;
F2 = &Cer;
*F1 == *F2; // comparaison entre deux objets de classes différentes ?

Message édité par xterminhate le 01-05-2004 à 17:20:43

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

qui n'a aucun sens. essaie mon programme. quand on compare un Foo et un Bar dynamiquement, une exception est lancée

Slayne

Yaouchhh

Forme est abstraite, tu ne peux pas l'instancier xterm, juste utiliser des pointeurs vers des Formes.
En revanche oui je veux pouvoir comparer 2 formes différentes ... ce qui me retournera false.

Taz : Tu n'as pas du comprendre ce dont je parlais. Je ne veux pas toucher à la classe mère, je veux juste que la classe mère teste ces propres champs.

Par exemple, voila la meilleur solution que j'ai trouvé pour l'instant :

Code :

class Forme
{
private :
Point centre;
protected :
virtual bool operator==(const Forme &f){
return this->centre == f.centre;
}
}
class Cercle
: public Forme
{
private :
double rayon;
public :
bool operator==(const Forme &f){
if (typeid(*this) == typeid(f))
{
return this->Forme::operator==(f) && this->rayon == f.rayon;
}
return false;
}
}

Ca ferait ce que je veux, sans duplication, mais ca a le defaut de transformer mon operator== en methode de classe.
Si tu trouves que c'est moche, dis le moi

Message édité par Slayne le 01-05-2004 à 17:29:46

Taz

bisounours-codeur

cai toi qui a rien compris : ton code est complètement pourri d'une part (ne jamais utilisé typeid quand on ne sait pas ce que c'est)

après t'as rien compris : rien ne t'empêche de rendre non-virtuelle ma Base::less et de faire en sorte que chaque Classe::less(const Class& ) const fasse appel à Base::less(const Base & ) const

Taz

bisounours-codeur

toute façon tout ça est un peu tendancieux, confier aux opérateur a X b de faire les conversions ... je pense qu'il vaut mieux essayer d'éviter et de s'assurer soit même par cast de ce que sont les objets avec de leur appliquer des opérations typées et symétriques

xterminhate

Si vis pacem, para bellum.

Ce n'est pas compliqué : il ne faut pas avoir à comparer deux objets de type différents ! C'etait l'une de mes premières remarques au sujet de la conception de ton programme....

Message édité par xterminhate le 01-05-2004 à 17:37:49

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Ou alors, il faut que == soit défini dans Forme et compare des attributs qui sont communs à toutes les formes hérités (genre : surface, nb de sommets, ...etc).

Message édité par xterminhate le 01-05-2004 à 17:39:25

---------------
Cordialement, Xterm-in'Hate...

Slayne

Yaouchhh

Taz a écrit :

cai toi qui a rien compris : ton code est complètement pourri d'une part (ne jamais utilisé typeid quand on ne sait pas ce que c'est)
Supair

après t'as rien compris : rien ne t'empêche de rendre non-virtuelle ma Base::less et de faire en sorte que chaque Classe::less(const Class& ) const fasse appel à Base::less(const Base & ) const
Ca revient a faire ce que j'ai fait quoi sauf pour Base

Et le castage est impossible si je veux créer une liste de Forme par exemple. Quand je voudrais en supprimer une, il faudra bien que je compare 2 formes différentes entre elles
Merci quand meme de ta participation mais si c'est pour dire des conneries ou insulter tu peux laisser ta place merci :hello:

Slayne

Yaouchhh

xterminhate a écrit :

Ou alors, il faut que == soit défini dans Forme et compare des attributs qui sont communs à toutes les formes hérités (genre : surface, nb de sommets, ...etc).

C'est ça

xterminhate

Si vis pacem, para bellum.

Ben alors tu n'as pas besoin de tout ce qui s'est écrit ici bas. Tu fais juste == non virtual, non friend, dans Forme et terminé.

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Déclares les attributs communs dans Forme et mets les à jour depuis des classes dérivées....

Message édité par xterminhate le 01-05-2004 à 17:50:12

---------------
Cordialement, Xterm-in'Hate...

Slayne

Yaouchhh

xterminhate a écrit :

Ben alors tu n'as pas besoin de tout ce qui s'est écrit ici bas. Tu fais juste == non virtual, non friend, dans Forme et terminé.

Ca marchera pour comparer les attributs communs aux 2 formes c'est vrai, mais pas pour comparer les spécifités de chaque forme.
Le == de forme teste si le centre de chaque forme est le meme, le == de cercle appelera celui de forme et testera ces propres attributs (son rayon).

xterminhate

Si vis pacem, para bellum.

Un truc comme ca, te conviendrait il ?

1) comparer les attributs communs des deux objets
2) comparer le type des deux objets
3') si les types sont identiques alors comparer les attributs spécifiques.
3" ) si les types sont différents, ... rien.

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Pour comparer le type de deux objets c'est pas compliqué en le faisant soit même.

Tu déclares un fonctions : virtual std::string type() = 0; dans Forme et par exemple : virtual std::string type() { reutrn "Cercle"; } dans la classe cercle, etc...

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

et tu fais un truc tout bete : objet1.type() == objet2.type()...

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

non. à partir du moment ou les types diffèrent, il faut lancer une erreur. donc

si même type
- comparer
sinon
- lancer exception

je comprends pas vos histoires, si Cercle décide que operator==(Cercle, Cercle) reposent sur operator==(Forme, Forme) il doit pouvour le faire, et si carré décide que son operator)== n'a besoin de rien d'autre, il faut le laisser faire aussi

Taz

bisounours-codeur

xterminhate a écrit :

et tu fais un truc tout bete : objet1.type() == objet2.type()...

jamais

edit: non mais tu le fais exprès ou t'as vraiment rien compris à rien ?

Message édité par Taz le 01-05-2004 à 18:00:24

xterminhate

Si vis pacem, para bellum.

qu'est ce que ca gène ?

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

que tu ne t'y connais absolument pas, que tu conseilles tout et n'importe quoi, tu ferais mieux d'apprendre un peu avant d'être de mauvais conseil

xterminhate

Si vis pacem, para bellum.

C'est facile de dire ca Je m'enerve pas au moins !

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

ben c'est pas sympa de conseiller des choses entre débutants quand on y connait rien ...

xterminhate

Si vis pacem, para bellum.

Pas de problèmes

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Taz a écrit :

jamais

Pourquoi pas ?

---------------
Cordialement, Xterm-in'Hate...

Publicité

Page : 1 2

Page Suivante

Page Précédente

Haut de page

FORUM HardWare.fr

Programmation

C++

Ou placer un virtual operator== ?

Sujets relatifs
Placer image dans le coin	[CSS] placer une image en haut a droite d'un cadre avec IE...
[C] Placer un délai dans l'éxecution d'un programme	placer une boite de dialogue
visual c++ placer des controles sur une form	problème avec 'operator type();' : ambiguité
Overlord operator <<, aide svp	Surcharge d operateur = et virtual , ca va ensemble ?
manière la plus rapide de placer un bit	[PHP-objet] Dans quelle classe placer cette methode ?
Plus de sujets relatifs à : Ou placer un virtual operator== ?

Page générée en 0.146 secondes