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)
{
}

顺便说一句，它效率更高，并且表明您在使用initializer-list时了解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;
};

一次从这三个类别派生的唯一原因是懒惰。最初保存的几秒钟将使您的代码随着类的增长而难以维护。有时，您可能需要支持一个以上的地址或日期，例如录用日期，如果在这种情况下，随着变量在许多地方使用，您将不得不对代码进行更多更改。您将浪费所有最初保存的时间，以及更多。

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

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

派生类的C ++复制构造函数

1 个答案: