C++中类的设计简单问题

Question

运行无法达到效果

#include <iostream>
#include <string>
using namespace std;

class mystring{
private:
    char *p;
public:
    mystring(char *a){
        p = a;
    }
    void set(char b){
        p[0] = b;
    }
    void show(){
        cout << *p << endl;
    }
    ~mystring(){
        delete[]p;
        cout << "free memory!" <<endl;
    }
};

int main() {
    mystring s1("hello");
    mystring s2 = s1;
    s1.set('H');    // 修改字符串的第一个字符
    s1.show();
    s2.show();
    return 0;
}

报错内容为[Warning] ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
想要的效果为
Hello
hello
free memory!
free memory!

Answer 1

努力の小熊参考Chatgpt做出的回答：
代码中出现错误的原因是：将字符串常量直接赋值给了 char* 指针类型变量，但是字符串常量在 C++ 中是不可修改的，将其赋值给 char* 指针类型变量会导致编译器报错。正确的做法应该是使用 const char* 类型来保存字符串常量。

另外，代码中没有实现拷贝构造函数和赋值运算符重载函数，当 s1 赋值给 s2 时，s2 也指向了 s1 的数组，这样会导致析构函数释放内存时出现错误。

下面是改正后的代码：

#include <iostream>
#include <cstring>  // 需要包含这个头文件以使用 strcpy 函数
using namespace std;
 
class mystring{
private:
    char *p;
public:
    mystring(const char *a){
        p = new char[strlen(a) + 1];
        strcpy(p, a);
    }
    mystring(const mystring& other){
        p = new char[strlen(other.p) + 1];
        strcpy(p, other.p);
    }
    void set(char b){
        p[0] = b;
    }
    void show(){
        cout << p << endl;
    }
    ~mystring(){
        delete[]p;
        cout << "free memory!" <<endl;
    }
    mystring& operator=(const mystring& other){
        if (this != &other) {
            delete[] p;
            p = new char[strlen(other.p) + 1];
            strcpy(p, other.p);
        }
        return *this;
    }
};
 
int main() {
    mystring s1("hello");
    mystring s2 = s1;
    s1.set('H');    // 修改字符串的第一个字符
    s1.show();
    s2.show();
    return 0;
}

在改正后的代码中，使用 const char* 类型来保存字符串常量并使用 strcpy 函数复制到新分配的内存空间中；实现了拷贝构造函数和赋值运算符重载函数，避免了出现错误。