【问题标题】:Singleton with private destructor using std::unique_ptr使用 std::unique_ptr 的带有私有析构函数的单例
【发布时间】:2013-10-16 22:21:44
【问题描述】:

我在我的程序中创建了所有单例,并考虑到了该文档: http://erdani.com/publications/DDJ_Jul_Aug_2004_revised.pdf (如果有人想知道为什么是单例,它们都是工厂,其中一些存储了一些关于如何创建实例的全局设置。

它们每个看起来都像这样:

声明:

class SingletonAndFactory {
    static SingletonAndFactory* volatile instance;

public:
    static SingletonAndFactory& getInstance();

private:
    SingletonAndFactory();

    SingletonAndFactory(
        const SingletonAndFactory& ingletonFactory
    );

    ~SingletonAndFactory();
};

定义:

boost::mutex singletonAndFactoryMutex;

////////////////////////////////////////////////////////////////////////////////

// class SingletonAndFactory {

SingletonAndFactory* volatile singletonAndFactory::instance = 0;

// public:

SingletonAndFactory& SingletonAndFactory::getInstance() {
    // Singleton implemented according to:
    // "C++ and the Perils of Double-Checked Locking".
    if (!instance) {
        boost::mutex::scoped_lock lock(SingletonAndFactoryMutex);
        if (!instance) {
            SingletonAndFactory* volatile tmp = (SingletonAndFactory*) malloc(sizeof(SingletonAndFactory));
            new (tmp) SingletonAndFactory; // placement new
            instance = tmp;
        }
    }
    return *instance;
}

// private:

SingletonAndFactory::SingletonAndFactory() {}

// };

抛开问题单例的什么设计是最好的(因为它会引发一场毫无意义的火焰战争)我的问题是:用 std::unique_ptr 替换普通指针对我有好处吗?特别是,它会在程序退出时调用单例的析构函数吗?如果是这样,我将如何实现它?当我尝试添加类似friend class std::unique_ptr<SingletonAndFactory>; 的内容时,它并没有成功,因为编译器一直抱怨析构函数是私有的。

我知道这在我当前的项目中并不重要,因为没有工厂需要任何类型的清洁,但为了将来参考,我想知道如何实现这种行为。

【问题讨论】:

  • Thing& getThing() { static Thing thing; return thing; } 忘记锁,忘记单身。
  • 然后... 认真malloc + 新位置?哦,来吧...
  • @R.MartinhoFernandes:如果你要使用单例,你最好把它们弄错。有人提倡making wrong code look wrong
  • 我也想知道指针是否用于内存映射I/O,因为它被标记为volatile。 C++11 没有为volatile 分配与线程相关的语义。
  • 看起来volatile 是从链接文档中的一个示例复制而来的,该示例描述了volatile 如何不足以保证线程安全。您需要 C++11 来提供必要的原子性保证;一旦你有了 C++11,你还不如使用本地静态。

标签: c++ c++11 singleton destructor unique-ptr


【解决方案1】:

在 C++11 中,您可以在程序结束时使用本地静态来保证线程安全的延迟初始化和销毁​​:

SingletonAndFactory& SingletonAndFactory::getInstance() {
    static SingletonAndFactory instance;
    return instance;
}

请注意,这仍然会导致生命周期问题,因为它可能会在其他静态对象之前被销毁。如果他们试图从他们的析构函数中访问它,那么你就有麻烦了。

在此之前,这是不可能的(尽管许多编译器都保证了上述内容)。如您链接到的文档中所述,volatile 与线程同步无关,因此您的代码存在数据竞争和未定义的行为。选项是:

  • 利用锁定的(可能很大的)性能损失来检查指针
  • 使用编译器提供的任何不可移植的原子内在函数来测试指针
  • 忘记线程安全初始化,并确保在启动线程之前对其进行初始化
  • 不要使用单例

我赞成最后一个选项,因为它解决了单例反模式引入的所有其他问题。

【讨论】:

    【解决方案2】:

    删除的不是unique_ptr 本身,而是删除器。所以如果你想使用friend 方法,你必须这样做:

    friend std::unique_ptr<SingletonFactory>::deleter_type;
    

    但是,我不认为默认删除器不会将实际的 delete 委托给另一个函数,这会破坏这一点。

    相反,您可能想提供自己的删除器,可能像这样:

    class SingletonFactory {
        static std::unique_ptr<SingletonFactory, void (*)(SingletonFactory*)> volatile instance;
    
    public:
        static SingletonFactory& getInstance();
    
    private:
        SingletonFactory();
    
        SingletonFactory(
            const SingletonFactory& ingletonFactory
        );
    
        ~SingletonFactory();
    
        void deleter(SingletonFactory *d) { d->~SingletonFactory(); free(d); }
    };
    

    并在创建函数中:

    SingletonFactory* volatile tmp = (SingletonFactory*) malloc(sizeof(SingletonFactory));
    new (tmp) SingletonFactory; // placement new
    instance = decltype(instance)(tmp, &deleter);
    

    【讨论】:

    • 谢谢!这就是我一直在寻找的答案。我知道有几种方法可以实现单例行为(例如silviuardelean.ro/2012/06/05/few-singleton-approaches),其中一些方法的区别只是味道。我正在寻找一种方法来改进我自己的实现。 getInstance() 的胆量可以很容易地切换到其他东西,我可能很快就会这样做(因为我切换到 C++11),但我想从整体上改进我的设计。
    • @maddening 我没有把它放到答案中(因为我没有时间阅读您问题中的链接),但请密切注意其他人对您的问题所做的 cmets .您代码中的许多设计决策(位置newvolatile)充其量都是有问题的。
    • 我很清楚这一点。我目前正在寻找如何改进我的项目中的代码。它实际上更像是一个自学项目,我可以在其中进行实验以更好地理解 C++。我当然会使用其中的一些建议,不过 - 就目前而言 - 单身人士会留下来。
    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 2013-02-18
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2018-01-28
    • 1970-01-01
    相关资源
    最近更新 更多