[已解决]关于运算符重载的问题

Mr-Shuaibi

1. #include <iostream>
   
2. using namespace std;
   
3. 
   
4. class student
   
5. {
   
6. public:
   
7.         char* pname;
   
8.         int num;
   
9.         char sex;
   
10.         int age;
    
11.         student(char* t_name, int t_num, char t_sex, int t_age)
    
12.         {
    
13.                 int len = strlen(t_name) + 1;
    
14.                 pname = new char[len];
    
15.                 memset(pname, 0, len);
    
16.                 strcpy(pname, t_name);
    
17.                 num = t_num;
    
18.                 sex = t_sex;
    
19.                 age = t_age;
    
20.         }
    
21. 
    
22.         student()
    
23.         {}
    
24.         
    
25.         ~student()
    
26.         {
    
27.                 if (pname)
    
28.                 {
    
29.                         delete[]pname;
    
30.                 }
    
31.                 cout << "析构已执行" << endl;
    
32.     }
    
33. 
    
34.         student& operator=(const student& stud)
    
35.         {
    
36.                
    
37.                 sex = stud.sex;
    
38.                 num = stud.num;
    
39.                 age = stud.age;
    
40.         
    
41. 
    
42.                  if (pname)                             //？？？？？？？？？？？？
    
43.                 {
    
44.                         delete[]pname;
    
45.                 }
    
46.                 int len = strlen(stud.pname) + 1;
    
47.                 pname = new char[len];
    
48.                 memset(pname, 0, len);
    
49.                 strcpy(pname, stud.pname);
    
50.                 return *this;
    
51.         }
    
52. };
    
53. 
    
54. int main()
    
55. {
    
56.          student zhangsan("zhangsan", 1001, 'm', 18);
    
57.         student wangwu;
    
58.         wangwu = zhangsan;
    
59.         return 0;
    
60. }

复制代码

按照syc所讲的，在=运算符重载的函数中加上和析构函数中一样的if语句，这时候在一步一步调试的时候就会执行到此处报错，说读取位置时发生访问冲突，所以就将重载中的if语句删了，这样就能正常执行了，这是为啥？

最佳答案

月排行榜 / 总排行榜

dwer

2020-4-21 15:44:38

我觉得 在构造函数里加一句
    student()
        {
             pname = NULL;
       }

跳转到最佳答案楼层

JLDawson

这段代码运行正常，没有报错。如果有，请将截图发出来

Mr-Shuaibi

JLDawson 发表于 2020-4-21 11:46
这段代码运行正常，没有报错。如果有，请将截图发出来

可是我的析构函数里设置了输出“析构已执行”的命令，执行后什么也没有输出，一步一步调试后发现就是运算符重载的if出了问题，去掉就对了。

具体图片我编辑过了，在原帖子里。

JLDawson

Mr-Shuaibi 发表于 2020-4-21 12:23
可是我的析构函数里设置了输出“析构已执行”的命令，执行后什么也没有输出，一步一步调试后发现就是运 ...

1.“析构已执行”未输出在窗口的原因是，你步进调试未调试到最后。你一直步进到程序自动退出。每次步进查看窗口。一定是有输出的。
2.operator=函数中的if在当前展示的代码中应该是没有问题的，请楼主再次确认一下

dwer

我觉得 在构造函数里加一句
    student()
        {
             pname = NULL;
       }

Mr-Shuaibi

JLDawson 发表于 2020-4-21 14:09
1.“析构已执行”未输出在窗口的原因是，你步进调试未调试到最后。你一直步进到程序自动退出。每次步进查 ...

你看我的原帖子，下面有两张图，那个黑窗口就是我执行程序最后的结果，啥都没有，另一幅图就是我步进调试到重载程序里的if语句时出现的错误，到那一步之后按F10步进调试会一直出现这个窗口，根本调试不到最后，把重载程序里的if删了就行了，可以调试到最后，也会显示“析构已执行”，所以想问为什么删了就可以了。

Mr-Shuaibi

dwer 发表于 2020-4-21 15:44
我觉得 在构造函数里加一句
    student()
        {

对对对，这样就对了，感谢感谢

Mr-Shuaibi

dwer 发表于 2020-4-21 15:44
我觉得 在构造函数里加一句
    student()
        {

其实不加也可以，在重载函数里的if语句中的delete [ ]pname改为pname=NULL也是可以的，应该是一个作用。

dwer

Mr-Shuaibi 发表于 2020-4-21 16:14
其实不加也可以，在重载函数里的if语句中的delete [ ]pname改为pname=NULL也是可以的，应该是一个作用。

其实就是 指针没有初始化,然后分配了一个随机的值,  if(pname) 可以进去, 但是delete的时候那块是无效的

上面你那么不delete的 直接=NULL,那块内存还是被使用了,这么内存泄漏的

Mr-Shuaibi

dwer 发表于 2020-4-21 16:20
其实就是 指针没有初始化,然后分配了一个随机的值,  if(pname) 可以进去, 但是delete的时候那块是无效 ...

明白了，受教了，那我=NULL后，在if里再加上一句delete [ ]pname可不可以呢

freehawkzk

上面已经有人给出解决方案了，回复一下主要是提醒楼主，对于C/C++，请养成指针使用前必须初始化的习惯。请让指针无时无刻都处于一个你自己指定的状态下，不要依赖编译器的动作。

dwer

Mr-Shuaibi 发表于 2020-4-21 16:26
明白了，受教了，那我=NULL后，在if里再加上一句delete [ ]pname可不可以呢

这么是可以的