BlinCTApril 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
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!
SH
- Sak Hax
- April 25, 2024, 2 p.m.
C++ - Test 001. The first program and data types
- Result:33points,
- Rating points-10
l
- laei
- April 23, 2024, 9:19 a.m.
C ++ - Test 004. Pointers, Arrays and Loops
- Result:10points,
- Rating points-10
Last comments
Qt Linux - Lesson 001. Autorun Qt application under Linux как сделать автозапуск для флэтпака, который не даёт создавать файлы в ~/.config - вот это вопрос ))
АК
Qt WinAPI - Lesson 007. Working with ICMP Ping in Qt Без строки #include <QRegularExpressionValidator> в заголовочном файле не работает валидатор.
Анатолий КононенкоFeb. 5, 2024, 1:50 a.m.
EVADec. 25, 2023, 10:30 a.m.
Boost - static linking in CMake project under Windows Сделал всё по-как у вас, но выдаёт ошибку [build] LINK : fatal error LNK1104: не удается открыть файл "libboost_locale-vc142-mt-gd-x64-1_74.lib" Хоть убей, не могу понять в чём дел…
Qt/C++ - Lesson 056. Connecting the Boost library in Qt for MinGW and MSVC compilers Для решения твой проблемы добавь в файл .pro строчку "LIBS += -lws2_32" она решит проблему , лично мне помогло.
Now discuss on the forum
DA
Unlock Your Aesthetic Potential: Explore MSC in Facial Aesthetics and Cosmetology in India Embark on a transformative journey with an msc in facial aesthetics and cosmetology in india . Delve into the intricate world of beauty and rejuvenation, guided by expert faculty and …
Dr Gangil AcademicsApril 20, 2024, 7:45 a.m.
Мобильное приложение на C++Qt и бэкенд к нему на Django Rest Framework Евгений, добрый день! Такой вопрос. Верно ли следующее утверждение: Любое Android-приложение, написанное на Java/Kotlin чисто теоретически (пусть и с большими трудностями) можно написать и на C+…
Павел ДорофеевApril 14, 2024, 2:35 a.m.
Вернуть старое поведение QComboBox, не менять индекс при resetModel Добрый день! У нас много проектов в которых используется QComboBox, в версии 5.5.1, когда модель испускает сигнал resetModel, currentIndex не менялся. В версии 5.15 при resetModel происходит try…
Привет.
Здесь не надо ничего такого делать, память здесь будет очищаться за счёт этой строчки
То есть ты берёшь владение материалом и геометрией на pNode, а сама pNode должна будет уничтожаться дальше в дереве объектов OpenGL, попутно уничтожая материал и геометрию. QML всё это предусматривает.
Если по какой-то причине всё-таки происходит утечка памяти, то значит косяк где-то в логике метода updatePaintNode
Для проверки можно сделать для всех этих классов наследников и переопределить деструктор с выводом qDebug, как сделано в этой статье для QStandardItem . Это позволит убедиться, что удаление всё-таки происходит.
Оки спасибо, сейчас проверю.
В общем все таки утечка есть, по причине логики метода где вызывается данная функция.
Вот так она выглядит
То есть при первом входе создаюется обьект и вызывается lineNode для создания линий, а потом когда уже нода существует то мы снова заходим в метод и опять делаем new.
Так что вот с этим и проблема как раз.
Проблема в том, что если нода уже была создана, то ты заново пересоздаешь ноды, а не перерисовываешь. Посмотри как сделано в этой статье QML - Урок 032. Создаём Custom QuickItem из C++ с использованием средств OpenGL
Я не создаю там новые объекты, если oldNode существует, я перерисовываю ноду в уже существющих объектах. А ты просто создаешь каждый раз новые объекты m_pNodeLineOne, m_pNodeLineTwo, m_pNodeLineThree, m_pNodeLineFour если oldNode . Либо удаляй их, если они существуют и потом создавай новые, либо переопределяй некоторые свойства. Переопределение наверняка будет производительнее, чем создавать новые объекты.
В общем тебе нужен ещё один метод, который будет менять точки, то есть менять геометрию.
хотя, если вниметльно посмотреть на метод lineNode, то скорее всего проблема в том, что ты создаешь материал в начале метода. Старый скорее всего не удаляется, пока не была удалена нода, а просто заменяется, и остаётся висеть в памяти.
В общем я просто сделал вторйо метод который простог обнавляет цвет и материал, вроде сделал правильно.
Но вот на эмбедеде есл ия быстро переключаюсь межуд обьектами они начали как бы мерцать, у меня предположение что я что то еще должен добавить в метод но не пойму что.
Само собою на компе этого незаметно.
Попробуй не переустанавливать материал, а просто менять цвет