一个关于内存回收的问题

[Julykey]

假设有一个类：Line ，它包含了Point 类的头文件
然后Line里成员变量vector <Point*> m_neighborPoint以及下面这个函数

for() //将很多的点放到m_neighborPoint中
{
Point* tempPoint=new Point();
...........//这里是对tempPoint赋值
m_neighborPoint.push_back(tempPoint);
..........
//delete tempPoint;
//tempPoint=NULL;
}
当函数中没有最后两句时，程序正常运行。而添加里以后就报内存错误了

[珍惜生命远离CPP]

m_neighborPoint是干啥的,是不是另外又释放了一次?

[Julykey]

[Quote=引用 1 楼 akirya 的回复:]
m_neighborPoint是干啥的,是不是另外又释放了一次?
[/Quote]
是Line类的成员变量，它将在下面的程序中被访问。我并没有显式的去释放。析构函数中也没有

[y_l_stone]

释放后哪里又访问了？

[whg01]

delete tempPoint; 这句引起的。
m_neighborPoint.push_back(tempPoint); 这里把tempPoint的值添加到了vector里，即vector里增加了一个指针，指向的地址与tempPoint指向的地址相同。
你delete tempPoint，会把tempPoint指向的地址释放。
这样vector里相应的指针就指向了已释放的内存。

[Julykey]

[Quote=引用 4 楼 whg01 的回复:]
delete tempPoint; 这句引起的。
m_neighborPoint.push_back(tempPoint); 这里把tempPoint的值添加到了vector里，即vector里增加了一个指针，指向的地址与tempPoint指向的地址相同。
你delete tempPoint，会把tempPoint指向的地址释放。
这样vector里相应的指针就指向了已释放的内存。
[/Quote]
好像是这个原因。但是这些tempPoint不用释放内存了吗？
是不是我设计的不好，将Line类的成员变量改成vector <Point> m_neighborPoint是不是可以避免这个问题？

[vincent_nicole]

同意4楼
一样需要释放，需要合适的位置释放就可以了,比如在析构函数里面
vector<Point*>::iterator it = m_neighborPoint.begin();
while ( it != m_neighborPoint.end )
{
delete (*it);
++it;
}

[vincent_nicole]

你自己new出来的对象，要记住delete掉
你放对象在容器里面就不用了，变量的生命周期结束后就会释放了所占用的空间

[十秒耿直拆包选手]

[Quote=引用 7 楼 cryinwind 的回复:]
你自己new出来的对象，要记住delete掉
你放对象在容器里面就不用了，变量的生命周期结束后就会释放了所占用的空间
[/Quote]
同意6#的说法。c++书中都强调new必要对应一个delete，感觉7#这样会造成内存泄漏，不知是否正确，楼下解答！

[ahao]

。。。你都delete了，还指望不出错？

不需要的时候，再遍历vector，把里面的指针delete

[whg01]

[Quote=引用 9 楼 ahao 的回复:]
。。。你都delete了，还指望不出错？

不需要的时候，再遍历vector，把里面的指针delete
[/Quote]
需要释放时，再从vector里读取指针，然后释放。

[hjs2009]

这样的设计可能存在问题，问题的根源可能在于你申请的与释放的不对称。

解决的方法建议：
重新实现构造与析构函数，保证这二个函数对称。

然后保证 new 与 delete 的对称使用。

在C++中基本上就不会出现以上的内存问题那。

[gyang_pinner]

[Quote=引用 5 楼 julykey 的回复:]
引用 4 楼 whg01 的回复:
delete tempPoint; 这句引起的。
m_neighborPoint.push_back(tempPoint); 这里把tempPoint的值添加到了vector里，即vector里增加了一个指针，指向的地址与tempPoint指向的地址相同。
你delete tempPoint，会把tempPoint指向的地址释放。
这样vector里相应的指针就指向了已释放的内存。

好像是这个原因。但是这些tempPoint不用释放内存了吗？
是不是我设计的不好，将Line类的成员变量改成vector <Point> m_neighborPoint是不是可以避免这个问题？
[/Quote]
这样会增加开销，但更安全

[TADICAN]

如果想要效率，又要改变现有的程序设计，你可以考虑一下用智能指针来管理内存。

[ahao]

[Quote=引用 13 楼 tadican 的回复:]
如果想要效率，又要改变现有的程序设计，你可以考虑一下用智能指针来管理内存。
[/Quote]

恩，我一般就用boost::shared_ptr或者boost::ptr_container
不过，用了这个，估计LZ头会更大的。

[cmo_lyl]

AUTO_PTR