BlinCT
BlinCT1 апреля 2020 г. 20:09

Утечка памяти при динамическом выделении

Всем привет.
Такой вопрос, в классе есть приватный метод, в который передаются некоторые данные

QSGGeometryNode *NavRect::lineNode(QPoint beginPoint , QPoint endPoint, QColor colorLine, QSGGeometryNode *oldNode)
{
    auto material = new QSGFlatColorMaterial;
    material->setColor(colorLine);

    if(!oldNode)
    {
        auto pNode = new QSGGeometryNode;
        auto pLineGeometry=new QSGGeometry(QSGGeometry::defaultAttributes_Point2D(), 2);
        pLineGeometry->setLineWidth(1);
        pLineGeometry->setDrawingMode(GL_LINES);

        pLineGeometry->vertexDataAsPoint2D()[0].set(beginPoint.x(), beginPoint.y());
        pLineGeometry->vertexDataAsPoint2D()[1].set(endPoint.x(), endPoint.y());

        pNode->setGeometry(pLineGeometry);
        pNode->setFlags(QSGNode::OwnsGeometry | QSGNode::OwnsMaterial);
        pNode->setMaterial(material);

        return pNode;
    }
    else
    {
        oldNode->setMaterial(material);
    }

    return oldNode;
}

Как тут видите в нем аж 3 new, то есть один возвращается в метод класса для дальнейшей работы, а вот другие 2 нет.
Как правильно разабратся с ними чтоыб указатели не плавали после выхода из метода?
Если их сделать мемберами в данном классе это вроде не избавляет от проблемы?
Спасибо.

Рекомендуем хостинг TIMEWEB
Рекомендуем хостинг TIMEWEB
Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Вам это нравится? Поделитесь в социальных сетях!

8
Evgenii Legotckoi
  • 2 апреля 2020 г. 2:37
  • (ред.)

Привет.

Здесь не надо ничего такого делать, память здесь будет очищаться за счёт этой строчки

pNode->setFlags(QSGNode::OwnsGeometry | QSGNode::OwnsMaterial);

То есть ты берёшь владение материалом и геометрией на pNode, а сама pNode должна будет уничтожаться дальше в дереве объектов OpenGL, попутно уничтожая материал и геометрию. QML всё это предусматривает.

Если по какой-то причине всё-таки происходит утечка памяти, то значит косяк где-то в логике метода updatePaintNode

