使用.Copy执行复制构造函数的最佳方法？

Question

我需要在我的面向对象程序中执行此副本构造函数的帮助。 结果应该是将字符串1： Hello World复制到字符串2： This is a test 。

在我的.h文件中 ：

void Copy(MyString& one);

在我的.cpp文件中 ：

void MyString::Copy(MyString& one)
{
   one = String;
}

在我的main.cpp文件中 ：

String1.Print();
cout  << endl;
String2.Print();
cout  << endl;      
String2.Copy(String1);      
String1.Print();       
cout  << endl;
String2.Print();
cout  << endl;

输出 ：

Hello World
This is a test
is a test
This is a test

应该是 ：

Hello World
This is a test
Hello World
Hello World

请告诉我我在做什么错？

这是我的整个.cpp文件：

MyString::MyString()

{

char temp[] = "Hello World";

        int counter(0);
        while(temp[counter] != '\0') {
                counter++;
        }
        Size = counter;
        String = new char [Size];
        for(int i=0; i < Size; i++)
                String[i] = temp[i];

}

MyString::MyString(char *message)

{

      int counter(0);

       while(message[counter] != '\0') {

      counter++;

   }

        Size = counter;

        String = new char [Size];


         for(int i=0; i < Size; i++)

           String[i] = message[i];

}

MyString::~MyString()

{

        delete [] String;

}

int MyString::Length()

{
              int counter(0);

              while(String[counter] != '\0')
             {
                     counter ++;
             }

                return (counter);
}

void MyString:: Set(int index, char b)

{

         if(String[index] == '\0')

          {
                   exit(0);
          }

         else

        {

                   String[index] = b;
          }


}

void MyString::Copy(MyString& one)

{

        one = String;


}

char MyString:: Get(int i)
{

             if( String[i] == '\0')
             {
                     exit(1);
             }
            else
            {

                    return String[i];

            }
}



void MyString::Print()

{

        for(int i=0; i < Size; i++)

            cout << String[i];

             cout << endl;


}

Answer 1

void Copy(MyString& one); 不是复制构造函数，但是我想您所需要的只是一个可以复制的函数。

实现：

void MyString::Copy(MyString& one) {
   one = String;
}

这将使用String ，无论是什么，大概是您的类的内部存储，然后将您传入的字符串分配给它。

这意味着String2.Copy(String1)就像String1 = String2.String 。 所需的输出表明您要将String1复制到String2，但方向完全错误。

此外，此分配导致部分副本（“是测试”）的事实意味着，无论您执行的是哪种分配，也都被破坏了。 您没有显示该实现，所以我无法告诉您它是如何损坏的。

---

代码有很多问题。 您可能应该与另一位学生或助教一起讨论。 但是对于您输出的特定问题，您应该看三件事。 首先，谷歌“三个规则”。 其次考虑当左侧是MyString对象而右侧是char *时，“ =”的含义。 第三，一旦弄清楚了将如何精确复制数据，请反转分配方向（例如*this = out.String ），以使复制方向正确。