在小部件之间传输std :: unique_ptr

Question

我有一个Widget类，应该具有另一个Widget的唯一所有权。 我想实现一个将此小部件传递给另一个小部件的功能。 我了解到，实现单一所有权概念的最佳方法是std::unique_ptr ，但是我无法使其运行

这是代码：

class Widget
{
public:
    Widget(void) { }
    virtual ~Widget(void) { /* */ }
    void successor(Widget&& successor)
    {
        this->_successor = std::unique_ptr<Widget>(&successor);
    }
    void refer_value(int a)
    {
        _successor->value = a;  
    }
    auto refer_value(void) const -> int
    {
        return _successor->value;
    }
    auto get(void) -> Widget&
    {
        return *_successor;
    }

    int value;
private:
    std::unique_ptr<Widget> _successor;
};

编译没有问题，但是当我尝试运行此命令时：

int main(void)
{
    Widget a{};
    Widget b{};
    Widget c{};

    a.successor(Widget{});
    a.refer_value(5);

    b.successor(std::move(a.get()));
    std::cout << b.refer_value();
}

我遇到了细分错误。 问题是什么？

另外，如果我想传递Widget c ，我将不得不如何编写一个successor（）函数的实现。 我只能让它为右值运行。

Answer 1

这里

this->_successor = std::unique_ptr<Widget>(&successor);

您正在为对象创建一个unique_ptr ，一旦范围退出，该对象将被销毁。

也在这里

b.successor(std::move(a.get()));

您要在unique_ptr现有指针的地址，并将其分配给另一个 unique_ptr 。 在这里到处都是错误/双重释放。

我不知道高度的你想在这里做什么。 无论如何，如果我可以将自己限制在与内存管理相关的更改数量上，以使此代码有意义：

class Widget
{
public:
    Widget(void) { }

    virtual ~Widget(void) { /* */ }

    void successor(std::shared_ptr<Widget> successor)
    {
        this->_successor = std::move(successor);
    }

    void refer_value(int a)
    {
        _successor->value = a;
    }

    auto refer_value(void) const -> int
    {
        return _successor->value;
    }

    std::shared_ptr<Widget> get()
    {
        return _successor;
    }

    int value;

private:
    std::shared_ptr<Widget> _successor;
};

int main(void)
{
    Widget a{};
    Widget b{};
    Widget c{};

    a.successor(std::make_shared<Widget>());
    a.refer_value(5);

    b.successor(a.get());
    std::cout << b.refer_value();
}