AC
Alexandru Codreanu29 января 2020 г. 0:00

QMdiArea освобождение памяти при закрытии subWindow

QMdiArea

Доброго времени суток.
Пытаюсь разобраться с QMdiArea.
При создании subWindow все понятно, но не могу понять почему при закрытии subWindow не освобождается память.
Вот простой пример:

mainwindow.cpp

#include "mainwindow.h"
#include "ui_mainwindow.h"

MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent)
    , ui(new Ui::MainWindow)
{
    ui->setupUi(this);

    //инициализация
    mdiArea = new QMdiArea(this);
    //настройки
    mdiArea->setViewMode(QMdiArea::TabbedView);
    mdiArea->setTabsClosable(true);
    //установка в качестве центрального виджета
    setCentralWidget(mdiArea);

    connect(ui->openListDoc, SIGNAL(triggered()), this, SLOT(onOpenListDoc()));
    connect(ui->openDoc, SIGNAL(triggered()), this, SLOT(onOpenDoc()));
}

MainWindow::~MainWindow()
{
    delete ui;
}

void MainWindow::onOpenListDoc()
{
    auto listDoc = new ListDoc();
    listDoc->setAttribute(Qt::WA_DeleteOnClose);

    QMdiSubWindow *subWindow = new QMdiSubWindow;
    subWindow->setWidget(listDoc);
    subWindow->setAttribute(Qt::WA_DeleteOnClose); //должна освободиться память

    mdiArea->addSubWindow(subWindow);
    mdiArea->setTabsMovable(true);

    subWindow->show();

    /* определим если есть сигнал закрытия формы
     * в форме ListDoc по кнопке Close -> emit -> closeListForm */
    connect(listDoc, &ListDoc::closeListForm, this, &MainWindow::closeSubWindowsListDoc);
}

void MainWindow::onOpenDoc()
{

}

void MainWindow::closeSubWindowsListDoc()
{
    mdiArea->closeActiveSubWindow();
    mdiArea->removeSubWindow(listDoc); //должна освободиться память
}

1.Как правильно и грамотно реализовать работу с QMdiArea?
2.При закрытие subWindow освободить память.
2.Как из формы listDoc(форма списка документов) открыть саму форму документа (к примеру DocExample) в mdiArea?
Спасибо за понимание.

Рекомендуем хостинг TIMEWEB
Рекомендуем хостинг TIMEWEB
Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Вам это нравится? Поделитесь в социальных сетях!

8
Evgenii Legotckoi
  • 29 января 2020 г. 4:07

Добрый день,

метод removeSubWindow не гарантирует освобождения памяти.

вот описание метода

Removes widget from the MDI area. The widget must be either a QMdiSubWindow or a widget that is the internal widget of a subwindow. Note widget is never actually deleted by QMdiArea. If a QMdiSubWindow is passed in, its parent is set to nullptr and it is removed; but if an internal widget is passed in, the child widget is set to nullptr and the QMdiSubWindow is not removed.

Обратите внимание на следующее

Note widget is never actually deleted by QMdiArea.

Что переводится как,

Обратите внимание, что виджет никогда не удаляется QMdiArea .

Вам нужно вруную выполнять удаление этого виджета

void MainWindow::closeSubWindowsListDoc()
{
    mdiArea->closeActiveSubWindow();
    mdiArea->removeSubWindow(listDoc); //должна освободиться память
    listDoc->deleteLater();
}

Но меня напрягает то, что listDoc у вас объявлена как локальная переменная метода onOpenListDoc . То есть, если у вас этот код не падает, то предположу, что в заголовочном файле у вас есть ещё одна переменная с именем listDoc . Этот кусок должен быть написан иначе, без auto

void MainWindow::onOpenListDoc()
{
    listDoc = new ListDoc();

    // Остальной код
}

И не используйте устаревший синтаксис сигналов и слотов через макросы SIGNAL SLOT, это ведёт к большому числу проблем. Используйте через указатели, как здесь

