Question

此代码无法编译（请忽略在operator +中按值传递，我可以通过引用＆amp;替换它，但它仍然没有解决问题）我期待主要功能：

String s3 = s + s2; // COMPILATION ERROR

要编译好（因为我声明了复制构造函数）但是它给出了错误（＆＃34;没有匹配的构造函数＆＃34;）

#include <iostream>
#include <string>


class String {
public:
    String()
    {
        std::cout << "Constructor " << this << std::endl;
        data = new char[100];
    };

    String(char * str) : String()
    {
        std::cout << "Char * Constructor " << this << std::endl;
        strcpy(this->data,str);
    };

    String(String & rhs) : String() {
        std::cout << "Copy Constructor " << this << std::endl;
        strcpy(data, rhs.data);

    };


    void print() {
        printf("%s\n",data);
    }

    ~String() {
        std::cout << "Destructor " << this << std::endl;

        if (data) {
            delete data;
            data = nullptr;
        }
    };
    friend String operator+(String s1, String s2);

private:
    char * data;
};

String operator+(String s1, String s2)
{
    String temp;
    delete [] temp.data;
    temp.data =
    new char[strlen(s1.data) + strlen(s2.data) + 1];
    strcpy(temp.data, s1.data);
    strcat(temp.data, s2.data);
    return temp;
}


int main(int argc, const char * argv[])
{
    String s("herer");
    s.print();
    String s2 = s;
    s2.print();
    String s3 = s + s2; // COMPILATION ERROR
    return 0;
}

Answer 1

复制构造函数的参数必须是const&。

String(String const & rhs) : String() { ... }

Answer 2

您的代码有几处错误。第一

String(char * str)

需要

String(const char * str)

如果您想像main()中使用String s("herer");一样使用字符串文字。其次是你的复制构造函数

String(String & rhs)

需要采取

String(const String & rhs)

这样它就可以绑定到临时工。

第三，您需要包含<cstring>或<string.h>才能使用strcpy()。

最后，你在析构函数中使用了错误的delete。当您致电new时，您使用delete，当您使用new[]时，请致电delete []。您使用new[]分配data，但是要使用delete将其删除。

您可以看到代码here

的实际示例

C ++拷贝构造函数没有调用（编译错误）

2 个答案: