为什么代码在析构函数中失败?

Question

我已经经历了类似于"为什么析构函数被调用两次？"的stackoverflow问题。我的问题可以是类似的问题，但有一点变化。运行以下代码时出现错误：

struct Employee{
char *name;
char *tag;
Employee *employee;
Employee(){
name = NULL;
tag = NULL;
employee = NULL;
}
//copy constructor
Employee(const Employee &obj){
cout << "Copy constructor called" << endl;
name = (char*)malloc(sizeof(char)*strlen(obj.name));
strcpy(name, obj.name);
tag = (char*)malloc(sizeof(char)*strlen(obj.tag));
strcpy(tag, obj.tag);
employee = (struct Employee*)malloc(sizeof(obj.employee));
employee = obj.employee;
}
//overloaded assignment operator
void operator = (const Employee &obj){
cout << "Assignment operator called" << endl;
if (this == &obj){
return;
}
strcpy(name, obj.name);
strcpy(tag, obj.tag);
employee = obj.employee;
}
//destructor
~Employee(){
cout << "Destructor called" << endl;
if (name != NULL){
cout << "Freeing name" << endl;
free(name);
name = NULL;
}
if (tag != NULL){
cout << "Freeing tag" << endl;
free(tag);
tag = NULL;
}
if (employee != NULL){
cout << "Freeing employee" << endl;
free(employee);
employee = NULL;
}
}
};
Employee createNode(){
Employee emp;
emp.name = (char*)malloc(sizeof(char)* 25);
strcpy(emp.name, "Alan");
emp.tag = (char*)malloc(sizeof(char)* 25);
strcpy(emp.tag, "Engineer");
emp.employee = (struct Employee*)malloc(sizeof(struct Employee));//just created memory, no initialization
return emp;
}
Employee get(){
//Employee emp = createNode();
//return emp;
return createNode();
}
int main(){
Employee emp = get();
getchar();
return 0;
}

我调试了代码，发现当 main 函数退出时第二次调用析构函数时会引发错误。

1( 我想知道为什么代码无法运行？

2( 是否有任何内存泄漏？

3( 如何正确修复错误释放内存？

提前谢谢。

更新：

根据三个规则，我还添加了一个复制构造函数并重载了赋值运算符。但是错误(表达式：_crtisvalidheappointer(puserdata((正在引发。在检查谷歌后，我可以看到一些正在发生堆损坏的地方。当我在 createNode(( 中评论结构成员员工的初始化时，我可以看到在尝试释放析构函数中的员工时引发的错误。所以我怀疑问题出在员工结构成员身上。请帮我解决这个问题。我正在使用Visual Studio进行调试和运行。

Answer 1

您的问题是您的类中缺少复制构造和赋值运算符。因此，您将多次释放类中的字符串。

Answer 2

刚刚尝试了您的代码，并发现了一些导致崩溃的问题：

1(strlen返回没有空终止符的字符串长度，但strcpy需要额外的字节，所以你的分配应该看起来像这样：

name = (char*)malloc(strlen(obj.name)+1);

2(当你复制员工时，你复制指针，所以你有内存泄漏和员工指针作为一个悬空的。

此外，malloc 不能与构造函数一起使用，因此在

employee = (struct Employee*)malloc(sizeof(obj.employee));

员工里面有垃圾。