在 Widget 之间传输 std::unique_ptr

Transferring std::unique_ptr between Widgets

我有一个 Widget class,它应该拥有另一个 Widget 的唯一所有权。 我想实现一个将这个 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() 函数的实现。我只能得到它 运行ning 的右值。

这里

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

您正在为对象创建一个 unique_ptr,该对象将在作用域退出后销毁。

还有,这里

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

您正在获取 unique_ptr 中现有指针的地址,并将其分配给 另一个 unique_ptr。基本上这里到处都是 wrong/double 释放。

非常不知道你想在这里做什么。无论如何,如果我可以将自己限制在此处与内存管理相关的更改数量以使这段代码有意义的话:

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();
}