gcc-Woverloaded-virtual-Warnungen

Den folgenden C++ - code, den ich denke, ist richtig, aber produzieren einige Warnungen bei der Kompilierung mit "-Woverloaded-virtual" ist die Warnung falsch oder es gibt ein echtes problem mit diesem code?

Wenn, dass ist eine gefälschte Warnung, was kann ich tun um es zu vermeiden, definieren Sie die Ausnahme virtuellen Varianten abgeleitet get befreit von der Warnung, aber vielleicht ist eine bessere Lösung

G++ - Befehl:

   g++ -c -Woverloaded-virtual test.cpp 
test.cpp:22:18: warning: ‘virtual void intermediate::exception(const char*)’ was hidden [-Woverloaded-virtual]
test.cpp:32:18: warning:   by ‘virtual void derived::exception()’ [-Woverloaded-virtual]

C++ - code

using namespace std;

class base
{
public:

    virtual void exception() = 0;
    virtual void exception(const char*) = 0;
};

class intermediate : public base
{
public:

    virtual void exception()
    {
    cerr << "unknown exception" << endl;
    }

    virtual void exception(const char* msg)
    {
    cerr << "exception: " << msg << endl;
    }
};

class derived : public intermediate
{
public:

    virtual void exception() 
    { 
        intermediate::exception("derived:unknown exception");
    }
};
InformationsquelleAutor José | 2012-04-03
  1. 20

    Die Warnung bedeutet, dass:

    Wenn Sie nicht mit dynamischen dispatch dann Ihre derived class-Objekt kann nur aufrufen, 

     void exception()

    und es zu verstecken alle gleichnamigen Methoden aus der Basisklasse intermediate.

    In Ordnung, dass Ihr Objekt abgeleitete Klasse kann alle rufen die gleichnamigen Methoden in der Basis-Klasse intermediate zu können, müssen Sie die folgende Zeile zu Ihrer derived Klasse.

     using intermediate::exception;

    Natürlich, Sie sind in der besten position, um zu entscheiden, ob dies ein problem ist oder nicht.

    • Wie können wir diese Warnung unterdrücken, wenn es beabsichtigt ist?
    • Ich bin mir nicht sicher, aber ich würde vermuten void exception(const char* msg) = delete; funktionieren würde. EDIT: Nein, das löschen einer überlastung wie, dass ist ein Fehler. Aber ich wäre überrascht, wenn das war eigentlich immer absichtlich, in gut entwickelte code; Sie sollten in der Lage sein zu behandeln, eine Instanz von einer abgeleiteten Klasse wie eine Instanz der Basis-Klasse.
    • Wie bereits von den anderen Antworten, können Sie neu definieren private gut.
    InformationsquelleAutor Alok Save
  2. 14

    Wird die Warnung aufgrund der Tatsache, dass Sie nicht nennen derived::exception(const char*) auf ein Objekt vom Typ derived (oder durch einen Zeiger auf derived), auch wenn die Eltern-Klasse definiert, und es ist virtuell (also würde man erwarten, dass es in derived). Die Warnung entfernen, die Sie benötigen, zu entlarven, dass die Funktion in derived:

    class derived : public intermediate 
{ 
public: 
  virtual void exception(const char* msg) {intermediate::exception(msg);}
  virtual void exception()  
  {  
    intermediate::exception("derived:unknown exception"); 
  } 
};

    Oder wenn Sie nicht möchten, setzen Sie es keinen anderen, deklarieren Sie es als privat, mit keine defintion.

    class derived : public intermediate 
{ 
public: 
  virtual void exception()  
  {  
    intermediate::exception("derived:unknown exception"); 
  } 
private:
  void exception(const char* tmp);
};

    UPDATE: Nach der überprüfung (und wie bereits von der Als) die using-Direktive, werden Sie dies auch tun können:

    class derived : public intermediate 
{ 
public: 
  using intermediate::exception; //imports both declarations from intermediate
  virtual void exception() //will not clash with the imported declaration of the
                           //same signature, but properly overriders the parent
                           //class's defition
  {  
    intermediate::exception("derived:unknown exception"); 
  } 
};
    • Aussetzen die Funktion der Basisklasse in der abgeleiteten Klasse alle die OP braucht, um zu tun ist, fügen Sie using intermediate::exception;.
    • Ich Dont think so
    • Eigentlich nicht 🙂 ich dachte an das schreiben mit in meinem ursprünglichen post, entschied sich aber dagegen (aus dem gleichen Grund, dass ich schrieb nach Ihrem ersten Kommentar, hab ich da gelöscht, weil ich dachte, es würde clash). Nach Ihrem Kommentar (und schreiben mir durch den Instinkt), die ich doppelt überprüft, um sicherzustellen,. Nach der Feststellung, Sie waren korrekt, ich aktualisierte meine Antwort, nach dem ich bemerkt habe, Ihren zweiten Kommentar mit dem link. Natürlich haben Sie meine ford für ihn... Aber ich werde wieder updaten, so hat niemand für ungut 🙂
    • Sowieso, Ihre alternative ist großartig für die Armen, alten C++03 Programmierer.
    InformationsquelleAutor Attila
  3. 0

    Bitte die Funktion neu definieren:

    virtual void exception(const char* msg); in der Klasse "abgeleitet".

    Nun Ihr code wird kompiliert ohne warnings.

    • Es kompiliert ohne warnings, aber jetzt hast du erstellt ODR Probleme, wenn jemand versucht zu rufen, dass. Diese wird wahrscheinlich manifestieren sich als linker Fehler, die in der Regel schwieriger zu Lesen und zu verstehen, als dass der compiler Warnungen oder compiler-Fehler.
    InformationsquelleAutor Tamilalagan
Schreibe einen Kommentar