析构函数delete指针报错

Met32 · 发表于 2020-8-11 21:36:58

#include<iostream>
#include<string>
using namespace std;
class person{
public:
        char* pname;
                person(char *name)
                {       
                int len = strlen(name);
                pname = new char(len+1);
                memset(pname,0,len+1);
                strcpy(pname,name);
                cout << pname <<endl;
                }
                ~person(){
                        if(pname){
                                delete[] pname;
                                cout << "析构调用";
                        }
                }
};
int main(){
       
       
        {
                person st1("123231321231312123312");
                person st2("213123");
        }
        system("pause");
        return 0;
}

复制代码

在类中我new了一块空间pname，之后析构进行delete删除掉。
如上代码我实例化了两个，一个st1 一个st2
问下各位前辈，为什么我的st2一旦调用delete就出现错误呢。。
难道是 st1和st2的pname是一块内存空间吗？
不应该是如下图这样的吗？我实例化一个对象构造函数就new一块空间
析构函数delete指针报错

tony666 · 发表于 2020-8-11 23:46:42

拷贝构造函数没有给pname 赋值，pname默认值并不是 0 ，（vs好像都是 0xcdcdcdcd） ,
所以析构报错了

Met32 · 发表于 2020-8-11 21:38:02

代码如下：

#include<iostream>
#include<string>
using namespace std;
class person{
public:
char* pname;
person(char *name)
{
int len = strlen(name);
pname = new char(len+1);
memset(pname,0,len+1);
strcpy(pname,name);
cout << pname <<endl;
}
person(person &s){
cout << "拷贝构造调用"<<endl;
}
~person(){
if(pname){
//delete[] pname;
cout << "析构调用";
}
}
};
int main(){
{
person st1("123231321231312123312");
person st2(st1);
}
system("pause");
return 0;
}

复制代码

gkz525 · 发表于 2020-8-12 09:37:23

你拷贝后就有两个pname了（名字相同），当st1释放指针pname时，凡是叫pname的就都被释放了（所以两个pname都释放了），所以当st2来释放属于他的pname时，就找不到了。

User_Ghost · 发表于 2020-8-13 10:27:14

本帖最后由 User_Ghost 于 2020-8-13 10:31 编辑

调试的时候发现，st2的pname的地址是0x8，并不为0，所以析构函数才会按照st1的方式delete，所以应该在拷贝构造函数里给pname赋为空地址NULL，那么pname就是0x0了，也就不会delete了。构造函数因为有重载，所以执行完拷贝构造函数以后，不会执行其余构造函数，因此pname在构造的时候并没有动态分配空间。但至于为什么pname不为0，我个人认为应该是系统在指针变量定义时是随机分配地址，因此各种书籍上都建议在指针变量定义后初始化为空地址。

char *pname;
person(char *name)
{
int len = strlen(name);
pname = new char[len + 1];
memset(pname, 0, len + 1);
strcpy(pname, name);
cout << pname << endl;
}
person(person &s)
{
cout << "拷贝构造调用" << endl;
}
~person()
{
if (pname)
{
delete[] pname;

复制代码

JLDawson · 发表于 2020-8-13 16:01:16

错误原因：
1.楼主申请自由存储区的代码为：pname = new char(len+1);
只申请了一个字节的空间。
可更改为：pname = new char[len + 1];
2.拷贝函数中未对pname做任何操作，会出现随机值。如果为空不会进行delete。如果未野指针，那就会报错了。

帐号		自动登录	找回密码
密码			加入驿站

析构函数delete指针报错

站长提醒 /2