派生类的C ++ Copy构造函数

Question

所以我有这个类有三个父函数，换句话说，它是从其他三个类派生的。

我正在尝试制作一个复制构造函数，这就是我所拥有的：

// Copy constructor
    extPersonType (extPersonType &obj) : addressType(obj), personType(obj), dataType(obj)
    {
        cout << "Copy constructor active." << endl;
        phone = obj.phone;
        ident = obj.ident;
    }

这是我的其他三个类的副本构造函数。

// copy constructor
    addressType(extPersonType &obj)
    {
        street = obj.street;
        city = obj.city;
        state = obj.state;
        zipcode = obj.zipcode;
    }


// copy constructor
    personType(extPersonType &obj)
    {
        firstname = obj.firstname;
        lastname = obj.lastname;
    }


// copy constructor
    dataType(extPersonType &obj)
    {
        day = obj.day;
        month = obj.month;
        year = obj.year;
    }

请记住，每个人都有自己的头文件和cpp文件。虽然在这种情况下我使用了内联函数定义。

然而，这是我得到的错误：

[traine@joker Assignment2]$ make
g++ ExtPerson.cpp -c
In file included from ExtPerson.h:5:0,
                 from ExtPerson.cpp:3:
Data.h:19:26: error: expected ‘)’ before ‘&’ token
   dataType(extPersonType &obj)
                      ^
In file included from ExtPerson.h:6:0,
                 from ExtPerson.cpp:3:
Person.h:18:28: error: expected ‘)’ before ‘&’ token
   personType(extPersonType &obj)
                        ^
In file included from ExtPerson.h:7:0,
                 from ExtPerson.cpp:3:
Address.h:20:29: error: expected ‘)’ before ‘&’ token
   addressType(extPersonType &obj)
                         ^
make: *** [ExtPerson.o] Error 1

谁知道我做错了什么？我只是对如何在派生类中创建复制构造函数以及如何在其他类中调用其他复制构造函数感到困惑。任何帮助将不胜感激，谢谢。

Answer 1

您可能需要添加前向声明，因为在声明基类时尚未声明派生类。

class extPersonType;

但这不是必要的。为什么不按照正常模式声明基础构造函数？这是接受与类相同类型的参数。这样可以正常工作，并且不依赖于派生类。

personType(const PersonType &obj) 
    : firstname(obj.firstname)
    , lastname(obj.lastname)
{
}

顺便说一句，它更高效，并表明您在使用初始化列表时了解C ++。它将避免调用默认构造函数，然后调用赋值运算符。

https://en.cppreference.com/w/cpp/language/initializer_list

但是，使用派生将不相关的对象组合在一起仍然是一个糟糕的设计。地址不符合IS-A与人的关系。所以让extPersonType来自addressType没有多大意义。

当extPersonType类有一个addressType成员时，你应该使用包容。

class extPersonType : public personType
{
    addressType address;
    dataType birth_date;
};

一次从3个班级派生的唯一原因是懒惰。您最初保存的几秒钟将使您的代码难以维护，因为您的课程将会增长。并且在某些时候，您可能需要支持多个地址或日期，如招聘日期，如果在多个地方使用该变量，您将不得不对代码进行更多更改。你将浪费你最初保存的所有时间等等。

顺便说一下，在代码中拼错单词是个坏主意。正确的拼写是日期而不是数据，因为我们可以很容易地看到它是来自成员的日期而不是任意数据。

顺便说一句，阅读有关设计和编码的好书可能是一个好主意，因为这是每个程序员应该掌握的非常基本的东西。