Для проверки можно сделать для всех этих классов наследников и переопределить деструктор с выводом qDebug, как сделано в этой статье для QStandardItem . Это позволит убедиться, что удаление всё-таки происходит.

    BlinCT
    • 2 апреля 2020 г. 3:35

    Оки спасибо, сейчас проверю.

      BlinCT
      • 2 апреля 2020 г. 4:40

      В общем все таки утечка есть, по причине логики метода где вызывается данная функция.
      Вот так она выглядит

      QSGNode *NavRect::updatePaintNode(QSGNode *oldNode, QQuickItem::UpdatePaintNodeData *updatePaintNodeData)
      {
          Q_UNUSED(updatePaintNodeData)
      
          QColor foreground = Qt::white;
          QColor background = Qt::black;
      
          auto width = boundingRect().width();
          auto height = boundingRect().height();
      
          if(hasFocus())
          {
              background = Qt::white;
              foreground = Qt::black;
          }
      
          setProperty("foreground", foreground);
          setProperty("background", background);
      
      
          if(!oldNode)
          {
              //! complete creation of objects and attaching them to a node, background color
              m_pBackgroundNode = window()->createRectangleNode();
              m_pBackgroundNode->setRect(boundingRect());
              m_pBackgroundNode->setColor(background);
      
              //! draw lines
              m_pNodeLineOne = lineNode(QPoint(0, 0), QPoint(width, 0), foreground);
              m_pNodeLineTwo = lineNode(QPoint(width, 0), QPoint(width, height), foreground);
              m_pNodeLineThree = lineNode(QPoint(width, height), QPoint(0, height), foreground);
              m_pNodeLineFour = lineNode(QPoint(0, height), QPoint(0, 0), foreground);
      
              m_pBackgroundNode->appendChildNode(m_pNodeLineOne);
              m_pBackgroundNode->appendChildNode(m_pNodeLineTwo);
              m_pBackgroundNode->appendChildNode(m_pNodeLineThree);
              m_pBackgroundNode->appendChildNode(m_pNodeLineFour);
      
              return m_pBackgroundNode;
          }
      
          //! if a node exists, update the background
          m_pBackgroundNode->setColor(background);
      
          //! updating lines
          m_pNodeLineOne = lineNode(QPoint(0, 0), QPoint(width, 0), foreground, m_pNodeLineOne);
          m_pNodeLineTwo = lineNode(QPoint(width, 0), QPoint(width, height), foreground, m_pNodeLineTwo);
          m_pNodeLineThree = lineNode(QPoint(width, height), QPoint(0, height), foreground, m_pNodeLineThree);
          m_pNodeLineFour = lineNode(QPoint(0, height), QPoint(0, 0), foreground, m_pNodeLineFour);
      
          return oldNode;
      }
      

      То есть при первом входе создаюется обьект и вызывается lineNode для создания линий, а потом когда уже нода существует то мы снова заходим в метод и опять делаем new.
      Так что вот с этим и проблема как раз.

        Evgenii Legotckoi
        • 2 апреля 2020 г. 4:50
        • (ред.)

        Проблема в том, что если нода уже была создана, то ты заново пересоздаешь ноды, а не перерисовываешь. Посмотри как сделано в этой статье QML - Урок 032. Создаём Custom QuickItem из C++ с использованием средств OpenGL

        Я не создаю там новые объекты, если oldNode существует, я перерисовываю ноду в уже существющих объектах. А ты просто создаешь каждый раз новые объекты m_pNodeLineOne, m_pNodeLineTwo, m_pNodeLineThree, m_pNodeLineFour если oldNode . Либо удаляй их, если они существуют и потом создавай новые, либо переопределяй некоторые свойства. Переопределение наверняка будет производительнее, чем создавать новые объекты.

        В общем тебе нужен ещё один метод, который будет менять точки, то есть менять геометрию.

          Evgenii Legotckoi
          • 2 апреля 2020 г. 4:53
          • (ред.)

          хотя, если вниметльно посмотреть на метод lineNode, то скорее всего проблема в том, что ты создаешь материал в начале метода. Старый скорее всего не удаляется, пока не была удалена нода, а просто заменяется, и остаётся висеть в памяти.

            BlinCT
            • 3 апреля 2020 г. 12:52

            В общем я просто сделал вторйо метод который простог обнавляет цвет и материал, вроде сделал правильно.
            Но вот на эмбедеде есл ия быстро переключаюсь межуд обьектами они начали как бы мерцать, у меня предположение что я что то еще должен добавить в метод но не пойму что.

            QSGGeometryNode *NavRect::nonExistLineNode(QPoint beginPoint , QPoint endPoint, QColor colorLine, QSGGeometryNode *oldNode)
            {
                m_pMaterial = new QSGFlatColorMaterial;
                m_pMaterial->setColor(colorLine);
            
                if(!oldNode)
                {
                    m_pGeometryNode = new QSGGeometryNode;
                    m_pGeometry = new QSGGeometry(QSGGeometry::defaultAttributes_Point2D(), 2);
                    m_pGeometry->setLineWidth(1);
                    m_pGeometry->setDrawingMode(GL_LINES);
            
                    m_pGeometry->vertexDataAsPoint2D()[0].set(beginPoint.x(), beginPoint.y());
                    m_pGeometry->vertexDataAsPoint2D()[1].set(endPoint.x(), endPoint.y());
            
                    m_pGeometryNode->setGeometry(m_pGeometry);
                    m_pGeometryNode->setFlags(QSGNode::OwnsGeometry | QSGNode::OwnsMaterial);
                    m_pGeometryNode->setMaterial(m_pMaterial);
            
                    return m_pGeometryNode;
                }
                else
                {
                    oldNode->setMaterial(m_pMaterial);
                }
            
                return oldNode;
            }
            
            QSGGeometryNode *NavRect::existLineNode(QPoint beginPoint, QPoint endPoint, QColor colorLine, QSGGeometryNode *oldNode)
            {
                m_pMaterial->setColor(colorLine);
            
                oldNode->setMaterial(m_pMaterial);
            
                return oldNode;
            }
            
              BlinCT
              • 3 апреля 2020 г. 12:53

              Само собою на компе этого незаметно.

                Evgenii Legotckoi
                • 6 апреля 2020 г. 4:19

                Попробуй не переустанавливать материал, а просто менять цвет

                  Комментарии

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

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

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

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

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

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

                  • Результат:20баллов,
                  • Очки рейтинга-10
                  Последние комментарии
                  ИМ
                  Игорь Максимов22 ноября 2024 г. 19:51
                  Django - Урок 017. Кастомизированная страница авторизации на Django Добрый вечер Евгений! Я сделал себе авторизацию аналогичную вашей, все работает, кроме возврата к предидущей странице. Редеректит всегда на главную, хотя в логах сервера вижу запросы на правильн…
                  Evgenii Legotckoi
                  Evgenii Legotckoi31 октября 2024 г. 21:37
                  Django - Урок 064. Как написать расширение для Python Markdown Добрый день. Да, можно. Либо через такие же плагины, либо с постобработкой через python библиотеку Beautiful Soup
                  A
                  ALO1ZE19 октября 2024 г. 15:19
                  Читалка fb3-файлов на Qt Creator Подскажите как это запустить? Я не шарю в программировании и кодинге. Скачал и установаил Qt, но куча ошибок выдается и не запустить. А очень надо fb3 переконвертировать в html
                  ИМ
                  Игорь Максимов5 октября 2024 г. 14:51
                  Django - Урок 064. Как написать расширение для Python Markdown Приветствую Евгений! У меня вопрос. Можно ли вставлять свои классы в разметку редактора markdown? Допустим имея стандартную разметку: <ul> <li></li> <li></l…
                  d
                  dblas55 июля 2024 г. 18:02
                  QML - Урок 016. База данных SQLite и работа с ней в QML Qt Здравствуйте, возникает такая проблема (я новичок): ApplicationWindow неизвестный элемент. (М300) для TextField и Button аналогично. Могу предположить, что из-за более новой верси…
                  Сейчас обсуждают на форуме
                  m
                  moogo22 ноября 2024 г. 15:17
                  Mosquito Spray System Effective Mosquito Systems for Backyard | Eco-Friendly Misting Control Device & Repellent Spray - Moogo ; Upgrade your backyard with our mosquito-repellent device! Our misters conce…
                  Evgenii Legotckoi
                  Evgenii Legotckoi24 июня 2024 г. 22:11
                  добавить qlineseries в функции Я тут. Работы оень много. Отправил его в бан.
                  t
                  tonypeachey115 ноября 2024 г. 14:04
                  google domain [url=https://google.com/]domain[/url] domain [http://www.example.com link title]
                  NSProject
                  NSProject4 июня 2022 г. 10:49
                  Всё ещё разбираюсь с кешем. В следствии прочтения данной статьи. Я принял для себя решение сделать кеширование свойств менеджера модели LikeDislike. И так как установка evileg_core для меня не была возможна, ибо он писался…

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