如何正确的释放类中动态创建的内存？

oncreating

直接上代码，红色注释是我的问题：

#include <vector>

using namespace std;

struct Point
{
        double x;
        double y;
        Point(){x=121.12345;y=31.54321;}
};

class Line
{
public:
        Line(int num)
        {
                length = 0;
                pt_num = num;
                pt     = new Point[num];
        }
        ~Line()
        {
                // 如何正确销毁？
                if (pt)
                {
                        delete [] pt;
                        pt = NULL;
                }
        }

public:
        int length;
        int pt_num;
        Point *pt;
};

typedef vector<Line> Polyline;

int _tmain(int argc, _TCHAR* argv[])
{
        Polyline polyline;

        {
                Line line(4);
                polyline.push_back(line);
        }

        // use polyline...
        // 但此时line中的pt已被销毁

        return 0;
}

hellioncu

你需要给Line增加拷贝构造函数

folklore

回复 2# hellioncu


    直接上 std::sharedptr就好了。

framily

我是来打酱油的

流氓无产者

hellioncu 发表于 2015-07-20 22:11
你需要给Line增加拷贝构造函数

貌似最近很用功啊

VIP_fuck

还是sharedptr吧，不然你还得考虑异常之类的。

lxyscls

        {
                Line line(4);
                polyline.push_back(line);
        }
把括号注释掉

oncreating

非常感谢大家，修改了一下，能达到我目的了


#include <vector>

using namespace std;

struct Point
{
        double x;
        double y;
        Point(){x=121.12345;y=31.54321;}

        Point& operator=(const Point& rhs)
        {
                this->x = rhs.x;
                this->y = rhs.y;
                return *this;
        }
};

class Line
{
public:
        Line(){pt_num=0;pt=NULL;}
        Line(int num)
        {
                length = 0;
                pt_num = num;
                pt     = new Point[num];
        }
        Line(const Line& rhs)
        {
                if (this == &rhs)
                {
                        return;
                }

                length = rhs.length;
                pt_num = rhs.pt_num;
                pt = new Point[rhs.pt_num];
                for (int i = 0; i < rhs.pt_num; i++)
                {
                        pt[i] = rhs.pt[i];
                }
        }
        ~Line()
        {
                if (pt)
                {
                        delete [] pt;
                        pt = NULL;
                }
        }

        Line& operator = (const Line& rhs)
        {
                if (this == &rhs)
                {
                        return *this;
                }

                length = rhs.length;
                pt_num = rhs.pt_num;
                pt = new Point[rhs.pt_num];
                for (int i = 0; i < rhs.pt_num; i++)
                {
                        pt[i] = rhs.pt[i];
                }

                return *this;
        }

public:
        int length;
        int pt_num;
        Point *pt;
};

typedef vector<Line> Polyline;

int _tmain(int argc, _TCHAR* argv[])
{
        {
                Polyline polyline;

                {
                        Line line(2);
//                         line.pt[0].x = 11;
//                         line.pt[0].y = 22;
//                         line.pt[1].x = 33;
//                         line.pt[1].y = 44;

                        polyline.push_back(line);
                }

        }

        return 0;
}