[已解决]求助各位老师 关于赋值运算符重载问题

赵国梁

#define  _CRT_SECURE_NO_WARNINGS
#include<iostream>
using namespace std;


class dog
{
public:
        int age;
        int num;
        char* p_name;

        ~dog()
        {
                if (p_name)
                {
                        delete[] p_name;
                        p_name = NULL;
                }
        }
        dog& operator=(dog& p)
        {
                age = p.age;
                num = p.num;

                if (p_name)
                {
                        delete[] p_name;
                        p_name = NULL;
                }

                int name_lens = strlen(p.p_name) + 1;
                p_name = new char[name_lens];
                memset(p_name, 0, name_lens);
                strcpy(p_name, p.p_name);

                return *this;
        }
};

void test()
{

        dog d1;
        d1.age = 18;
        d1.num = 1;
        char str[] = "zhaoguoliang";
        d1.p_name = str;
        dog d2;
        d2 = d1;
        cout << d2.age << "  " << d2.num << "  " << d2.p_name << endl;

}
int main()
{
        test();
       

        system("pause");
        return 0;

}

错误

最佳答案

月排行榜 / 总排行榜

tony666

2020-6-10 14:07:59

    char str[] = "zhaoguoliang";
    d1.p_name = str;

这个str 是栈里的，不能用delete
类中的指针最好不要直接赋值

1.     void Set_Pname(const char* _p_name) {
   
2.         if (p_name) {
   
3.             delete[] p_name;
   
4.             p_name = NULL;
   
5.         }
   
6.         p_name = new char[strlen(_p_name) + 1];
   
7.         strcpy(p_name, _p_name);
   
8.     }
   
9. 调用
   
10. d1.Set_Pname("VC驿站");
    

复制代码

跳转到最佳答案楼层

赵国梁

一直出现如图问题  也不知道哪里出了问题 各位老师帮助一下 感谢

赵国梁

这个是用复制运算符重载 解决内存重复释放问题     但是一直这个样子

Syc

赵国梁 发表于 2020-6-9 13:35
这个是用复制运算符重载 解决内存重复释放问题     但是一直这个样子

程序执行完之后崩溃了吗？
还是什么情况，编译没问题，执行报错了？

赵国梁

Syc 发表于 2020-6-9 16:07
程序执行完之后崩溃了吗？
还是什么情况，编译没问题，执行报错了？

是的 老师  生成没问题  执行就这样

Syc

给 dog 类加个构造函数就可以了：

1. dog()
   
2. {
   
3.         age = 0;
   
4.         num = 0;
   
5.         p_name = NULL;
   
6. }

复制代码

赵国梁

Syc 发表于 2020-6-9 17:59
给 dog 类加个构造函数就可以了：

老师 还是不行

zhizhuo.li

是你的析构器重复调用，释放已经被释放的内存造成的

赵国梁

zhizhuo.li 发表于 2020-6-9 21:19
是你的析构器重复调用，释放已经被释放的内存造成的

老哥  我该怎么改啊

tony666

    char str[] = "zhaoguoliang";
    d1.p_name = str;

这个str 是栈里的，不能用delete
类中的指针最好不要直接赋值

1.     void Set_Pname(const char* _p_name) {
   
2.         if (p_name) {
   
3.             delete[] p_name;
   
4.             p_name = NULL;
   
5.         }
   
6.         p_name = new char[strlen(_p_name) + 1];
   
7.         strcpy(p_name, _p_name);
   
8.     }
   
9. 调用
   
10. d1.Set_Pname("VC驿站");
    

复制代码