在 Qt 中控件同时使用智能指针和父子关系，会有二次删除风险吗？

[WangLiCha]

比如一个像下面这样定义的 QDialog 窗体：

#ifndef PLAYDIALOG_H
#define PLAYDIALOG_H

#include <memory>
#include <QVBoxLayout>
#include <QDialog>

class QPushButton;

class PlayDialog : public QDialog
{
    Q_OBJECT
public:
    explicit PlayDialog(QWidget* parent = nullptr);

private:
    QVBoxLayout* m_layout;

    QPushButton* m_button1;
    std::shared_ptr<QPushButton> m_button2;
    QSharedPointer<QPushButton> m_button3;
};

#endif // PLAYDIALOG_H

#include "playdialog.h"

#include <memory>
#include <QDialog>
#include <QLayout>
#include <QPushButton>
#include <QVBoxLayout>

PlayDialog::PlayDialog(QWidget* parent) : QDialog(parent), m_layout(new QVBoxLayout(this))
{

    m_button1 = new QPushButton("BUTTON1", this);
    m_button2 = std::make_shared<QPushButton>("BUTTON2", this);
    m_button3 = QSharedPointer<QPushButton>::create("BUTTON3", this);

    m_layout->addWidget(m_button1);
    m_layout->addWidget(m_button2.get());
    m_layout->addWidget(m_button3.get());

    setLayout(m_layout);
}

其中的 QPushButton 都设置了 QDialog 窗体为父控件，m_button2和m_button3分别用 C++原生和 Qt 的智能指针进行了包装。如果这个时候关掉父窗体，因为父子级关系三个按钮都会被释放，但是受智能指针管理的m_button2和m_button3按理说也会被释放，这种时候会存在二次删除风险吗？是不是在 Qt 中不应该用智能指针管理设置了父子级关系的 QWidget 控件？还是说 Qt 封装过的 QSharedPointer 可以放心使用？

[Gavin999]

人家又成熟的父子关系管理，为什么还要用智能指针呢？炫技吗？但一个智能指针也没啥好炫的啊，我只能说画蛇添足。

[jones2000]

控件这些东西， 直接放在窗口销毁的函数里面，统一释放不就完事了，要什么智能指针。

[augustheart]

跑一下就知道了。只要不崩就说明他们处理过了。没处理过肯定就是崩。
啊，不对，你这用 get ，应该必崩啊

[ipwx]

m_button2 = std::make_shared<QPushButton>("BUTTON2", this);

这一行就不对了吧，都不用 addWidget(m_button2.get())

[WangLiCha]

@augustheart 不过确实没有崩，Qt 5.12.12 ，MSVC

[augustheart]

@WangLiCha 那有点意思，得看看代码。
也许是析构函数那边 qt 自己的计数器起作用了吧

[openmm]

只能说这样子用隐患很大，哪天就崩了

[Gwkang]

一般来说没啥问题，但需要保证子控件先析构，因为子控件析构的时候会在父对象里面清除他们的关系，这样就不会出现重复释放的问题