c++ - 如何删除单例指针？

Question

我正在实现一个单例模式。在这里，我正在 GetInstance 中创建一个 Singleton* 的新实例，当我尝试在析构函数中删除它时，它会无限循环。在这种情况下如何避免内存泄漏？

请引用以下代码:

#define NULL 0
class Singleton  
{ 
    private :  
        static Singleton* m_pInstance;  
        Singleton(){};  

    public :

    static Singleton* GetInstance()
    {
        if(m_pInstance == NULL)
        {
            m_pInstance  = new Singleton();         
        }
        return m_pInstance;
    }

    ~Singleton()
    { 
        //delete m_pInstance; // The system goes in infinate loop here if i uncomment this  
        m_pInstance = NULL;
    }
};

Singleton*  Singleton ::m_pInstance = NULL;   

int main()  
{
    Singleton* pInstance = Singleton::GetInstance();
    delete pInstance;  
}     

Answer 1

当然会导致无限循环!

你调用了析构函数，但是析构函数也调用了析构函数，所以析构函数又调用了析构函数......又......

如果你想使用delete，你必须在析构函数的外部使用它，并且不要在析构函数中再次调用它。

为此，您可以使用另一个静态方法来镜像 GetInstance() 方法:

class Singleton  
{ 
public :

   ...

   // this method is a mirror of GetInstance
   static void ResetInstance()
   {
      delete m_pInstance; // REM : it works even if the pointer is NULL (does nothing then)
      m_pInstance = NULL; // so GetInstance will still work.
   }

   ...

   ~Singleton()
   { 
       // do destructor stuff : free allocated resources if any.
       ...
   }

注意:其他人警告你不要使用单例，他们是对的，因为这种模式经常被滥用。所以使用前请三思。但不管怎样，这是学习的好方法!