April 1, 2020, 8:09 p.m.

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

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

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 нет.
Как правильно разабратся с ними чтоыб указатели не плавали после выхода из метода?
Если их сделать мемберами в данном классе это вроде не избавляет от проблемы?
Спасибо.

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.
8

Привет.

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

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

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

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

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

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

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

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.
Так что вот с этим и проблема как раз.

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

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

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

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

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

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;
}

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

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

Comments

Only authorized users can post comments.
Please, Log in or Sign up
How to become an author?

Contribute to the evolution of the EVILEG community.

Learn how to become a site author.

Learn it
Donate

Good day, Dear Users!!!

I am Evgenii Legotckoi, developer of EVILEG. And it is my hobby project, which helps to learn programming another programmers and developers

If the site helped you, and you want also support the development of the site, than you can donate by following ways

PayPalYandex.Money
Timeweb

Let me recommend you the excellent hosting on which EVILEG is located.

For many years, Timeweb has been proving his stability.

For projects on Django I recommend VDS hosting

View Hosting Timeweb
AS
May 26, 2020, 12:29 p.m.
Artem Sun-Dun-Chan

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

  • Result:50points,
  • Rating points-4
MN
May 25, 2020, 12:33 p.m.
Mitja Nagibin

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

  • Result:50points,
  • Rating points-4
f
May 25, 2020, 6:05 a.m.
falcon

C++ - Test 001. The first program and data types

  • Result:66points,
  • Rating points-1
Last comments
May 29, 2020, 7:30 a.m.
Evgenij Legotskoj

Qt/C++ - Lesson 039. How to paint stroke in QSqlTableModel by value in the column?

У меня работает. Исправлял в проекте, который приложен к статье. А что происходит в вашем коде, с учётом места вызова этого кода, я знать не могу ;) Дебажьте и добавляйте условия, кото…
MA
May 29, 2020, 7:27 a.m.
Mihail A

Qt/C++ - Lesson 039. How to paint stroke in QSqlTableModel by value in the column?

QModelIndexList rowIndexes = ui->tableView->selectionModel()->selectedRows(); model->removeRows(rowIndexes.first().row(), rowIndexes.size()); model-&…
May 29, 2020, 7:14 a.m.
Evgenij Legotskoj

Django - Tutorial 036. How to add authentication through social networks. VKontakte

Неправильно прописали URL, на который возвращается ответ от OAuth ВКонтакте. Настраивайте ваше приложение в консоли разработчика ВКонтакте
AL
May 29, 2020, 6:24 a.m.
Aleksandr Leonidov

Django - Tutorial 036. How to add authentication through social networks. VKontakte

Здравствуйте! После клика на ссылку авторизации выдает json словарь с ошибкой: {"error":"invalid_request","error_description":"redirect_uri is incorrect, check application redirect uri in …
May 28, 2020, 4:14 p.m.
Evgenij Legotskoj

Qt/C++ - Lesson 039. How to paint stroke in QSqlTableModel by value in the column?

Ну в моём примере, который в статье сработало так model->setData(model->index(1, 1), 7); Поскольку model->index(1, 0) - это индекс колонки id, которая скрыта, поэтому…
Now discuss on the forum
May 29, 2020, 8:52 a.m.
Vladimir Sergeevich

Масштабирование двумя пальцами на мобильных платформах

Я планировал описать этот момент на блоге, но никак руки не доходят (уже год). Летом дойдут. Тем не менее, у меня в репозитории лежит рабочий код игрушки "пазлы", где есть все это. …
May 29, 2020, 7:51 a.m.
Evgenij Legotskoj

Графическое ускорение

Зависит от платформы и поддерживаемых технологий. В QML в первую очередь используется OpenGL и отрисовка производится средствами GPU. Но может переключаться на использование CPU и прог…
IP
May 29, 2020, 2:55 a.m.
Igor' Poroshin

QTablwView + QSqlQueryModel скрыть пустой столбец

Да, понятно. В данном случае лучше использовать серверную процедуру (если такие поддерживаются), в которой будет проверяться наличие всех пустых строк у нужного столбца и вызываться соответ…
RG
May 28, 2020, 7:21 p.m.
Rovshan Gurbanov

Сборка под старые версии Android

У меня SDK почти все версии есть, NDK есть версии 10, 17, 21. Но собирается приложение только с NDK v21 под Android версии 7.0 и выше Версия Qt у меня 5.14.2
May 28, 2020, 8:58 a.m.
Evgenij Legotskoj

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

Да, соглашусь. Просто удаление происходит позже, а не сразу.
About
Services
© EVILEG 2015-2020
Recommend hosting TIMEWEB