vector的析构问题

[libralibra]

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

示例代码:

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中

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

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

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容器,还是有些迷糊.析构函数从流程上好像是对的,比较郁闷

[libralibra]

没人看?

[W170532934]

楼主的问题主要就是利用了指针，你的这个析构函数是不是删除指针了？你分享了指针，但是那个内存只有一块，导致了多次析构。这是问题的关键。你可以用智能指针类来表示指针了。

[libralibra]

请问分享指针是什么意思?析构函数从道理上来讲是可以的,但是目前的确出错在析构
这个类写了很长了,改起来太费劲了,是不是必须智能指针?有没有不用智能指针的解决办法?

[Mario_Q]

楼主你构造函数的代码本身就写的有问题。
this->children->parent = a; 这里编译错误。
看不懂你想构造出一个什么东西。

[beckhanyan]

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

[mars_man]

这里描述的是一个树状结构，应该是使用上出了问题。LZ是怎么删除其中一个节点的，比如在使用中，某一个节点分支不要了，是怎么把这个分支剔除出去的？

另外还有个问题this->children[i]->~A()会有内存泄露，不如直接delete this->children[i]。

[cryingbee]

感觉也是直接调delete比较好，还有你最后一个swap没有意义，对象已经析构了，无所谓children里面是什么。