2

如果在我使用下面的单例实例后终止进程。是否存在内存泄漏？

  1. 如果是，我该如何解决这个问题？
  2. 我需要添加“destroyInstance”成员函数吗？
  3. 如果是，当多线程已经引用单例实例时，如何保证多线程安全？（我需要添加引用计数吗？）

我想知道如果我需要考虑所有这些，单例模式似乎非常复杂。

#include <mutex>

class Singleton 
{
    static Singleton *singletonInstance;
    Singleton() {}
    static std::mutex m_;

  public:

    static Singleton* getSingletonInstance()
    {
        std::lock_guard<std::mutex> lock(m_);
        if(singletonInstance == nullptr)
        {
            singletonInstance = new Singleton();
        }
        return singletonInstance;
    }
}
4

1 回答 1

6

您不需要对单例进行手动内存管理。这是您的示例std::unique_ptr：

#include <memory>
#include <mutex>

class Singleton 
{
    static std::unique_ptr<Singleton> singletonInstance;
    Singleton() {}
    static std::mutex m_;

  public:

    static Singleton* getSingletonInstance()
    {
        std::lock_guard<std::mutex> lock(m_);
        if (!singletonInstance)
        {
            singletonInstance = std::make_unique<Singleton>();
        }
        return singletonInstance.get();
    }
};

当std::unique_ptr进程正常结束时将调用析构函数。此外，不需要同步析构函数。

但更好的是，您应该使用 Scott Meyer 的单例：

class Singleton {
public:
    static Singleton& getSingletonInstance() {
        static Singleton instance;
        return &instance;
    }
};

上面的例子是线程安全的，即使没有使用显式锁。

对于您 Q3，即使成员函数getSingletonInstance可能是线程安全的，单例的使用也可能不是。共享可变状态本质上是线程不安全的。

于 2019-02-18T13:59:03.843 回答