问两个关于内存泄漏的问题？

[zhvfeng]

1：下列代码是否有问题：
CPtrArray *array = new CPtrArray();
MyClass *myClass = new MyClass();
array.Add(myClass);
delete array;
myClass对象是否还暂用资源？

2：有一个Timer，调用时需要访问一个全局的CPtrArray对像，但这个CPtrArray对象可能会随时被一个新的对象替换，怎样能把旧的对象删除，直接delete有异常。这样也不行：
CPtrArray *temp = gl_array;
gl_array = newArray;
delete temp;
同样有错误。

[zhvfeng]

to s_gao(Simon):有可能就是这个原因，我再仔细看看。
谢谢大家帮忙，分不多，大家凑合一下吧！

[七伤拳]

1、有问题，CPtrArray 只是保存对象指针，移去一个元素时不会删除该对象，需要自己删除。

2、你是在哪里删除对象？和Timer有什么关系

[hahu]

temp=NULL;
delete temp;
只删除指针，不删除对象

[hahu]

可能我理解错了
可以先
delete gl_array;
gl_array=NULL;

[zhvfeng]

第一个问题：那使用集合岂不是很麻烦，还要遍历删除，大家有什么好方法？
第二个问题：我是指timer一直在运行，如果直接delete，timer访问不到gl_array，当然要抛异常，如果不删，又会引起内存泄漏，怎么办好？

[luoshayu]

CPtrArray *temp = gl_array;
gl_array = newArray;
delete temp;

这个东西有问题呀
gl_array有可能是一个非法的指针,temp当然也是非法的指针,以后改变gl_array的值时temp的值不会变,要删除当然会出错了

[bluebohe]

1:
当然占用资源，你应该先把数组清空,然后再删除。myClass对象可用
2：对象指针只要被删除，就给他赋值为空，如果为空，则不删除

[bluebohe]

1：你应该先遍历删除，再删除集合，才不会内存泄漏

[七伤拳]

1、如果你不想遍历删除的话可以使用STL里面的 auto_ptr

2、把你相关的代码帖上来，你 gl_array 有没有初始化为NULL的？最好应该这样子写：
CPtrArray *temp = gl_array;
gl_array = newArray;
if (NULL != temp)
{
delete temp;
temp = NULL;
}

[vollin]

一个常识或者说一种默认的做法。
1.定义一个指针时将其设初始化为NULL;
2.使用new 为指针分配内存或否；
3.每次使用完成或指针的作用域末先判断其是否为空，如不为空则delete.
如此可免于内存泄漏。
一般说来只有自己new的东西才会有内存泄漏，以上的方法可完全避免，但有时是因为系统（MFC）自己分配了东西，因为有异常没有delete，这时才需要去catch异常然后自己delete.

[vollin]

写漏了一点，第3点，在delete后将指针重新赋值为NULL

[vollin]

再有，除非是使用了所谓的智能指针，否则，只要是自己new的都要自己delete

[sharkhuang]

内存泄漏

[sharkhuang]

1.泄漏!
2.你应该*temp = newArray. temp是全局指针

[zhvfeng]

感谢大家发表自己的建议！
1：看来只能遍历删除了，不明白，MFC为什么不自动删除里面的东西。
2：是这样的，我有一个button，每点一次，gl_array就会指向一个新的对象，而这个过程中timer一直运行，所以原来的对象不知该怎么删除，我试过点button之前先停止timer，然后delete gl_array，再gl_array ＝ new CPtrArray(),还是有错误。

[七伤拳]

你的button单击事件处理是异步的吗？windows里面消息处理是排队的，你在处理按钮单击事件时是不会去处理WM_TIMER事件的。

问题是出在下面代码里的哪一句？报的是什么错？
CPtrArray *temp = gl_array;
gl_array = newArray;
delete temp;

[lu_zi]

对于集合来说，集合只是一个数据管理的功能，对于你是什么数据，是指针还是类实例，它都没有特殊的处理，它只负责保存你要保存的东西，至于你怎么使，是你的事。这是面象对象的一个特点，如果你有特殊应用可以作特殊的程序来完成。

[zhvfeng]

如果我这样注释：
CPtrArray *temp = gl_array;
gl_array = newArray;
//delete temp;
就一点问题都没有，问题当然出在delete上，不过不知道为什么会这样？

[lchlovely]

TO zhvfeng(追风):

多个指针指向同一个位置,删掉任意一个指针,其它指针指向的东西就没有了,
为了防止野指针,应该把他们置为NULL,

[vollin]

这里当然会错了，因为你这里根本就没有new 所有不需要delete
你这里只不过是一个指针赋值，如果按我介绍的方法，如果你把newArray删除了，那么，newArray已经被赋值为NULL,这样，你的temp也就NULL了.
要注意的是，你new 的如果是newArray,那为了免于错误你就应当尽可能的去delete　newArray,而不是temp。因为temp只相当于newArray的引用，它不应当负责delete。
这其实只是一个习惯的问题，只要你习惯以后，就根本不会再出现这样的问题了。
不要总是试图去delete 一个引用其它指针值的指针（虽然从程序上来说是没有问题的）。而应该delete一个你不再需要的对象，并将其指针赋为NULL，即使是在程序的结尾，这是一个好的习惯，如果你经常写程序就会发现。因为你以后会更改或增加你的程序，所以无论在什么地方，只要有delete就将其赋值为NULL.