0

我在弄清楚如何编写我的复制构造函数时遇到了一些麻烦......

这是我的构造函数：

public:
Person(const char * aName, const char * userID, const char * domain, Date aDate) {
//This constructor should create heap objects

    name = new char[strlen(aName) + 1]; 
    strcpy (name, aName);
    emailAddress = new EmailAddress(userID, domain); 
    birthday = new Date(aDate); 

    cout << "\nPerson(...) CREATING: ";
    printOn(cout);
}

这就是我要为我的复制构造函数做的事情：

Person(const Person & p){
    name = new char[strlen(p.name)+1];
    strcpy(name, p.name);
    emailAddress = new EmailAddress(*p.emailAddress);
    birthday = new Date(*p.date);

    cout << "\nPerson(const Person &) CREATING: ";
    printOn(cout);
}

我不确定在我的复制构造函数中为我的新 Date 和新 EmailAddress 传递什么，我现在正在做的事情根本不起作用！

这是我的赋值运算符（我不知道在这里再次传递什么日期和电子邮件地址......）： 

Person & operator=(const Person & p) {
        if(&p != this) {
           delete [] name;
           delete emailAddress;
           delete birthday;
           name = new char[strlen(p.name) + 1];
           strcpy (name, p.name);
               emailAddress = new EmailAddress();
               birthday = new Date();
        }
        return *this;
    }

任何帮助将不胜感激！

编辑：

日期定义

class Date{ //this class is complete
//This class represents a date

public:
      Date(int aDay, int aMonth, int aYear) : day(aDay), month(aMonth), year(aYear) {} 
      Date(const Date & aDate) :  day(aDate.day), month(aDate.month), year(aDate.year)     {};
      void printOn(ostream & out) const {out << day <<"/" << month << "/" << year;}
4

2 回答 2

4

我将把 JimR 的评论分解成一个答案，因为这是一个很好的建议：如果可能的话，使用 C++ 类型，而不是 C 类型。此外，尽可能避免使用指针。

使用 C++ 类型

例如，您的班级，正如我们所看到的那样，看起来像这样：

class Person {
    char *name;
    Email *emailAddress;
    Date *birthday;
};

如果Date和Email足够轻量级（它们应该如此），请将它们作为值保存在对象本身中；另外，做name一个std::string：

class Person {
    std::string name;
    Email emailAddress;
    Date birthday;
};

您的构造函数变得非常琐碎：

Person::Person(const std::string &aName, const std::string &userID, const std::string &domain, Date aDate) :
    name(aName),
    emailAddress(userID, domain),
    birthday(aDate)
{ }

更好的是，C++ 为您创建的默认复制构造函数现在可以正常工作了。析构函数和赋值运算符也是如此。免费。更好的是，所有这些现在都是异常安全的：如果由于某种原因引发了异常，那么一切都会被清理干净。

于 2013-10-22T01:05:10.343 回答
0

我得到了它的工作 - 谢谢大家帮助我！我必须传入 *p.emailAddress 和 *p.birthday

Person(const Person & p){
    name = new char[strlen(p.name)+1];
    strcpy(name, p.name);
    emailAddress = new EmailAddress(*p.emailAddress);
    birthday = new Date(*p.birthday);

    cout << "Person(const & Person)... CREATING: ";
    printOn(cout); 
}

Person & operator=(const Person & p) {
        if(&p != this) {
           delete [] name;
           delete emailAddress;
           delete birthday;
           name = new char[strlen(p.name) + 1];
           strcpy (name, p.name);
           emailAddress = new EmailAddress(*p.emailAddress);
           birthday = new Date(*p.birthday);
        }
        return *this;
    }
于 2013-10-22T00:41:05.793 回答