如何删除列出的“ QGraphicsPathItem”对象以控制进程的内存使用?

时间:2018-10-13 20:00:42

标签: c++ qt qgraphicsview qgraphicsitem qlist

我想在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 :(得分:3)

您的代码有几个错误:

  • 为什么要创建QPainterPath指针?由于addPath()会获取该值的副本,因此不必永久拥有该信息。

  • 为什么要添加2个具有相同myPath?的项目,只需一项就足够了,并将其存储在列表中。

  • 为什么创建2个pathItem?使用addPath()创建一个pathItem并使用new QGraphicsPathItem创建另一个,则不需要第二个。

  • 为什么要在循环中使用scene->addItem(PathItemList.back());,这没有意义。

请记住,创建指针时,您的任务是确保在关闭应用程序之前消除内存。

考虑到上述情况,代码应如下:

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