Как удалить указанный объект «QGraphicsPathItem» для управления использованием памяти процесса? - PullRequest
Новая
       74

Как удалить указанный объект «QGraphicsPathItem» для управления использованием памяти процесса?

0 голосов
/ 13 октября 2018

Я хочу нарисовать несколько путей в QGraphicsView.Я хочу управлять утопающими путями.Таким образом, в методе path_ploter(double) я рисую путь и передаю указатель объекта на PathItemList.

Я перерисовываю пути в слоте plot_fn().Я подключаю этот слот к кнопке.Когда я нажимаю кнопку plot, несмотря на удаление всех указателей элементов пути в списке перед путями перерисовки, использование памяти процессом увеличивается.Если я удаляю указатель в списке, как только в созданном, например, //delete PathItemList.back(); в path_ploter(double) методе, память использования процесса не увеличивается.В этом случае пути не отображаются.почему увеличивается использование памяти при удалении указателя в слоте plot?Как я могу контролировать это?

в заголовочном файле: 

#pragma once

#include <QtWidgets/QMainWindow>
#include "ui_QtGuiApplication1.h"
#include <QGraphicsView>
#include <QGraphicsPathItem>
#include <QGraphicsLineItem>
#include <QtWidgets/QPushButton>

class QtGuiApplication1 : public QMainWindow
{
    Q_OBJECT

public:
    QtGuiApplication1(QWidget *parent = Q_NULLPTR);

private:
    Ui::QtGuiApplication1Class ui;
    QGraphicsView* qGraph;
    QGraphicsScene* scene;
    QGraphicsPathItem* pathItem = new QGraphicsPathItem();
    QList<QGraphicsPathItem *>PathItemList;
    void path_ploter(double);
private slots:
    void plot_fn();
};

в исходном файле: 

#include "QtGuiApplication1.h"

QtGuiApplication1::QtGuiApplication1(QWidget *parent)
    : QMainWindow(parent)
{
    ui.setupUi(this);

    //////////////
    qGraph = new QGraphicsView(ui.centralWidget);
    qGraph->setGeometry(QRect(0, 0, 300, 300));
    scene = new QGraphicsScene(qGraph);
    scene->setSceneRect(0, 0, 300, 300);
    qGraph->setScene(scene);    
    qGraph->show();
    ////////////
    QPushButton* btn_Ok = new QPushButton(ui.centralWidget);
    btn_Ok->setObjectName(QStringLiteral("btn_Ok"));
    btn_Ok->setGeometry(QRect(300, 300, 75, 23));
    btn_Ok->setText("Plot");
    connect(btn_Ok, SIGNAL(clicked()), this, SLOT(plot_fn()));
    //plot_fn();
    //path_ploter(1);
}

void QtGuiApplication1::path_ploter(double alpha)
{
    QPainterPath* myPath = new QPainterPath();
    QPolygon pol;

    QPen graphPen;
    graphPen.setColor(QColor(255, 0, 0, 255));
    graphPen.setWidth(2);


    QPoint pos;

    for (size_t i = 0; i < 100; i++)
    {

        pos.setX(i);
        pos.setY(i*alpha);
        pol.append(pos);
    }

    myPath->addPolygon(pol);
    pol.clear();


    pathItem = scene->addPath(*myPath, graphPen);
    PathItemList << pathItem;
    //delete PathItemList.back();
    pathItem = new QGraphicsPathItem();

}

void QtGuiApplication1::plot_fn()
{

    for (size_t i = 0; i < PathItemList.size(); i++)
    {
        scene->removeItem(PathItemList.at(i));
    }
    qDeleteAll(PathItemList.begin(), PathItemList.end());
    PathItemList.clear();

    for (size_t i = 0; i < 3; i++)
    {
        path_ploter(i+1);

    }

}

1 Ответ

0 голосов
/ 13 октября 2018

Ваш код содержит несколько ошибок:

  • Почему вы создаете указатель QPainterPath?Нет необходимости иметь эту информацию постоянно, поскольку addPath() берет копию этого значения.

  • Почему вы добавляете 2 элемента с одинаковым myPath?, Достаточно только одногои сохраните его в списке.

  • Почему вы создаете 2 pathItem?создайте pathItem с addPath(), а другой с new QGraphicsPathItem, второй не требуется.

  • Почему вы используете scene->addItem(PathItemList.back()); в цикле for?, он не делаетсмысл.

Помните, что при создании указателя ваша задача - убедиться, что память будет удалена до закрытия приложения.

Учитывая вышеизложенное,код должен быть следующим: 

void QtGuiApplication1::path_ploter(double alpha)
{
    QPainterPath myPath;
    QPolygonF pol;
    QPen graphPen;
    graphPen.setColor(QColor(255, 0, 0, 255));
    graphPen.setWidth(2);
    for (size_t i = 0; i < 100; i++){
        pol.append(QPointF(i, i*alpha));
    }

    myPath.addPolygon(pol);
    QGraphicsPathItem *pathItem = scene->addPath(myPath, graphPen);
    PathItemList << pathItem;
}

void QtGuiApplication1::plot_fn()
{
    // Remove the item from the scene
    for(QGraphicsPathItem * pathItem: PathItemList){
        scene->removeItem(pathItem);
    }
    // Delete the memory pointed by the pointers
    qDeleteAll(PathItemList.begin(), PathItemList.end());
    // Clean the container
    PathItemList.clear();
    for (size_t i = 0; i < 3; i++)
    {
        path_ploter(i+1);
    }
}

Наконец, рекомендуется использовать новый синтаксис подключения: 

connect(btn_Ok, &QAbstractButton::clicked, this, &QtGuiApplication1::plot_fn);
...