delete对象的内存泄露问题

hgrany

看看父类的析构函数是不是没有写成虚函数.

w_anthony

我看LZ的意思，貌似是这样的：
class A {
public:
    int a;   
};
class B{
public:
    B() { pA = new A; }
    ~B() { printf("B::~B"); }
    Delete() {delete this; }
    A* pA;
};
int main(int, char**)
{
    B* pB = new B;
    //...
    pB->Delete();
    return 0;
}
如果确实是与此类似的话，pB->Delete()里面的delete this，只是把main里面的new B释放了，但是B::B()里面的new A却没有人释放，就算B::~B被调用，但这个new A是不会被自动释放的

wells-xu

说了这么多 感觉在猜灯迷 不如直接上代码靠谱

狗蛋

new和delete必须成对。谁写了new，谁就必须要保证对应的delete被调用。

幻の上帝

回复 24# 狗蛋


    placement new?

srdgame

我看LZ的意思，貌似是这样的：
class A {
public:
    int a;   
};
class B{
public:
    B() { pA ...
w_anthony 发表于 2011-11-07 11:20

    楼主不至于这么弱的吧

狗蛋

回复  狗蛋


    placement new?
幻の上帝 发表于 2011-11-07 15:58

    还没到这个程度

就是最基本的编程规范：new和delete必须成对。谁new的，谁就负责delete。

或者说，一切资源的分配和归还都要遵循这个原则：怎么申请的，怎么归还；谁申请的，就由谁归还（如果做不到，就写文档着重指出——能带些脏字的话效果更为显著 ）。

他这个问题很简单。外部对象析构了（有日志），内部对象还在。估计只要照着这个原则认真捋一捋，问题就找到了。


嗯，还有个地方一下子没想到：不要玩那么多的继承层次。如果玩的话，请保证基类析构函数是虚函数。否则……

llj

首先非常感谢大家热情的帮助。

这个问题， 在今天尝试了一些方法（比如， 把里面的成员对象问题先注释等），加调试代码， 代码review，最后把问题找到了。  代码类似这样的：

class A
{
public :
A() :count(1) {}

ref() { count++ }

unref() { --count; if( 0==count ) delete this; }

virtual ~A() {}

private :
Atomic count;
}

class B public A
{
public :
B() {}
virtual ~B() {}
}

main()
{
  A* b = new B() ;
b->unref();
}

这段代码这样析构是没有问题的。

但是， 在复杂环境下，比如多线程， 业务逻辑复杂度， 还有开发人员之间的沟通， 模块调用等等，

就会在使用过程中，倒置一些很奇怪的现象。


这次的这个问题， 就是在模块之间接口的使用，产生了一些问题， 倒置内存泄漏， 现象比较乱，不太好定位。

再次感谢大家！

cddesire

2楼说的有理 ，主要看它们的对应关系。

w_anthony

首先非常感谢大家热情的帮助。

这个问题， 在今天尝试了一些方法（比如， 把里面的成员对象问题先注释等 ...
llj 发表于 2011-11-07 21:29

有多线程的话，这样ref和unref本身就有问题