0

我需要帮助在我的面向对象程序中执行这个复制构造函数。结果应该是将字符串 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;


}
4

1 回答 1

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），以便你在正确的方向上复制。

于 2012-04-21T01:28:22.220 回答