Предоставляет ли этот код утечки памяти?

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

Это explaines, что если один объект 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; 
} 

или fianlly, создать его как объект, так что он будет удален при выходе из области видимости:

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(); 
}