比如一个像下面这样定义的 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 可以放心使用?
1
Gavin999 148 天前
人家又成熟的父子关系管理,为什么还要用智能指针呢?炫技吗?但一个智能指针也没啥好炫的啊,我只能说画蛇添足。
|
2
jones2000 148 天前
控件这些东西, 直接放在窗口销毁的函数里面,统一释放不就完事了,要什么智能指针。
|
3
augustheart 148 天前
跑一下就知道了。只要不崩就说明他们处理过了。没处理过肯定就是崩。
啊,不对,你这用 get ,应该必崩啊 |
4
ipwx 148 天前
m_button2 = std::make_shared<QPushButton>("BUTTON2", this);
这一行就不对了吧,都不用 addWidget(m_button2.get()) |
5
WangLiCha OP @augustheart 不过确实没有崩,Qt 5.12.12 ,MSVC
|
6
augustheart 148 天前
@WangLiCha 那有点意思,得看看代码。
也许是析构函数那边 qt 自己的计数器起作用了吧 |
7
openmm 148 天前
只能说这样子用隐患很大,哪天就崩了
|
8
Gwkang 148 天前
一般来说没啥问题,但需要保证子控件先析构,因为子控件析构的时候会在父对象里面清除他们的关系,这样就不会出现重复释放的问题
|