c++ - unique_ptr.get() 方法在使用原始指针分配时调用析构函数？

Question

下面的程序使用std::unique_ptr<T>避免手动内存管理。我尝试过两种方法来实现它。问题出在第二种方法中，在分配给原始指针之前，析构函数被调用。这会导致程序崩溃，因为后面的代码会尝试访问无效内存。

我在第二种方法中的意图是，如何将智能指针与现有代码库一起使用，以便我可以利用智能指针提供的自动内存管理。因此，我没有更改声明中的指针类型(即从 Widget* w 到 std::unique_ptr<Widget> w )。

有人可以详细解释一下吗？最佳做法应该是什么？。或者我遗漏了什么？

#include<iostream>
#include<memory>
class Widget {
public:
    Widget() { std::cout << "Widget::Widget()" << std::endl; }
    virtual ~Widget() { std::cout << "Widget::~Widget()" << std::endl; }
    virtual void draw() = 0;
};

class WindowsButton : public Widget {
public:
    WindowsButton() = default;
    ~WindowsButton() = default;
    void draw() { std::cout << "WindowsButton"<<std::endl; }
};

int main() {    
    // Working Code
    // std::unique_ptr<Widget> w = std::unique_ptr<Widget>(new WindowsButton());
    // w.get()->draw();

    //In this way program is crashing while calling the w->draw()
    Widget* w = std::unique_ptr<Widget>(new WindowsButton()).get();
    w->draw();
}

Answer 1

在第二种情况下，您正在创建一个 unique_ptr 的临时实例，并在其上调用 get() 成员函数。 unique_ptr 对象将在完整表达式的末尾(分号处)被销毁。

Widget* w = std::unique_ptr<Widget>(new WindowsButton()).get();
//          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^      ^
//          unnamed temporary instance                        destroyed here

当然，当实例被销毁时，unique_ptr 将删除 托管对象，并调用 ~Widget()。

w->draw();

然后将取消引用指向无效内存位置的指针，从而导致未定义的行为。