碰到一个内存释放的问题,望高手看看..搞不定啊~~~~谢谢

superseiya

碰到一个内存释放的问题,望高手看看..搞不定啊~~~~
源代码如下:
#include <iostream.h>
#include <vector>
using namespace std;
class T1
{
public:
int x,y;
};

class Test
{
public:
int x,y;
T1* iptr;

Test():x(1),y(1)
{
iptr=0;
}

    ~Test()
{
if(iptr)
delete iptr;  ///这里老是出错..
}
};

vector<Test> vc;

void test()
{
   Test tt;
           tt.iptr=new T1;
vc.push_back(tt);
}
void main()
{
test();
}

自由蜂

void test()
{
   Test tt;
   tt.iptr=new T1;
   vc.push_back(tt);
}
其执行流程如下：
1。调用Test构造函数，构造tt，iptr = 0;
2。创建T1，iptr = &T1
3。vc创建一个自身数据Test，复制tt的数据到里面，即Test data = tt;
4。tt释构，delete iptr,把T1删除了，但vc中的数据的指针iptr仍指向T1。
5。main（）结束时，vc释构，再次要调用Test的释构函数，delete iptr，出错，因为对象已经删除。
End。
请注意vector的执行。

superseiya

谢谢啊 ~~~

iverson

vector 里尽量放指针，这样效益更高。

Enigmaya

浅拷贝引起的

另外：
if(iptr) 这句不是必须的，delete可以处理null的情况
delete iptr;

千里马肝

自由蜂正解

PS，std::container是以value copy的形式在操作，所以，请确定这种行为是你所期望的。

lwevil

[em1]

luoqi

void test()
{
   Test tt;
           tt.iptr=new T1;
vc.push_back(tt);
}

tt是临时变量, 在离开此函数之前会被释构, 所以iptr所指对象会被删除.
同时, iptr会成为野指针, 在再次使用时就会exception,,
最好在delete iptr时, 使iptr=0;

push_back(class), 会产生, 很多你意想不到的后果, 效率问题(class会自复制), 及自复制产生的问题... 就不多说了,
所以最好改成& or *, 且最好加上 const

其实LZ要做的是, 学习一下<<C++高质量编程>>
当一个对象具有指针时, 要谨慎他的 赋值构造函数 及 COPY构造函数.
因为编译器在生成对象时, 如果你没有这两个函数, 会自动生成一个,
所以, 该对象的指针是会被复制的, 但指针所指的内容不会被复制.