Discussion :

[Kalite]

Bonjour,

Je suis sous C++ builder 5.02.

Lorsque je compile le code suivant j'obtient le warning :
'D::f(int)' hides virtual fonction 'B::f(char)'

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
 
class B {
public:
   virtual void f(char)
   {
      printf("In B::f()\n");
   }
};
 
class D : B {
public:
   virtual void f(int)
   {
      printf("In D::f()\n");
      f('c');
   }
};

J'ai donc ajouté le using qui va bien de la façon suivante:

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
 
class B {
public:
   virtual void f(char)
   {
      printf("In B::f()\n");
   }
   virtual void f(int) = 0;
};
 
class D : B {
public:
   using B::f;
   virtual void f(int)
  {
      printf("In D::f()\n");
   }
};

Mais j'ai toujours le warning donc je voudrait savoir si l'instruction est supporté ou si j'ai fait une erreur ?

[cob59]

Et avec explicit devant chacune des 2 méthodes ?

[Kalite]

Et avec explicit devant chacune des 2 méthodes ?

Le mot clé explicit n'est que pour les constructeur si je ne m'abuse pas.
(Je vais faire le test quand même)
( --> Confirmer ce n'est pas autorisé)

[jblecanard]

Salut

Ton using n'a pas d'utilité ici. using sert pour la résolution du nom lors d'un appel, pas pour la déclaration ou la définition (ça n'a d'ailleurs pas de sens).Quoi qu'il en soit, les deux snippets de code compilent sans erreurs et sans warnings dans VC9. Et cela n'a rien d'étonnant : les deux fonctions sont distinctes de par la différence entre leurs signatures respectives.

M'est avis que ton compilo fait du sale travail, ou alors nous n'avons pas le code complet . Edit : vu que c'est un warning (et pas une erreur comme je l'ai cru), j'aurais tendance à dire que c'est un warning un peu zélé. Il doit y avoir une option pour virer ces warnings en particulier.

[JolyLoic]

Et cela n'a rien d'étonnant : les deux fonctions sont distinctes de par la différence entre leurs signatures respectives.

Le warning ne me semble pas usurpé. En effet, pour pas mal de gens, le code en question sera étonnant :

Code :

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

1
2
3
4
5
D *d = new D();
B *b = d;
 
d->f(0); // Affiche In D::f()
b->f(0); // Affiche In B::f()

Car pour pas mal de gens, la fonction f est redéfinie dans D, alors que ce n'est pas le cas.

Le mot clé explicit n'est que pour les constructeur si je ne m'abuse pas.
(Je vais faire le test quand même)
( --> Confirmer ce n'est pas autorisé)

En C++11, c'est aussi pour les opérateurs de conversion (plus besoin de ruser pour l'idiome du safe bool).

[koala01]

Salut,

Il faut cependant prendre une chose (qui n'est peut être qu'une erreur de copier coller) en compte: en dehors de toute précision contraire, l'héritage par défaut appliqué à une classe est... privé...

C'est peut être ce que tu veux faire, mais, il faut alors savoir que tu ne pourra pas faire passer ta classe dérivée (D dans ton cas) comme étant du type de la classe de base ( B dans ton cas): l'héritage privé n'a, en effet pas la sémantique d'une relation "EST-UN" mais bel et bien celle d'une relation "EST IMPLEMENTE COMME", avec son cortège de restrictions dont l'impossibilité d'accéder, depuis la classe dérivée, aux membres publiques de la classe de base.

Si tu souhaites effectivement pouvoir utiliser l'interface (publique) de ta classe de base, il vaut peut être mieux te tourner vers un héritage publique (en précisant explicitement l'accessibilité public lors de l'héritage)!!!

Cela te donnerait un code proche de

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
class B
{
    public:
       virtual void f(char)
       {
          printf("In B::f()\n");
       }
       virtual void f(int) = 0; // need to be implemented in derived classes
};
class D : public B
{
      // void f(char) will use B's implementation, if not reimplemented
      virtual void f(int)
      {
          printf("In D::f()\n");
          f('c');
      }
};