面对现实,超越自己
逆水行舟,不进则退
posts - 269,comments - 32,trackbacks - 0

有一天有个同事在通过vld调试一个内存泄漏问题,折腾了很久然后找到我。我瞥了一眼他的代码,发现问题和我曾经遇到的一模一样:

 1 class Base {
 2 public:
 3     ~Base();
 4 };
 5 
 6 class Derived : public Base {
 7 privated:
 8     std::vector<int> m_data;    };
 9 
10 Base *obj = new Derived();
11 delete obj;

当然,实际代码比这个复杂得多(这也是导致从发现问题到找到问题耗费大量时间的原因)。vld在报内存泄漏时,当然报的位置是new的地方。这个同事检查了这个对象的整个生命周期,确定他正确地释放了这个对象。

问题的关键就在于:Base类的析构函数不是virtual。因为不是virtual,所以在对一个Base类型的指针进行delete时,就不会调用到派生类Derived的析构函数。而派生类里的析构函数会用于析构其内部的子对象,也就是这里的m_data。这样,就造成了内存泄漏。

这其实是一个很低级的失误。但毫不客气地说C++中有很多这种少个关键字或者代码位置不对就会造成另一个结果的例子。事实上,针对这些悲剧也有很多书提出一些准则来让大家去无脑遵守。例如针对这个例子,我就记得曾有书说,只要你觉得你的类会被继承,那么最好给析构函数加上virtual。

posted on 2012-09-14 13:54 王海光 阅读(407) 评论(0)  编辑 收藏 引用 所属分类: C++

只有注册用户登录后才能发表评论。
网站导航: 博客园   IT新闻   BlogJava   知识库   博问   管理