c++ - 在 C++ 中连接两个自定义字符串对象？

Question

我正在为作业定义自己的字符串类。我注意到以下代码

class MyString {
public:
    MyString(const char* s = NULL) {len = strlen(s); str = new char[len + 1]; strcpy(str, s);}
    ~MyString() {delete [] str;}
    friend ostream& operator << (ostream& ost, const MyString& s) { ost << s.str; return ost;}
    friend MyString operator + (const MyString &s1, const MyString &s2) { 
        int length = strlen(s1.str) + strlen(s2.str); 

        char *str = new char[length + 1];
        strcpy(str, s1.str);
        strcat(str, s2.str);
        return MyString(str);
    }
private:
    char * str;
    int len;
};

int main () {
    MyString s1 = MyString("hello");
    MyString s2 = MyString("world");
    cout << s1 + s2 << endl;
    return 0;
}

有效，因为返回对象是在最后一刻创建的。但是下面的代码

class MyString {
public:
    MyString(const char* s = NULL) {len = strlen(s); str = new char[len + 1]; strcpy(str, s);}
    ~MyString() {delete [] str;}
    friend ostream& operator << (ostream& ost, const MyString& s) { ost << s.str; return ost;}
    friend MyString operator + (const MyString &s1, const MyString &s2) { 
        int length = strlen(s1.str) + strlen(s2.str); 
        MyString temp;
        temp.str = new char[length + 1];
        strcpy(temp.str, s1.str);
        strcat(temp.str, s2.str);
        return temp;
    }
private:
    char * str;
    int len;
};

int main () {
    MyString s1 = MyString("hello");
    MyString s2 = MyString("world");
    cout << s1 + s2 << endl;
    return 0;
}

不，给我一个运行时错误。因此，如果在重载运算符中定义、修改和返回了一个临时对象，为什么第二种方法会失败，我感到很困惑。

Answer 1

问题是，当您在此处默认构造 temp 时:

MyString temp;

你必须执行:

MyString(const char* s = NULL) {len = strlen(s); ... }

空指针上的

strlen 未定义。如果您改为将默认参数更改为:

MyString(const char* s = "")

---

然而，这两种解决方案仍然很差，因为它们都泄漏了内存。在前者中，您永远不会delete[] 临时str。在后者中，您的默认构造函数分配了一个新的 str 成员，然后您立即用一个新分配的 str 成员覆盖它。原件泄露。