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
                • Ответ был помечен как решение.

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

                  Комментарии

                  Только авторизованные пользователи могут публиковать комментарии.
                  Пожалуйста, авторизуйтесь или зарегистрируйтесь
                  AD

                  C++ - Тест 004. Указатели, Массивы и Циклы

                  • Результат:50баллов,
                  • Очки рейтинга-4
                  m
                  • molni99
                  • 26 октября 2024 г. 1:37

                  C++ - Тест 004. Указатели, Массивы и Циклы

                  • Результат:80баллов,
                  • Очки рейтинга4
                  m
                  • molni99
                  • 26 октября 2024 г. 1:29

                  C++ - Тест 004. Указатели, Массивы и Циклы

                  • Результат:20баллов,
                  • Очки рейтинга-10
                  Последние комментарии
                  i
                  innorwall11 ноября 2024 г. 22:12
                  Django - Урок 055. Как написать функционал auto populate field Freckles because of several brand names retin a, atralin buy generic priligy
                  i
                  innorwall11 ноября 2024 г. 18:23
                  QML - Урок 035. Использование перечислений в QML без C++ priligy cvs 24 Together with antibiotics such as amphotericin B 10, griseofulvin 11 and streptomycin 12, chloramphenicol 9 is in the World Health Organisation s List of Essential Medici…
                  i
                  innorwall11 ноября 2024 г. 15:50
                  Qt/C++ - Урок 052. Кастомизация Qt Аудио плеера в стиле AIMP It decreases stress, supports hormone balance, and regulates and increases blood flow to the reproductive organs buy priligy online safe Promising data were reported in a PDX model re…
                  i
                  innorwall11 ноября 2024 г. 14:19
                  Алгоритм сортировки кучей The role of raloxifene in preventing breast cancer priligy precio
                  i
                  innorwall11 ноября 2024 г. 13:55
                  PyQt5 - Урок 006. Работа с QTableWidget buy priligy 60 mg 53 have been reported by Javanovic Santa et al
                  Сейчас обсуждают на форуме
                  i
                  innorwall11 ноября 2024 г. 20:56
                  добавить qlineseries в функции buy priligy senior brother Chu He, whom he had known for many years
                  i
                  innorwall11 ноября 2024 г. 10:55
                  Всё ещё разбираюсь с кешем. 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
                  9Anonim25 октября 2024 г. 9:10
                  Машина тьюринга // Начальное состояние 0 0, ,<,1 // Переход в состояние 1 при пустом символе 0,0,>,0 // Остаемся в состоянии 0, двигаясь вправо при встрече 0 0,1,>…
                  ИМ
                  Игорь Максимов3 октября 2024 г. 4:05
                  Реализация навигации по разделам Спасибо Евгений!

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