fonction virtuelle pure et classe abstraite

Question

J'ai les classes suivantes, Base et Derived et quand je compile le compilateur se plaint qu'il ne peut pas créer une instance de DLog parce qu'il est abstrait. Est-ce que quelqu'un peut me dire comment je peux corriger cette erreur?

Je suppose que c'est parce que les deux fonctions virtuelles pures ne sont pas implémentées dans le dérivé.

class Logger 
{ 
public: 

    virtual void log(int debugLevel, char* fmt, ...) = 0; 
    virtual void log(string logText, int debugLevel, string threadName = "") = 0; 

    static Logger* defaultLogger() {return m_defaultLogger;} 
    static void setDefaultLogger(Logger& logger) {m_defaultLogger = &logger;} 

protected: 

    static Logger* m_defaultLogger; 
}; 

class DLog : public Logger 
{ 
public: 
    DLog(); 
    ~DLog(); 

    static DLog *Instance(); 
    static void Destroy(); 

    void SetLogFilename(std::string filename); 
    void SetOutputDebug(bool enable); 
    std::string getKeyTypeName(long lKeyType); 
    std::string getScopeTypeName(long lScopeType); 
    std::string getMethodName(long lMethod); 

    virtual void log(string logText, int debugLevel) 
    { 
     Log(const_cast<char*>(logText.c_str())); 
    } 

    void Log(char* fmt, ...); 

private: 

    static DLog *m_instance; 

    std::string m_filename; 
    bool m_bOutputDebug; 
}; 

// Dlog instantion comme un singleton

DLog *DLog::Instance() 
{ 
    if (!m_instance) 
     m_instance = new DLog(); 
    return m_instance; 
} 

Answer 1

virtual void log(string logText, int debugLevel, string threadName = "") = 0; 

n'a pas été mis en œuvre en classe Dlog. Vous devez l'implémenter car c'est purement virtuel dans la classe de base.

Vous vouliez probablement dire dans votre première surcharge de log en DLog:

virtual void log(string logText, int debugLevel, string /*threadname*/) 
{ 
    Log(const_cast<char*>(logText.c_str())); 
} 

EDIT: Vous avez également pas mis en œuvre la surcharge de

virtual void log(int debugLevel, char* fmt, ...) = 0; 

Notez ici si que l'utilisation du const_cast est un très mauvaise idée et est un comportement indéfini. Vous pouvez obtenir un comportement bien défini en faisant quelque chose comme ceci:

virtual void log(string logText, int debugLevel, string /*threadname*/) 
{ 
    logText.push_back('\0'); // Add null terminator 
    Log(&logText[0]); // Send non-const string to function 
    logText.pop_back(); // Remove null terminator 
} 

Mieux encore bien, il suffit de faire const-correct « Connexion » en premier lieu.

Answer 2

En dérivant votre DLog classe de Logger vous assurer que vous fournirez la mise en œuvre pour tous (en supposant que vous ne voulez pas DLog comme une classe abstraite) des méthodes virtuelles pures déclarées dans la classe de base. Ici, vous n'avez pas fourni d'implémentation aux fonctions virtuelles pures donc la classe DLog devient une classe abstraite. En C++, vous ne pouvez pas créer une instance d'une classe abstraite, vous obtenez donc l'erreur du compilateur. BTW, votre classe de base manque le destructeur virtuel.