vector<T *>的析构有关问题

vector<T *>的析构问题
定义了一个类,类似于html中的tag解析,每个tag都有parent节点,有一个vector<A *> children保存当前的孩子节点
一共有2个构造函数,如果无参,初始化参数,调用第二构造函数添加孩子节点
第二构造函数接受一个父指针,构造时设置父指针,并递归调用自身添加孩子节点
现在构造出来的对象是ok的,已经尝试读取过文件内容,都是正确的

示例代码:

C/C++ code

class A{public:    int num;    A *parent;    vector<A *> children;    ...    // 无参数调用    A(void)    {        this->num = 3;        this->parent = 0;        for (int i=0;i<this->num;++i)            this->children.push_back(new A(this,this->num-1));    }    // 子构造函数,被上面的无参函数调用    A(A *a,int n)    {        this->num = n;        this->children->parent = a;        for (int i=0;i<this->num;++i)            this->children.push_back(new A(this,this->num-1));    }};

但是目前析构好像有问题,代码运行后报错,跟踪调试到下面的函数在xmemory中

C/C++ code

void deallocate(pointer _Ptr, size_type)        {    // deallocate object at _Ptr, ignore size        ::operator delete(_Ptr);        }

应该是析构函数出错了,这样的类结构析构的时候怎么析构?目前的写法是

C/C++ code

A::~A(void){    if (this->parent!=0) this->parent = 0;    for (int i=0;i<(int)this->children.size();++i)        this->children[i]->~A();    vector <A *> ().swap(this->children);}

初次使用STL容器,还是有些迷糊.析构函数从流程上好像是对的,比较郁闷

[解决办法]
楼主的问题主要就是利用了指针，你的这个析构函数是不是删除指针了？你分享了指针，但是那个内存只有一块，导致了多次析构。这是问题的关键。你可以用智能指针类来表示指针了。
[解决办法]
这里描述的是一个树状结构，应该是使用上出了问题。LZ是怎么删除其中一个节点的，比如在使用中，某一个节点分支不要了，是怎么把这个分支剔除出去的？

另外还有个问题this->children[i]->~A()会有内存泄露，不如直接delete this->children[i]。
[解决办法]
感觉也是直接调delete比较好，还有你最后一个swap没有意义，对象已经析构了，无所谓children里面是什么。
[解决办法]
楼主你构造函数的代码本身就写的有问题。
this->children->parent = a; 这里编译错误。
看不懂你想构造出一个什么东西。

[解决办法]
删除对象应该用delete啊，析构函数执行过后vector会自动释放内存的。。不需要你手动swap

查看更多 下一篇