AC
Alexandru CodreanuJan. 29, 2020, midnight

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?
Спасибо за понимание.

We recommend hosting TIMEWEB
We recommend hosting TIMEWEB
Stable hosting, on which the social network EVILEG is located. For projects on Django we recommend VDS hosting.

Do you like it? Share on social networks!

8
Evgenii Legotckoi
  • Jan. 29, 2020, 4:07 a.m.

Добрый день,

метод 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
    • Jan. 29, 2020, 11:17 a.m.
    • (edited)

    Спасибо за оперативный ответ.
    С функциональными указателями не разобрался до конца. Как раз читаю 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
      • Jan. 29, 2020, 11:29 a.m.

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

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

        AC
        • Jan. 29, 2020, 12:36 p.m.

        Да, по поводу точке останова не подумал, хотя использовал, но для стека вызова другой функции.
        Создание виджета 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
          • Jan. 30, 2020, 3:49 a.m.
          • (edited)

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

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

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

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

            AC
            • Feb. 4, 2020, 10:15 a.m.
            • (edited)

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

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

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

              Evgenii Legotckoi
              • Feb. 4, 2020, 10:21 a.m.
              • (edited)

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

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

                AC
                • Feb. 4, 2020, 10:33 a.m.
                • The answer was marked as a solution.

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

                  Comments

                  Only authorized users can post comments.
                  Please, Log in or Sign up
                  Г

                  C++ - Test 001. The first program and data types

                  • Result:66points,
                  • Rating points-1
                  t

                  C++ - Test 001. The first program and data types

                  • Result:33points,
                  • Rating points-10
                  t

                  Qt - Test 001. Signals and slots

                  • Result:52points,
                  • Rating points-4
                  Last comments
                  G
                  GoattRockSept. 3, 2024, 8:50 p.m.
                  How to Copy Files in Linux Задумывались когда-нибудь о том, как мы привыкли доверять свои вещи службам грузоперевозок? Сейчас такие услуги стали неотъемлемой частью нашей жизни, особенно когда речь идет о переездах между …
                  ВР
                  Влад РусоковAug. 2, 2024, 8:47 a.m.
                  How to Copy Files in Linux Screenshot_20240802-065123.png
                  d
                  dblas5July 5, 2024, 6:02 p.m.
                  QML - Lesson 016. SQLite database and the working with it in QML Qt Здравствуйте, возникает такая проблема (я новичок): ApplicationWindow неизвестный элемент. (М300) для TextField и Button аналогично. Могу предположить, что из-за более новой верси…
                  k
                  kmssrFeb. 9, 2024, 2:43 a.m.
                  Qt Linux - Lesson 001. Autorun Qt application under Linux как сделать автозапуск для флэтпака, который не даёт создавать файлы в ~/.config - вот это вопрос ))
                  Qt WinAPI - Lesson 007. Working with ICMP Ping in Qt Без строки #include <QRegularExpressionValidator> в заголовочном файле не работает валидатор.
                  Now discuss on the forum
                  Evgenii Legotckoi
                  Evgenii LegotckoiJune 24, 2024, 10:11 p.m.
                  добавить qlineseries в функции Я тут. Работы оень много. Отправил его в бан.
                  F
                  FynjyJuly 22, 2024, 11:15 a.m.
                  при создании qml проекта Kits есть но недоступны для выбора Поставил Qt Creator 11.0.2. Qt 6.4.3 При создании проекта Qml не могу выбрать Kits, они все недоступны, хотя настроены и при создании обычного Qt Widget приложения их можно выбрать. В чем может …
                  BlinCT
                  BlinCTJune 25, 2024, 8 a.m.
                  Нарисовать кривую в qml Всем привет. Имеется Лист листов с тосками, точки получаны интерполяцией Лагранжа. Вопрос, как этими точками нарисовать кривую? ChartView отпадает сразу, в qt6.7 появился новый элемент…
                  BlinCT
                  BlinCTMay 5, 2024, 12:46 p.m.
                  Написать свой GraphsView Всем привет. В Qt есть давольно старый обьект дял работы с графиками ChartsView и есть в 6.7 новый но очень сырой и со слабым функционалом GraphsView. По этой причине я хочу написать х…
                  Evgenii Legotckoi
                  Evgenii LegotckoiMay 2, 2024, 9:07 p.m.
                  Мобильное приложение на C++Qt и бэкенд к нему на Django Rest Framework Добрый день. По моему мнению - да, но то, что будет касаться вызовов к функционалу Андроида, может создать огромные трудности.

                  Follow us in social networks