V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
WangLiCha
V2EX  ›  Qt

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

  •  
  •   WangLiCha · 139 天前 · 1112 次点击
    这是一个创建于 139 天前的主题,其中的信息可能已经有所发展或是发生改变。

    比如一个像下面这样定义的 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 可以放心使用?

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

    这一行就不对了吧,都不用 addWidget(m_button2.get())
    WangLiCha
        5
    WangLiCha  
    OP
       139 天前
    @augustheart 不过确实没有崩,Qt 5.12.12 ,MSVC
    augustheart
        6
    augustheart  
       139 天前
    @WangLiCha 那有点意思,得看看代码。
    也许是析构函数那边 qt 自己的计数器起作用了吧
    openmm
        7
    openmm  
       139 天前
    只能说这样子用隐患很大,哪天就崩了
    Gwkang
        8
    Gwkang  
       139 天前
    一般来说没啥问题,但需要保证子控件先析构,因为子控件析构的时候会在父对象里面清除他们的关系,这样就不会出现重复释放的问题
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3670 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 10:36 · PVG 18:36 · LAX 02:36 · JFK 05:36
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.