此代码是否提供内存泄漏？

Question

最后我安装了 Ubuntu 并设置了 Qt+Valgrind 以防止我在 Windows 中无法做到的内存泄漏。所以我不明白这段代码是否提供内存泄漏？事实上，Valgrind 说我只有 500 多个问题，但对泄漏一无所知。我

#include <QWidget>
#include <QFrame>
#include <QVBoxLayout>
#include <QApplication>

int main(int argc, char *argv[])

{
    QApplication a(argc, argv);

    QWidget * wdgt = new QWidget;  //this line should be the cause of leakage 
                                   //if it exist (as far as i know)
    QVBoxLayout *layout = new QVBoxLayout;
    QFrame * frame = new QFrame;

    frame->setFrameStyle(QFrame::Panel | QFrame::Plain);
    frame->setLineWidth(5);
    layout->addWidget(frame);

    wdgt->setLayout(layout);
    wdgt->setFixedSize(800,600);
    wdgt->show();

    return a.exec();
}

Answer 1

看到这个帖子：Creating and deallocating a Qt widget object

它解释了如果一个Qt对象有一个父对象，当父对象被销毁时，它会被自动删除。

在您的代码中：

• wdgt 是 layout 的父级，因为你做了 wdgt->setLayout(layout)。
• wdgt 是 frame 的父级，因为您执行了 layout->addWidget(frame) 并且 layout 的父级是 wdgt。正如 thuga 所说，布局将所有权传递给他们自己的父级。

在您的代码中，只有 wdgt 是孤立的（没有 Qt 父级自动删除它）。

要解决此问题，您可以给他一个父母：

QWidget * wdgt = new QWidget(&app);

这样wdgt 是app 的孩子，然后在app 被销毁时会自动删除。

或自行删除：

int main(int argc, char *argv[])
{
    ...
    int res = a.exec();
    delete wdgt; // this will delete wdgt, but also frame and layout
    return res;
}

或者，最后，将它创建为一个对象，以便在超出范围时将其删除：

int main(int argc, char *argv[])
{
    QApplication a(argc, argv);

    QWidget wdgt;

    QVBoxLayout *layout = new QVBoxLayout;
    QFrame * frame = new QFrame;

    frame->setFrameStyle(QFrame::Panel | QFrame::Plain);
    frame->setLineWidth(5);
    layout->addWidget(frame);

    wdgt.setLayout(layout);
    wdgt.setFixedSize(800,600);
    wdgt.show();

    return a.exec();
}

顺便提一下，如果你做QVBoxLayout *layout = new QVBoxLayout(wdgt)，就不需要做wdgt->setLayout(layout)。所以这两段代码是等价的：

QVBoxLayout *layout = new QVBoxLayout(wdgt); // parenting upon construction

等同于：

QVBoxLayout *layout = new QVBoxLayout; // no parent
wdgt->setLayout( layout ); // reparenting

Answer 2

是的，您的代码会泄漏内存，因为您使用 new 创建对象而不使用 Qt 的内存管理。

去

QApplication a(argc, argv);
QWidget * wdgt = new QWidget(&app);
QVBoxLayout *layout = new QVBoxLayout(wdgt); // optional, setLayout does that
QFrame * frame = new QFrame(layout); // optional, addWidget does that

使用 Qt 的内存管理。

---

您也可以使用 C++11 共享指针：

QApplication a(argc, argv);
std::shared_ptr<QWidget> wdgt = std::make_shared<QWidget>();

QVBoxLayout *layout = new QVBoxLayout;
QFrame * frame = new QFrame;

一旦共享指针的最后一个用户超出范围，您的对象就会被自动删除。

Answer 3

您的代码会泄漏内存，但首先您不应该编写甚至需要担心资源泄漏的代码。让编译器为你处理：

// main.cpp
#include <QtWidgets>

int main(int argc, char *argv[]) {
    QApplication a(argc, argv);
    QWidget widget;
    QVBoxLayout layout(&widget);
    QFrame frame;

    frame.setFrameStyle(QFrame::Panel | QFrame::Plain);
    frame.setLineWidth(5);
    layout.addWidget(&frame);

    widget.setFixedSize(800,600);
    widget.show();
    return a.exec();
}