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
                  AD

                  C ++ - Test 004. Pointers, Arrays and Loops

                  • Result:50points,
                  • Rating points-4
                  m

                  C ++ - Test 004. Pointers, Arrays and Loops

                  • Result:80points,
                  • Rating points4
                  m

                  C ++ - Test 004. Pointers, Arrays and Loops

                  • Result:20points,
                  • Rating points-10
                  Last comments
                  i
                  innorwallNov. 14, 2024, 7:03 p.m.
                  Qt/C++ - Lesson 060. Configuring the appearance of the application in runtime I didnt have an issue work colors priligy dapoxetine 60mg revia cost uk August 3, 2022 Reply
                  i
                  innorwallNov. 14, 2024, 12:07 p.m.
                  Circuit switching and packet data transmission networks Angioedema 1 priligy dapoxetine
                  i
                  innorwallNov. 14, 2024, 11:42 a.m.
                  How to Copy Files in Linux If only females relatives with DZ offspring were considered these percentages were 23 order priligy online uk
                  i
                  innorwallNov. 14, 2024, 9:09 a.m.
                  Qt/C++ - Tutorial 068. Hello World using the CMAKE build system in CLion ditropan pristiq dosing With the Yankees leading, 4 3, Rivera jogged in from the bullpen to a standing ovation as he prepared for his final appearance in Chicago buy priligy pakistan
                  i
                  innorwallNov. 14, 2024, 4:05 a.m.
                  EVILEG-CORE. Using Google reCAPTCHA 2001; 98 29 34 priligy buy
                  Now discuss on the forum
                  i
                  innorwallNov. 14, 2024, 3:39 a.m.
                  добавить qlineseries в функции priligy amazon canada 93 GREB1 protein GREB1 AB011147 6
                  i
                  innorwallNov. 11, 2024, 10:55 a.m.
                  Всё ещё разбираюсь с кешем. priligy walgreens levitra dulcolax carbs The third ring was found to be made up of ultra relativistic electrons, which are also present in both the outer and inner rings
                  9
                  9AnonimOct. 25, 2024, 9:10 a.m.
                  Машина тьюринга // Начальное состояние 0 0, ,<,1 // Переход в состояние 1 при пустом символе 0,0,>,0 // Остаемся в состоянии 0, двигаясь вправо при встрече 0 0,1,>…

                  Follow us in social networks