首页 > 解决方案 > 共享指针(this)

问题描述

我在 main.exe 中有一个异常抛出:0x74AC4192:Microsoft C++ 异常:std::bad_weak_ptr 在内存位置 0x001AF0D0。

Gasstation::Gasstation(int n,int m)
{
    for (int i = 0; i < n; ++i)
    {
        pumps_.push_back(std::make_shared<Pumplace>());
    }
    cashregisters_ = std::make_shared<Cashregister> (shared_from_this(), m);
}

我也在标题中使用了这个:

class Gasstation : public std::enable_shared_from_this<Gasstation>

可能是什么问题呢?

标签: c++oopthisshared-ptr

解决方案


您的代码的问题在于您shared_from_this()在类本身的构造函数中调用,严格来说,它还没有被“共享”。在对象的智能指针存在之前调用构造函数。按照您的示例,如果创建一个shared_ptrto Gasstation

std::shared_ptr<Gasstation> gasStation = std::make_shared<Gasstation>(5,10);
//gasStation is available as a smart pointer, only from this point forward

它的限制enable_shared_from_this不能shared_from_this在构造函数中调用。

一种解决方案虽然不那么优雅,但有一个公共方法来设置 cashregisters_ 变量。构造后可以调用该方法:

Gasstation::Gasstation(int n, int m)
{
    for (int i = 0; i < n; ++i)
    {
        pumps_.push_back(std::make_shared<Pumplace>());
    }
    cashregisters_ = std::make_shared<Cashregsiter>(m);
}

Gasstation::initialise_cashregisters()
{
    cashregisters_->set_gasstation(shared_from_this());    
}

//driver code
std::shared_ptr<Gasstation> gasStation = std::make_shared<Gasstation>(5, 10);
gasStation->initialise_cashregisters();

此解决方案将要求您记住每次初始化 Gasstation 时都调用 initialise_cashregisters。

除此之外,您的选择是有限的,您可能不得不重新考虑您的设计。您是否考虑过在 Cashregister 中使用指向 Gasstation 的原始指针而不是智能指针?如果 cashregister_ 是一个私有变量,并且在分配给它的 Gasstation 的生命周期之后永远不会存在,那么使用原始指针可能是一种安全而优雅的选择。


推荐阅读