Ruslan Polupan
Мамыр 28, 2020, 6:13 Т.Қ.

Освобождение памяти QMainWindow::setCentralWidget

QMainWindow, setCentralWidget

Доброго времени суток.
Основное окно программы выглядит так


При нажании на кнопки слева создается диалог и помещается в центральный виджет MainWindow.

  1. void MainWindow::slotGetNumberButton()
  2. {
  3. DynamiicButton *button = static_cast<DynamiicButton*>(sender());
  4. const int clientID = button->getButtonID();
  5. ClientInfoDialog *clnInfDlg = new ClientInfoDialog(clientID);
  6. clnInfDlg->setAttribute(Qt::WA_DeleteOnClose);
  7. this->setCentralWidget(clnInfDlg);
  8. clnInfDlg->show();
  9. }

Как освободить память от предидущего виджета перед созданием ногвого?

Спасибо за любую инфу.

3

Ол саған ұнайды ма? Әлеуметтік желілерде бөлісіңіз!

10
Александр Панюшкин
  • Мамыр 28, 2020, 6:24 Т.Қ.

Руслан, добрый день.
Судя по документации это не требуется. QMainWindow должен сам следить за этим виджетом.

    Evgenii Legotckoi
    • Мамыр 28, 2020, 6:26 Т.Қ.
    • Жауап шешім ретінде белгіленді.

    Добрый день.
    Сделайте промежуточную переменную для сохранения виджета.

    1. void MainWindow::slotGetNumberButton()
    2. {
    3. DynamiicButton *button = static_cast<DynamiicButton*>(sender());
    4. const int clientID = button->getButtonID();
    5. if (m_clnInfDlg)
    6. {
    7. m_clnInfDlg->deleteLater();
    8. }
    9. m_clnInfDlg = new ClientInfoDialog(clientID);
    10. clnInfDlg->setAttribute(Qt::WA_DeleteOnClose);
    11. this->setCentralWidget(clnInfDlg);
    12. clnInfDlg->show();
    13. }

    Ну само собой в заголовке класса должны быть эта переменная m_clnInfDlg и должна инициализироваться nullptr

      Evgenii Legotckoi
      • Мамыр 28, 2020, 6:28 Т.Қ.

      Это в том случае, если QMainWindow будет закрыто. А у Руслана окно не закрывается, а просто меняется виджет. То есть происходит накопление объектов в памяти.

        Александр Панюшкин
        • Мамыр 28, 2020, 6:43 Т.Қ.

        Я немного поспорю. Для этого сделал маленькую програмку - стандартный QMainWindows, в который по очереди подключаю в центральный виджет два лейбла, у которых в деструкторе есть дебажные строки.
        Соответственно, при запуске программы по вашей логике ни один деструктор не должен сработать до закрытия программы. Но на практике происходит удаление первого объекта после установки второго.
        Получается, что QMainWindow всё же удаляет объекты!
        Причем делает это довольно грубо. Так при попытке создать эти лейблы не в куче удаление w1 привело к падению программы.
        Исходный код прикладываю.
        Где правда?
        Archive_q8tpBqw.zip Archive_q8tpBqw.zip

          Александр Панюшкин
          • Мамыр 28, 2020, 6:45 Т.Қ.

          И к коду Руслана у меня вопрос - зачем делать clnInfDlg->show();?
          В центральном виджете всё должно и без этого отобразиться.

            Ruslan Polupan
            • Мамыр 28, 2020, 6:46 Т.Қ.

            Увы, каждое нажатие кнопки все равно отгребает 4к памяти..... А там в Диалоге еще модели будут со справочниками из базы данных.

              Ruslan Polupan
              • Мамыр 28, 2020, 6:48 Т.Қ.

              Согласен... По привычке диалого дописал. Но память жрет....

                Александр Панюшкин
                • Мамыр 28, 2020, 6:54 Т.Қ.

                Попробуте прописать в деструкторе ClientInfoDialog меточку и посмотреть, возможно объект и удаляется, но просто не полностью. Т.е., возможно, проблема не в этой части кода.

                  Ruslan Polupan
                  • Мамыр 28, 2020, 6:54 Т.Қ.

                  Пока ходил делать чай. Память начала освобождатья :-) Так что похоже вопрос решился.
                  Спасибо.

                    Evgenii Legotckoi
                    • Мамыр 28, 2020, 6:58 Т.Қ.

                    Да, соглашусь. Просто удаление происходит позже, а не сразу.

                      Пікірлер

                      Тек рұқсаты бар пайдаланушылар ғана пікір қалдыра алады.
                      Кіріңіз немесе Тіркеліңіз