AC
Alexandru CodreanuJan. 29, 2020, 11 a.m.

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, 3:07 p.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, 10:17 p.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, 10:29 p.m.

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

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

        AC
        • Jan. 29, 2020, 11: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, 2:49 p.m.
          • (edited)

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

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

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

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

            AC
            • Feb. 4, 2020, 9:15 p.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, 9:21 p.m.
              • (edited)

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

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

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

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

                  Comments

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

                  Qt - Test 001. Signals and slots

                  • Result:84points,
                  • Rating points4
                  Ua

                  Qt - Test 001. Signals and slots

                  • Result:42points,
                  • Rating points-8
                  ОК

                  Qt - Test 001. Signals and slots

                  • Result:47points,
                  • Rating points-6
                  Last comments
                  ИМ
                  Игорь МаксимовNov. 22, 2024, 9:51 p.m.
                  Django - Tutorial 017. Customize the login page to Django Добрый вечер Евгений! Я сделал себе авторизацию аналогичную вашей, все работает, кроме возврата к предидущей странице. Редеректит всегда на главную, хотя в логах сервера вижу запросы на правильн…
                  Evgenii Legotckoi
                  Evgenii LegotckoiOct. 31, 2024, 11:37 p.m.
                  Django - Lesson 064. How to write a Python Markdown extension Добрый день. Да, можно. Либо через такие же плагины, либо с постобработкой через python библиотеку Beautiful Soup
                  A
                  ALO1ZEOct. 19, 2024, 5:19 p.m.
                  Fb3 file reader on Qt Creator Подскажите как это запустить? Я не шарю в программировании и кодинге. Скачал и установаил Qt, но куча ошибок выдается и не запустить. А очень надо fb3 переконвертировать в html
                  ИМ
                  Игорь МаксимовOct. 5, 2024, 4:51 p.m.
                  Django - Lesson 064. How to write a Python Markdown extension Приветствую Евгений! У меня вопрос. Можно ли вставлять свои классы в разметку редактора markdown? Допустим имея стандартную разметку: <ul> <li></li> <li></l…
                  d
                  dblas5July 5, 2024, 8:02 p.m.
                  QML - Lesson 016. SQLite database and the working with it in QML Qt Здравствуйте, возникает такая проблема (я новичок): ApplicationWindow неизвестный элемент. (М300) для TextField и Button аналогично. Могу предположить, что из-за более новой верси…
                  Now discuss on the forum
                  f
                  firstlunoxodFeb. 15, 2025, 1:46 p.m.
                  Рисование на QGraphicsScene при зажатой кнопке мыши Подскажите, пожалуйста! Как данный класс можно дополнить, чтобы созданные объекты можно было перемещать мышкой по сцене?
                  Дмитрий
                  ДмитрийFeb. 3, 2025, 4:24 p.m.
                  Создание deb-пакета. Как создать ярлык на рабочем столе после установки собственного deb-пакета? Всем привет. Сделал свой deb-пакет с программой. Всё устанавливается и работает. Ставлю по пути /usr/bin/my_application. Как для пользователя при установке пакета сразу создать ярлык на раб…
                  NW
                  Nayo WaiJan. 30, 2025, 7:22 p.m.
                  не запускается компьютер!!! Не запускается компьютер (точнее работает блок , но сам монитор вообще жесть)В общем я ничего с интернета не скачивала в последнее время. На компе никаких левых пр…
                  n
                  nklyJan. 3, 2025, 12:52 p.m.
                  Нужно запретить перемещение только некоторых итемов, остальные перемещать можно. Вопрос решен. Узнать QModelIndex элемента на который мы перетаскиваем другой элемент, можно с помощью функции indexAt(event->position().toPoint()) представления QTreeViev вызываемой в переопр…
                  M
                  MarselAug. 17, 2023, 12:26 a.m.
                  OAuth2.0 через VK, получение email Спасибо большое за помощь и простите за то что отнял время своей невнимательностью.

                  Follow us in social networks