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
Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.Вам это нравится? Поделитесь в социальных сетях!
Комментарии
Только авторизованные пользователи могут публиковать комментарии.
Пожалуйста, авторизуйтесь или зарегистрируйтесь
Пожалуйста, авторизуйтесь или зарегистрируйтесь
AD
- Akiv Doros
- 12 ноября 2024 г. 1:58
C++ - Тест 004. Указатели, Массивы и Циклы
- Результат:50баллов,
- Очки рейтинга-4
m
- molni99
- 26 октября 2024 г. 11:37
C++ - Тест 004. Указатели, Массивы и Циклы
- Результат:80баллов,
- Очки рейтинга4
m
- molni99
- 26 октября 2024 г. 11:29
C++ - Тест 004. Указатели, Массивы и Циклы
- Результат:20баллов,
- Очки рейтинга-10
Последние комментарии
Django - Урок 055. Как написать функционал auto populate field Freckles because of several brand names retin a, atralin buy generic priligy
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…
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…
Алгоритм сортировки кучей The role of raloxifene in preventing breast cancer priligy precio
PyQt5 - Урок 006. Работа с QTableWidget buy priligy 60 mg 53 have been reported by Javanovic Santa et al
Сейчас обсуждают на форуме
добавить qlineseries в функции buy priligy senior brother Chu He, whom he had known for many years
Всё ещё разбираюсь с кешем. 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
IscanderChe1 ноября 2024 г. 1:43
Машина тьюринга // Начальное состояние 0 0, ,<,1 // Переход в состояние 1 при пустом символе 0,0,>,0 // Остаемся в состоянии 0, двигаясь вправо при встрече 0 0,1,>…
ИМ
Реализация навигации по разделам Спасибо Евгений!
Игорь Максимов3 октября 2024 г. 14:05
Привет.
Здесь не надо ничего такого делать, память здесь будет очищаться за счёт этой строчки
То есть ты берёшь владение материалом и геометрией на 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, то скорее всего проблема в том, что ты создаешь материал в начале метода. Старый скорее всего не удаляется, пока не была удалена нода, а просто заменяется, и остаётся висеть в памяти.
В общем я просто сделал вторйо метод который простог обнавляет цвет и материал, вроде сделал правильно.
Но вот на эмбедеде есл ия быстро переключаюсь межуд обьектами они начали как бы мерцать, у меня предположение что я что то еще должен добавить в метод но не пойму что.
Само собою на компе этого незаметно.
Попробуй не переустанавливать материал, а просто менять цвет