转载自:http://blog.csdn.net/jxluofeng/article/details/19766801
《问题》危险的代码:
int* p=new int(1);
delete p;
delete p;
探讨一:
连续两次对同一个指针delete ,会造成严重的错误。编译器会检测出这样的错误吗?或许一些编译器会的,但别太过指望编译器。
探讨二:
第一次delete后,p自动为空(NULL)了吗?不是的。
探讨三:
在delete之前会自动检查p是否为空(NULL),如果为空(NULL)就不再delete了吗?确实是如此。
探讨四:
删除为空(NULL)的指针是不会有任何问题的吗?确实是如此。
探讨五:
#define SAFE_DELETE(p) delete (p); p = 0;
这样就就万事大吉了吗?好像不是的。
delete p+1;//在C++中是正确的
SAFE_DELETE(p+1)将会导致错误
探讨六:
没有好的方法解决重复释放这样的问题,只能靠程序员的细心了。
《结论》安全的代码:
int* p=new int(1);
delete p;
p = NULL;
(1)delete 一次以后,p成了野指针,它作为地址的值还是有效地没还可以访问它以前指向的内存,不过那片内存被重新格式化了;
(2)p不等于NULL,用 if(p) 语句不能判断它指向的内存是否有效(此时它指向的内存无效,p本身有效);
(3)delete 一次以后,不能再次delete,否则会报错;
(4)此时如果误用p指针,仍然可以修改内存的值和从该处取出数值,但此时数据不受保护,该内存空间可能被重新被分配给别的变量;
(5)如果p指向的空间再次被new函数分配,即使是分配给别的指针,即使分配大小与原来不一样,p又恢复了效力,可以改变内存的值,甚至可以重新被delete,p的作用与新分配的指针一样;
转载自: C++ 一不小心被delete两次
2012-09-11 12:10 by 捣乱小子, 2720 阅读, 31 评论, 收藏, 编辑
2012-09-11 18:04:看到园里朋友们这么热心,有那么一会心里好兴奋。:)
2012-09-11 下午:本文曾出现很严重的错误,但不要紧,感谢园友们的即使指正。
C++类中,有时候使用到传值调用(对象实体做参数),遇到这种情况,可要小心了!特别是当你所传值的对象生命周期较长,而非临时对象(生命周期段)的时候。来看看下面的情况:
#include <iostream> using namespace std; class Text { private: char * str; public : Text(){str = new char [20];::memset(str,0,20);} void SetText( char * str) { strcpy(this->str,str); } char * GetText() const{ return str;} ~Text() { cout << "~Text Destruction" << endl; delete [] str; cout << "~Text Over" << endl; } }; void Print(Text str) { cout << str.GetText() << endl; } int main() { Text t; t.SetText( "abc" ); Print(t); return 1; } |
上面执行的结果程序崩溃了。原因:
Print(Text str)在对str进行复制构造的时候,没有进行深度拷贝;当 Print退出的时候,因为是临时对象(函数初始时构造),对str进行析构,此时还没有任何破绽;但回到main,继而退出main 的时候,又对t进行析构,但此时t内的str中的内容已经被销毁。由于对一内存空间实施了两次销毁,于是出现内存出错。
解决方法:
- 重写浅拷贝。像一下版本,不同的情况要作出适当的调整:
#include <iostream>
using
namespace
std;
class
Text
{
private
:
char
* str;
public
:
Text(){str =
new
char
[20];::
memset
(str,0,20);}
Text(Text &t)
{
str =
new
char
[20];
strcpy
(str,t.GetText());
}
void
SetText(
char
* str)
{
strcpy
(
this
->str,str);
}
char
* GetText()
const
{
return
str;}
~Text()
{
cout <<
"~Text Destruction"
<< endl;
delete
[] str;
cout <<
"~Text Over"
<< endl;
}
};
void
Print(Text str)
{
cout << str.GetText() << endl;
}
int
main()
{
Text t;
t.SetText(
"abc"
);
Print(t);
return
1;
}
- (推荐)不使用传值调用。就像下面书写如下Print版本:
void
Print(Text &str)
{
cout << str.GetText() << endl;
}
- 评论中的小伙伴还提供了一个方法,很值得推荐,同时这也是编程良好的习惯:指针在使用的时候需要作空判断,在删除指针时候需要置空。
后记——语言的探讨
C++,以至于其他的程序语言,都是伟大的创造。他们是规则的世界,一愣一脚皆规则。当对某一规则有所熟知之后,就似发现新大陆似的。但想想,这些规则可经不起碰撞打击,殃及的是coder,是我们。这些规则不像1+1=2,“三点定面”来的铁定,总之比起数学理论来说,似乎程序规则脆弱很多,总之在它们身上看到的价值是有限的,总我还是选择了计算机...为之奈何,不得而知。