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

139 天前
 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_button2m_button3分别用 C++原生和 Qt 的智能指针进行了包装。如果这个时候关掉父窗体,因为父子级关系三个按钮都会被释放,但是受智能指针管理的m_button2m_button3按理说也会被释放,这种时候会存在二次删除风险吗?是不是在 Qt 中不应该用智能指针管理设置了父子级关系的 QWidget 控件?还是说 Qt 封装过的 QSharedPointer 可以放心使用?

1112 次点击
所在节点    Qt
8 条回复
Gavin999
139 天前
人家又成熟的父子关系管理,为什么还要用智能指针呢?炫技吗?但一个智能指针也没啥好炫的啊,我只能说画蛇添足。
jones2000
139 天前
控件这些东西, 直接放在窗口销毁的函数里面,统一释放不就完事了,要什么智能指针。
augustheart
139 天前
跑一下就知道了。只要不崩就说明他们处理过了。没处理过肯定就是崩。
啊,不对,你这用 get ,应该必崩啊
ipwx
139 天前
m_button2 = std::make_shared<QPushButton>("BUTTON2", this);

这一行就不对了吧,都不用 addWidget(m_button2.get())
WangLiCha
139 天前
@augustheart 不过确实没有崩,Qt 5.12.12 ,MSVC
augustheart
139 天前
@WangLiCha 那有点意思,得看看代码。
也许是析构函数那边 qt 自己的计数器起作用了吧
openmm
139 天前
只能说这样子用隐患很大,哪天就崩了
Gwkang
139 天前
一般来说没啥问题,但需要保证子控件先析构,因为子控件析构的时候会在父对象里面清除他们的关系,这样就不会出现重复释放的问题

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1055015

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX