Qt: czy "nowy bez usuwania" powoduje przecieki pamięci za pomocą elementów sterujących?

Question

Szukałem na przykład Qt here:

i wewnątrz konstruktora, mają:

Window::Window() 
{ 
    editor = new QTextEdit(); // Memory leak? 
    QPushButton *sendButton = new QPushButton(tr("&Send message")); // Memory leak? 

    connect(sendButton, SIGNAL(clicked()), this, SLOT(sendMessage())); 

    QHBoxLayout *buttonLayout = new QHBoxLayout(); // Memory leak? 
    buttonLayout->addStretch(); 
    buttonLayout->addWidget(sendButton); 
    buttonLayout->addStretch(); 

    QVBoxLayout *layout = new QVBoxLayout(this); // Memory leak? 
    layout->addWidget(editor); 
    layout->addLayout(buttonLayout); 

    setWindowTitle(tr("Custom Type Sending")); 
} 

te linie z komentarzami

// Memory leak? 

nie są te przecieki pamięci?

Jeśli tak, ponieważ klasa okna nie ma konstruktora, powinienem utworzyć wszystkie te zmienne (edytor już jest) Zmienne elementu okna?

Lub ... czy Qt wewnętrznie "usuwa" te zmienne składowe, gdy wykracza poza zakres?

Answer 1

Nie, funkcja addWidget() zachowa prawo własności do widżetu. Następnie zniszczy widżety, które posiada.

Dodatkowo można przeczytać here że:

Jak z QObjects, QWidgets mogą być tworzone z obiektów nadrzędnych do wskazują własności, zapewniając, że obiekty są usuwane, gdy nie są one już używane. W przypadku widgetów te relacje nadrzędny-podrzędny mają dodatkowe znaczenie: Każdy widżet podrzędny jest wyświetlany w obszarze zajmowanym przez jego widget nadrzędny. Oznacza to, że po usunięciu widgetu okna wszystkie znajdujące się w nim widżety podrzędne również zostaną usunięte.

Answer 2

Jeśli istnieje wyjątek zgłoszony między new i addWidget, to tak, istnieje przeciek pamięci. W przeciwnym razie nadrzędna kontrola przejmuje na siebie własność pamięci.

QHBoxLayout *buttonLayout = new QHBoxLayout(); // Memory leak? 
//make sure you don't throw here 
buttonLayout->addWidget(sendButton); 

Answer 3

Oprócz prawidłowej odpowiedzi Klaim za:

chciałbym przechowywać te wskaźniki w std::auto_ptr tymczasem przekazać je do rodzica.

std::auto_ptr<QHBoxLayout> buttonLayout(new QHBoxLayout()); 
// make things which could throw... 
layout->addLayout(buttonLayout.release()); 

W ten sposób na pewno nie będziesz mieć przecieków.

Answer 4

Nie zostanie podwójnie usunięty z powodu wywołania .release().

Uwaga std :: unique_ptr zastępuje std :: auto_ptr. Mam nadzieję, że QT będzie obsługiwać semantykę ruchu, a wtedy release() będzie zamiast layout-> addLayout (std :: move (buttonLayout)) i bez wywoływania wywołania pojawi się błąd kompilacji.