C++编程小心指针被delete两次
在C++类中,有时候会使用到传值调用(即使用对象实体做参数),当遇到这种情况,可要小心了!尤其是当你所传值的对象生命周期较长,而非临时对象(生命周期段)的时候。来看看下面的情况:
#include<iostream> usingnamespacestd; classText { private: char*str; public: Text(){str=newchar[20]; ::memset(str,0,20); } voidSetText(char*str) { strcpy(this->str,str); } char*GetText()const{returnstr;} ~Text() { cout<<"~TextDestruction"<<endl; delete[]str; cout<<"~TextOver"<<endl; } }; voidPrint(Textstr) { cout<<str.GetText()<<endl; } intmain() { Textt; t.SetText("abc"); Print(t); return1; }
上面执行的结果是程序崩溃了。原因是:
Print(Textstr)在对str进行复制构造的时候,没有进行深度拷贝;当Print退出的时候,因为是临时对象(函数初始时构造),对str进行析构,此时还没有出现任何问题;但回到main,继而退出main的时候,又对t进行析构,但此时t内的str中的内容已经被销毁。由于对一内存空间实施了两次销毁,于是就出现了内存出错。
解决方法如下:
重写前拷贝。像以下版本,不同的情况要作出适当的调整:
#include<iostream> usingnamespacestd; classText { private: char*str; public: Text(){str=newchar[20];::memset(str,0,20);} Text(Text&t) { str=newchar[20]; strcpy(str,t.GetText()); } voidSetText(char*str) { strcpy(this->str,str); } char*GetText()const{returnstr;} ~Text() { cout<<"~TextDestruction"<<endl; delete[]str; cout<<"~TextOver"<<endl; } }; voidPrint(Textstr) { cout<<str.GetText()<<endl; } intmain() { Textt; t.SetText("abc"); Print(t); return1; }
此处推荐不使用传值调用。就像下面书写如下Print版本:
voidPrint(Text&str) { cout<<str.GetText()<<endl; }
除非对象内所有的成员读属非指针内存内容,那么谨慎使用文章前面提到的用法。