Discussion :

[Beuss]

Bonjour c'est encore moi,
j'ai un nouveau problème, j'ai cherché sur le net, sur la FAQ et sur le forum, je ne trouve pas pourquoi ça ne marche pas. Le compilo me dit que je n'ai pas de méthode qui prend un string en paramètre... Pourtant j'en ai une définie dans la classe mère. D'où ça vient, c'est un problème avec la surcharge et l'héritage ? Comment faire ? (je me doute que j'ai encore du faire une grosse bourde mais il y a longtemps que je n'ai pas fait de C++...)

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
 
 
class c1
{
    public:
        virtual void func() { cout << "func"; }
        virtual void test(const char* c)= 0;
        virtual void test(const string &c) { test(c.c_str()); }
        virtual ~c1() { }
};
 
class c2 : public c1
{
    public:
        virtual void test(const char* c) { cout << c << endl; }
};
 
 
int main(void)
{
    c2 t;
    std::string toto("toto");
    t.func();
    t.test(toto);
    return 0;
}

voilà ce que dit g++ :

main.cpp: In function 'int main()':
main.cpp:30: erreur: no matching function for call to 'c2::test(std::string&)'
main.cpp:21: note: candidats sont: virtual void c2::test(const char*)

Merci d'avance

[Pragmateek]

C'est normal il appelle la fonction "test" de "c2" qui prend en paramètre un "char*" et pas un paramètre "string".
Tu veux appeler "test" de la classe "c1"?

[Matthieu Brucher]

Les fonctions de c1 sont cachées par la fonction de c2 car elles ont le même nom. Il te faut la déclarer explicitement.

[Pragmateek]

Les fonctions de c1 sont cachées par la fonction de c2 car elles ont le même nom. Il te faut la déclarer explicitement.

On ne peut pas bricoler autrement avec l'opérateur de résolution de portée par exemple?

[Swoög]

on peut toujours bricoler...
essaie

Code :

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

t.c1::test(toto);

Mais c'est tout sauf clean...

[Pragmateek]

Mais c'est tout sauf clean...

Ca évite quand même de redéfinir la fonction.

[Beuss]

OK merci de ces précisions, donc quand on redéfinit une méthode d'une classe mère il faut redéfinir toutes celles qui ont le même nom ? C'est pas très pratique quand même...

[Matthieu Brucher]

Oui, c'est comme ça

[Beuss]

Merchi

[Swoög]

OK merci de ces précisions, donc quand on redéfinit une méthode d'une classe mère il faut redéfinir toutes celles qui ont le même nom ? C'est pas très pratique quand même...

En théorie, quand tu redéfinies une fonctions dans une classe fille, c'est pour modifier son comportement... donc en toute logique, le comportement devrait être modifier pour toutes les combinaisons de paramètres...

Il faut voir une méthode membre comme une action que peut réaliser l'objet, les paramètres n'étant qu'un amas de données qui permettent à l'objet d'obtenir les informations nécessaire à son action ... et les différentes surdéfinitions modifient simplement la façon de traiter ces données

Donc si tu modifies l'action dans la classe file, tu modifies aussi les données nécessaires à l'action, si tu ne respécifie pas la façon d'interpréter tel ou tel ensemble de données, il est normal que l'objet ne puisse effectuer la nouvelle action avec cet amas de données là... et pour plus de cohérence, l'action de la classe mère est masquée par l'action de la classe fille...

Je ne sais pas si c'est extrêment clair, mais je pense que tu dois comprendre le pourquoi de ce phénomène


C'est peut-être pas pratique, parce que tu ne l'utilises pas dans la bonne optique

[Beuss]

C'est un point de vue
Comme tu le vois ici, la surcharge est utilisée pour une raison de compatibilité avant migration totale vers string, cela permet d'avoir l'ancien code et le nouveau qui fonctionnent, il n'y a donc pas de raison de la modifier (ce qui me semble logique puisqu'elle fait appel à une fonction virtuelle pure...)

[Swoög]

dans ce cas, déclares plutôt la fonction avec le type std::string en paramètre, comme char* est implicitement convertible en std::string, il n'y aura plus aucun problème...