delete重复释放同一块内存报错要怎么修改呀(除了不用指针)#36课后小作业

ROCKLET

1. #include <iostream>
   
2. #include <string>
   
3. using namespace std;
   
4. 
   
5. class CStudent
   
6. {
   
7.         char* p_name;
   
8.         char sex;
   
9.         int id;
   
10.         int age;
    
11. 
    
12. public:
    
13.         CStudent() {};
    
14.         CStudent(const char* t_name, char t_sex, int t_id, int t_age):sex(t_sex), id(t_id), age(t_age)
    
15.         {
    
16.                 int len_p = strlen(t_name);
    
17.                 p_name = new char[len_p + 1];
    
18.                 memset(p_name, 0, len_p);
    
19.                 strcpy(p_name, t_name);
    
20.         }
    
21.         ~CStudent()
    
22.         {
    
23.                 cout << "Student " << *p_name << "\t has been deleted!" << endl;
    
24. 
    
25.                 if (p_name!= NULL)
    
26.                 {
    
27.                         delete[]p_name;
    
28.                         p_name = NULL;
    
29.                 }
    
30.         }
    
31. };
    
32. 
    
33. 
    
34. void test()
    
35. {
    
36.         CStudent stud_1("zhangsan", 'f', 1001, 20);
    
37.         CStudent stud_2;
    
38.         stud_2 = stud_1;
    
39. }
    
40. 
    
41. int main()
    
42. {
    
43.         test();
    
44.         return 0;
    
45. }

复制代码

zhengl33

stud_2 = stud_1; 这条语句错了，这样写会导致，stud_2.p_name =stud_1.p_name。然后stud_1析构的时候stud_1.p_name对应的内存被删除了，等到stud_2析构时，stud_2.p_name != NULL，然后调用delete，但是这块内存已经在stud_1析构时已经delete了，所以会报错。
如果一定要stud_2 = stud_1这样写，请重载 =，具体请看重载课程。
或者在test函数内这样写。
void test()
{
        CStudent stud_1("zhangsan", 'f', 1001, 20);
        CStudent stud_2;
        stud_2 = stud_1;

        int len = strlen(stud_1.p_name);               ///计算stud_1.p_name长度
        stud_2.p_name = new char[len + 1]{0};   ///为stud_2.p_name申请内存
        strcpy(stud_2.p_name, stud_1.p_name);  ///为stud_2.p_name复制内容
}

ROCKLET

zhengl33 发表于 2021-12-20 15:01
stud_2 = stud_1; 这条语句错了，这样写会导致，stud_2.p_name =stud_1.p_name。然后stud_1析构的时候stud ...

谢谢！看来是需要进行运算符重载