C++Talk.NET C++ language newsgroups   Archives   FAQ   Search   Memberlist   Usergroups   Register   Profile   Log in to check your private messages   Log in  visiteur, surcharge de méthode virtuelle, et warning de GCC          C++Talk.NET Forum Index -> C++ (French) View previous topic :: View next topic   Author Message Matthieu Moy Guest Posted: Tue Jan 25, 2005 10:15 am    Post subject: visiteur, surcharge de méthode virtuelle, et warning de GCC Bonjour, J'utilise dans un programme C++ le design pattern du visiteur. Jusqu'ici, tout a l'air de bien marcher. Je viens de découvrir l'option -Woverloaded-virtual de GCC, qui d'une manière générale me parait très utile, seulement voila : j'ai une hierarchie de classes de visiteurs, certaines classes donnent des implementations par défaut, et d'autres ne surchargent que certaines méthodes. Du coup, le -Woverloaded-virtual de GCC s'affole. Le problème reproduit sur un exemple a peu près minimaliste : #include <iostream> using namespace std; struct base; struct d1; struct d2; struct visitor { virtual void visit(d1 * d) {cout << "d1" << endl;} virtual void visit(d2 * d) {cout << "d2" << endl;} }; struct vcaps : visitor { virtual void visit(d1 * d) {cout << "D1" << endl;} }; struct base { virtual void accept(visitor * v) = 0; }; struct d1 : base { virtual void accept(visitor * v) {v->visit(this);} }; struct d2 : base { virtual void accept(visitor * v) {v->visit(this);} }; int main () { d1 d; d2 dd; vcaps v; d.accept(&v); dd.accept(&v); } GCC me dit : visitor.cpp:11: warning: `virtual void visitor::visit(d2*)' was hidden visitor.cpp:15: warning: by `virtual void vcaps::visit(d1*)' Question : est-ce que je risque vraiment quelque chose avec du code comme ça, ou est-ce que GCC s'affole pour rien ? Bien sur, si je renomme mes fonctions "visit" en "visit_d1" et "visit_d2", le warning disparait. Question 2 : est-ce qu'il vaudrait mieux que je renomme toutes les fonctions "visit" de mes visiteurs ? Et vous, vous les faites comment vos visiteurs ? -- Matthieu Back to top Jean-Marc Bourguet Guest Posted: Tue Jan 25, 2005 10:28 am    Post subject: Re: visiteur, surcharge de méthode virtuelle, et warning de Matthieu Moy <MatthieuNOSPAM.Moy (AT) imag (DOT) fr.invalid> writes: Quote: Question : est-ce que je risque vraiment quelque chose avec du code comme ça, ou est-ce que GCC s'affole pour rien ? Quel est le pb indique par le warning? Que si on utilise la classe *derivee* pour faire un appel a la fonction, on ne la fonction virtuelle n'est pas visible et donc ne sera pas appellee. Quand est-ce que ca pose un probleme? Si on appelle la fonction a partie de la classe derivee et que la resolution de surcharge ne trouve pas (on a un message d'erreur) ou pire trouve un autre membre (on n'a pas de message mais un comportement different de celui attentu, en particulier on va avoir un comportement different suivant qu'on utilise la classe derivee ou la classe de base). Est-ce que dans le cas du pattern visiteur ca pose un gros probleme? Generalement non: les appels sont faits a partir de la classe de base. Le probleme peut avoir lieu pour des membres non virtuels. Quote: Bien sur, si je renomme mes fonctions "visit" en "visit_d1" et "visit_d2", le warning disparait. Question 2 : est-ce qu'il vaudrait mieux que je renomme toutes les fonctions "visit" de mes visiteurs ? Plus simple: using visit; A+ -- Jean-Marc FAQ de fclc++: http://www.cmla.ens-cachan.fr/~dosreis/C++/FAQ C++ FAQ Lite en VF: http://www.ifrance.com/jlecomte/c++/c++-faq-lite/index.html Site de usenet-fr: http://www.usenet-fr.news.eu.org Back to top Matthieu Moy Guest Posted: Tue Jan 25, 2005 10:36 am    Post subject: Re: visiteur, surcharge de méthode virtuelle, et warning de xavier <xtrochu (AT) yahoo (DOT) com> writes: Quote: En ajoutant struct vcaps : visitor { using visitor::visit; // Injecte tous les noms visit de visitor virtual void visit(d1 * d) {cout << "D1" << endl;} }; Merci, Je ne connaissais pas. Il y a une raison pour laquelle ce n'est pas le "comportement" par défaut de C++ ? -- Matthieu Back to top xavier Guest Posted: Tue Jan 25, 2005 10:42 am    Post subject: Re: visiteur, surcharge de méthode virtuelle, et warning de En ajoutant Quote: struct vcaps : visitor { using visitor::visit; // Injecte tous les noms visit de visitor virtual void visit(d1 * d) {cout << "D1" << endl;} }; Le warning : Quote: visitor.cpp:11: warning: `virtual void visitor::visit(d2*)' was hidden visitor.cpp:15: warning: by `virtual void vcaps::visit(d1*)' n'apparait plus. Et de fait, la méthode visit(d2*) n'est plus cachée. Quote: Question : est-ce que je risque vraiment quelque chose avec du code comme ça, ou est-ce que GCC s'affole pour rien ? Comme le warning l'indique, un appel du type : //... d2 b; vcaps v; v.visit(&b); // [1] // ... va échouer avec l'erreur : visitor.cpp:35: error: no matching function for call to `vcaps::visit(d2*)' visitor.cpp:16: note: candidates are: virtual void vcaps::visit(d1*) Si tu n'ajoute pas la ligne "using visitor::visit" xavier Back to top Jean-Marc Bourguet Guest Posted: Tue Jan 25, 2005 10:42 am    Post subject: Re: visiteur, surcharge de méthode virtuelle, et warning de Matthieu Moy <MatthieuNOSPAM.Moy (AT) imag (DOT) fr.invalid> writes: Quote: Il y a une raison pour laquelle ce n'est pas le "comportement" par défaut de C++ ? La regle correspond exactement a ce qui est fait pour des blocs imbriques: on ne va voir dans le bloc englobant (resp dans les classes de base) que si on ne trouve pas dans le bloc courant (resp dans la classe derivee). On peut arguer en faveur d'une autre regle, mais utiliser celle-la a l'avantage de ne pas ajouter encore une autre regle de recherche des noms et avait l'avantage, maintenant vraissemblablement disparu a cause d'autres changements, d'eviter des problemes (pour les details historiques, voir The Design And Evolution of C++, 3.5.3). A+ -- Jean-Marc FAQ de fclc++: http://www.cmla.ens-cachan.fr/~dosreis/C++/FAQ C++ FAQ Lite en VF: http://www.ifrance.com/jlecomte/c++/c++-faq-lite/index.html Site de usenet-fr: http://www.usenet-fr.news.eu.org Back to top Display posts from previous: All Posts1 Day7 Days2 Weeks1 Month3 Months6 Months1 Year Oldest FirstNewest First         C++Talk.NET Forum Index -> C++ (French) All times are GMT Page 1 of 1   Jump to: Select a forum C/C++----------------C LanguageC Language (Moderated)C++ Language (Moderated)C++ language (comp.lang.c++)C++ language, library and standards Non-English C/C++ Newsgroups----------------C++ (German)C++ (French)  You cannot post new topics in this forum You cannot reply to topics in this forum You cannot edit your posts in this forum You cannot delete your posts in this forum You cannot vote in polls in this forum Powered by phpBB © 2001, 2006 phpBB Group SEO toolkit © 2004-2006 webmedic.