connect(listDoc, &ListDoc::closeListForm, this, &MainWindow::closeSubWindowsListDoc);

    AC
    • 29 января 2020 г. 11:17
    • (ред.)

    Спасибо за оперативный ответ.
    С функциональными указателями не разобрался до конца. Как раз читаю New_Signal_Slot_Syntax
    Да вы правы в заголовочном файле объявлена переменная listDoc

    mainwindow.h

    #ifndef MAINWINDOW_H
    #define MAINWINDOW_H
    
    #include <QMainWindow>
    #include <QMdiArea>
    #include <QMdiSubWindow>
    
    #include "listdoc.h"
    #include "docexample.h"
    
    QT_BEGIN_NAMESPACE
    namespace Ui { class MainWindow; }
    QT_END_NAMESPACE
    
    class MainWindow : public QMainWindow
    {
        Q_OBJECT
    
    public:
        MainWindow(QWidget *parent = nullptr);
        ~MainWindow();
    
    private slots:
        void onOpenListDoc();
        void onOpenDoc();
        void closeSubWindowsListDoc();
    
    private:
        Ui::MainWindow *ui;
        QMdiArea   *mdiArea;
        ListDoc    *listDoc;
        DocExample *docExample;
    };
    #endif // MAINWINDOW_H
    

    mainwindow.cpp

    #include "mainwindow.h"
    #include "ui_mainwindow.h"
    
    MainWindow::MainWindow(QWidget *parent)
        : QMainWindow(parent)
        , ui(new Ui::MainWindow)
    {
        ui->setupUi(this);
    
        //инициализация
        mdiArea = new QMdiArea(this);
        //настройки
        mdiArea->setViewMode(QMdiArea::TabbedView);
        mdiArea->setTabsClosable(true);
        //установка в качестве центрального виджета
        setCentralWidget(mdiArea);
    
        connect(ui->openListDoc, SIGNAL(triggered()), this, SLOT(onOpenListDoc()));
        connect(ui->openDoc, SIGNAL(triggered()), this, SLOT(onOpenDoc()));
    }
    
    MainWindow::~MainWindow()
    {
        delete ui;
    }
    
    void MainWindow::onOpenListDoc()
    {
        listDoc = new ListDoc(mdiArea);
        listDoc->setAttribute(Qt::WA_DeleteOnClose);
    
        QMdiSubWindow *subWindow = new QMdiSubWindow;
        subWindow->setWidget(listDoc);
        subWindow->setAttribute(Qt::WA_DeleteOnClose); //должна освободиться память
    
        mdiArea->addSubWindow(subWindow);
        mdiArea->setTabsMovable(true);
    
        subWindow->show();
    
        /* определим если есть сигнал закрытия формы
         * в форме ListDoc по кнопке Close -> emit -> closeListForm */
        connect(listDoc, &ListDoc::closeListForm, this, &MainWindow::closeSubWindowsListDoc);
    }
    
    void MainWindow::onOpenDoc()
    {
    
    }
    
    void MainWindow::closeSubWindowsListDoc()
    {
        mdiArea->closeActiveSubWindow();
        mdiArea->removeSubWindow(listDoc); //должна освободиться память
        listDoc->deleteLater();
    }
    
    listDoc->deleteLater();
    

    Не совсем понимаю когда будет удален виджет.
    В случае с listDoc понятно, а в случае с DocExample не понятно когда будет удален (в какой момент).
    Попробую описать последовательность действий:
    1.Выделяем память под виджет DocExample для открытия (для пользователя док.№1)
    2.Запись данных в БД SQLite
    3.Закрытие формы виджета DocExample
    4.Удаляем виджет:

    DocExample->deleteLater();

    5.Следующее открытие формы виджета DocExample (для пользователя док.№2) - выделение памяти

    в описание метода deleteLater() :

    The object will be deleted when control returns to the event loop

    Объект будет удален, когда управление вернется в цикл обработки событий.

    Насколько я понимаю при повторном открытии ( п.5 ) срабатывает метод

    deleteLater()

    Есть ли возможность отследить метод deleteLater() ?

      Evgenii Legotckoi
      • 29 января 2020 г. 11:29

      Если вы хотите понять, когда выполняется удаление объекта при использовании deleteLater() , то просто поставьте точку останова в деструкторе DocExample и пройдитесь в режиме отладки по программе, когда объект будет удалён, то вы сможете посмотреть стеку вызова функций и поймёте, когда сработало удаление объекта.

      Я не совсем понял из вашего описания, в какой моменты вы вообще хотите создать виджет docExample

        AC
        • 29 января 2020 г. 12:36

        Да, по поводу точке останова не подумал, хотя использовал, но для стека вызова другой функции.
        Создание виджета docExample не успел дописать

        void MainWindow::onOpenDoc()
        {
            docExample = new DocExample(mdiArea);
            docExample->setAttribute(Qt::WA_DeleteOnClose);
        
            QMdiSubWindow *subWindow = new QMdiSubWindow;
            subWindow->setWidget(docExample);
            subWindow->setAttribute(Qt::WA_DeleteOnClose);
        
            mdiArea->addSubWindow(subWindow);
            mdiArea->setTabsMovable(true);
        
            subWindow->show();
        
            /* определим если есть сигнал закрытия формы
             * в форме docExample по кнопке Close -> emit -> closeDocExampleForm */
            connect(docExample, &DocExample::closeDocExampleForm, this, &MainWindow::closeSubWindowsDocExample);    
        }
        
        void MainWindow::closeSubWindowsDocExample()
        {
            mdiArea->closeActiveSubWindow();
            mdiArea->removeSubWindow(docExample);
            docExample->deleteLater();
        }
        
          Evgenii Legotckoi
          • 30 января 2020 г. 3:49
          • (ред.)

          Так, ну хорошо, если следовать логике вашей программы и вопрос, то я так понимаю, вам нужно после закрытия одной формы, открыть сразу другую форму?

          Тогда вам достаточно вызвать метод onOpenDoc() в конце метода closeSubWindowsListDoc()

          void MainWindow::closeSubWindowsListDoc()
          {
              mdiArea->closeActiveSubWindow();
              mdiArea->removeSubWindow(listDoc);
              listDoc->deleteLater();
          
              onOpenDoc();
          }
          

          Думаю, что вам должно этого хватить, чтобы сделать последовательное открытие и закрытие окон.

            AC
            • 4 февраля 2020 г. 10:15
            • (ред.)

            Да с последовательным открытием и закрытием окон разобрался. Спасибо.
            Насчет устаревшего синтаксиса сигналов и слотов, исправил (разобрался):

            connect(ui->openListDoc, &QAction::triggered, this, &MainWindow::onOpenListDoc);
            connect(ui->openDoc, &QAction::triggered, this, &MainWindow::onOpenDoc);
            

            Но остался один вопрос:
            в каком случае использовать удаление виджета:
            1. DocExample->deleteLater();
            2. delete DocExample ;
            ... срабатывает в обоих случаях (но хотелось бы узнать на практике как правильно использовать).

              Evgenii Legotckoi
              • 4 февраля 2020 г. 10:21
              • (ред.)

              delete DocExample; можете использовать когда объект не является классом, наследованным от классов библиотеки Qt (ну вы тут deleteLater использовать и не сможете, этого метода просто нет) или не зависит от петли событий Qt. То есть не зависит от обработчиков событий кнопок и т.д. Или когда вы точно уверены, что объект не будет использоваться в другом потоке или другими объектами Qt. Это обычно с опытом приходит. А исходя из опыта уже используйте deleteLater() .

              В принципе, если вам нужно удалить виджет, или объект, наследованный от QObject или QGraphicsItem, то можете сначала через оператор delete удалить. Если приложение будет падать в случае удаления, то попробуйте через deleteLater()

                AC
                • 4 февраля 2020 г. 10:33
                • Ответ был помечен как решение.

                Спасибо за оперативный ответ.

                  Комментарии

                  Только авторизованные пользователи могут публиковать комментарии.
                  Пожалуйста, авторизуйтесь или зарегистрируйтесь
                  e
                  • ehot
                  • 31 марта 2024 г. 14:29

                  C++ - Тест 003. Условия и циклы

                  • Результат:78баллов,
                  • Очки рейтинга2
                  B

                  C++ - Тест 002. Константы

                  • Результат:16баллов,
                  • Очки рейтинга-10
                  B

                  C++ - Тест 001. Первая программа и типы данных

                  • Результат:46баллов,
                  • Очки рейтинга-6
                  Последние комментарии
                  k
                  kmssr8 февраля 2024 г. 18:43
                  Qt Linux - Урок 001. Автозапуск Qt приложения под Linux как сделать автозапуск для флэтпака, который не даёт создавать файлы в ~/.config - вот это вопрос ))
                  АК
                  Анатолий Кононенко5 февраля 2024 г. 1:50
                  Qt WinAPI - Урок 007. Работаем с ICMP Ping в Qt Без строки #include <QRegularExpressionValidator> в заголовочном файле не работает валидатор.
                  EVA
                  EVA25 декабря 2023 г. 10:30
                  Boost - статическая линковка в CMake проекте под Windows Ошибка LNK1104 часто возникает, когда компоновщик не может найти или открыть файл библиотеки. В вашем случае, это файл libboost_locale-vc142-mt-gd-x64-1_74.lib из библиотеки Boost для C+…
                  J
                  JonnyJo25 декабря 2023 г. 8:38
                  Boost - статическая линковка в CMake проекте под Windows Сделал всё по-как у вас, но выдаёт ошибку [build] LINK : fatal error LNK1104: не удается открыть файл "libboost_locale-vc142-mt-gd-x64-1_74.lib" Хоть убей, не могу понять в чём дел…
                  G
                  Gvozdik18 декабря 2023 г. 21:01
                  Qt/C++ - Урок 056. Подключение библиотеки Boost в Qt для компиляторов MinGW и MSVC Для решения твой проблемы добавь в файл .pro строчку "LIBS += -lws2_32" она решит проблему , лично мне помогло.
                  Сейчас обсуждают на форуме
                  a
                  a_vlasov14 апреля 2024 г. 6:41
                  Мобильное приложение на C++Qt и бэкенд к нему на Django Rest Framework Евгений, добрый день! Такой вопрос. Верно ли следующее утверждение: Любое Android-приложение, написанное на Java/Kotlin чисто теоретически (пусть и с большими трудностями) можно написать и на C+…
                  Павел Дорофеев
                  Павел Дорофеев14 апреля 2024 г. 2:35
                  QTableWidget с 2 заголовками Вот тут есть кастомный QTableView с многорядностью проект поддерживается, обращайтесь
                  f
                  fastrex4 апреля 2024 г. 4:47
                  Вернуть старое поведение QComboBox, не менять индекс при resetModel Добрый день! У нас много проектов в которых используется QComboBox, в версии 5.5.1, когда модель испускает сигнал resetModel, currentIndex не менялся. В версии 5.15 при resetModel происходит try…
                  P
                  Pisych27 февраля 2023 г. 4:04
                  Как получить в массив значения из связанной модели? Спасибо, разобрался:))
                  AC
                  Alexandru Codreanu19 января 2024 г. 11:57
                  QML Обнулить значения SpinBox Доброго времени суток, не могу разобраться с обнулением значение SpinBox находящего в делегате. import QtQuickimport QtQuick.ControlsWindow { width: 640 height: 480 visible: tr…

                  Следите за нами в социальных сетях