loop_in_codes

低调做技术__欢迎移步我的独立博客 codemaro.com 微博 kevinlynx

C++陷阱:virtual析构函数

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

class Base {
public:
    ~Base();
};

class Derived : public Base {
privated:
    std::vector<int> m_data;    
}; Base *obj = new Derived(); delete obj;

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

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

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

posted on 2012-09-13 17:31 Kevin Lynx 阅读(4511) 评论(8)  编辑 收藏 引用 所属分类: c/c++

评论

# re: C++陷阱:virtual析构函数 2012-09-13 17:49 preboy

到目前为止还没有犯过这错误,不过需要记住以后不犯这个错误  回复  更多评论   

# re: C++陷阱:virtual析构函数 2012-09-13 19:29 test

如果编译器发现被继承了。那么直接把那个类的~设置为virtual如何。  回复  更多评论   

# re: C++陷阱:virtual析构函数 2012-09-14 09:02 Kevin Lynx

@test
继承不一定会使用到多态  回复  更多评论   

# re: C++陷阱:virtual析构函数[未登录] 2012-09-15 09:36 korall

至少能给出一个警告  回复  更多评论   

# re: C++陷阱:virtual析构函数 2012-09-17 09:16 zaccheo

这个不能归咎于c++的陷阱吧  回复  更多评论   

# re: C++陷阱:virtual析构函数 2012-09-17 18:54 yrj

Visual C++ 有 C4265 警告,g++ 有 -Wnon-virtual-dtor  回复  更多评论   

# re: C++陷阱:virtual析构函数 2014-05-10 21:49 YJj

学习到了 thanks  回复  更多评论   

# re: C++陷阱:virtual析构函数 2016-05-06 16:21 小键

只要你觉得你的类会被继承,那么最好给析构函数加上virtual

--这并不是无脑遵守好吗,这非常有道理呀。  回复  更多评论   


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