创建新 QDialog 对象的安全方法

Question

我正在编写一个 Qt 应用程序，其中 QListWidget 填充有 QListWidgetItems，我想在双击 QListWidgetItem 时创建一个新对话框。所以我在this qt 教程中创建了一个新的对话框类。现在，我添加了on_listWidget_itemDoubleClicked(QListWidgetItem* item) 插槽，我应该在其中创建对话框类的实例。但我希望我的主应用程序仍然可以运行并继续，并且只有对话框窗口显示一些信息。所以我不能只创建一个对话框类的实例并用exec() 函数显示它。

所以我想出的是让我的对话框类中的std::unique_ptr 作为我的主类的类成员，并使用show() 方法调用对话框。每次调用on_listWidget_itemDoubleClicked(QListWidgetItem* item) 时，我都会使用std::make_unique 创建对话框类的新实例，它应该销毁旧对话框并创建一个新对话框。

这通常是一种安全的方法吗？还是有其他解决这种情况的标准方法？

编辑

这是我认为的示例：

主类：

class USB_Packet_Analyzer : public QWidget
{
    Q_OBJECT

public:
    USB_Packet_Analyzer(QWidget *parent = Q_NULLPTR);    
private:
    Ui::USB_Packet_AnalyzerClass ui;
    std::unique_ptr<DataViewer> dataViewer;
private slots:
    void on_listWidget_itemDoubleClicked(QListWidgetItem* item);
};


void USB_Packet_Analyzer::on_listWidget_itemDoubleClicked(QListWidgetItem* item)
{
    dataViewer = std::make_unique<DataViewer>(item, this);
    dataViewer->show();
}

对话框类：

class DataViewer : public QDialog
{
    Q_OBJECT

public:
    DataViewer(QListWidgetItem* item, QWidget *parent = Q_NULLPTR);
    ~DataViewer();

private:
    Ui::DataViewer ui;
};

Answer 1

是的，你正在做的很好。您可能需要解决一些小问题：

1. 关闭对话框不会释放内存。内存将被消耗，直到unique_ptr被销毁。

2. 关闭对话框时不会清除unique_ptr。现在这可能没问题，但如果您稍后尝试使用 unique_ptr 可能会出现问题。

这些现在可能不会造成问题，但将来可能会。这就是为什么我可以建议以下修改：

void USB_Packet_Analyzer::on_listWidget_itemDoubleClicked(QListWidgetItem* item)
{
    if (!dataViewer.isNull()) dataViewer->deleteLater();
    dataViewer = QPointer(new DataViewer(item, this));
    dataViewer ->setAttribute(Qt::WidgetAttribute::WA_DeleteOnClose);
    dataViewer->show();
}

QPointer 可让您轻松检测小部件是否已被删除。

虽然从技术上讲，您使用 unique_ptr 是安全的，但在智能指针中使用 QObject 类是危险的。 QObject 生命周期通常通过父子关系管理。因为智能指针不知道这一点，所以在执行此操作时很容易获得双重释放。因此，我建议您使用QPointer 版本。

Answer 2

你不需要一个特殊的定制对话框来显示几行信息 改用 MessageBox...

 QMessageBox::information(this, "some title", "my text");