Question

我正在尝试实现一个复制构造函数和一个重载的赋值构造函数。我有一个有三个数据成员的Employee类。 1-name 2-title 3-salary。

Employee::Employee(const Employee &emp)
{
    name = emp.name;
    title = emp.title;
    salary = emp.salary;
    cout << "\nOverloaded copy constructor called\n";
}
Employee Employee::operator = (const Employee &emp) //Overloading the assignment operator.
{
    name = emp.name;
    title = emp.title;
    salary = emp.salary;
    cout << "\nOverloaded assignment operator called\n";
    return emp;         //Or return *this.
}

这是我不明白的：
1-我没有得到“这个”指针。我的重载赋值运算符应该返回* this还是emp对象。因为在我看来，参数中的对象是赋值时的右手对象。所以我不应该用* this返回左手对象（如果这是*将要做的）？ 2-在main函数中，我尝试先调用赋值运算符，然后调用复制构造函数。所以我期待着我会看到我一个接一个地包含在那里的cout语句然而，这是我的输出：

重载的拷贝构造函数叫做
重载的赋值运算符称为重载的拷贝构造函数叫做
重载的复制构造函数叫

为什么会这样？

3 - 我必须用const传递对象吗？我正在学习的那本书没有。

主要是我写的

Employee x;
x = another;
Employee y = x;

“另一个”只是我之前在代码中初始化的一个（命名不佳的）Employee对象所以不应该是第一个分配输出
“重载的赋值运算符被调用”和第二个赋值（根据我的理解，这不是赋值而是副本）输出“重载的复制构造函数”

Answer 1

赋值运算符通常实现为

Employee& Employee::operator = (const Employee &emp)
{
    name = emp.name;
    title = emp.title;
    salary = emp.salary;
    cout << "\nOverloaded assignment operator called\n";
    return *this;      
}

注意返回类型的差异作为参考。您可能会发现此问题/答案有用Why must the copy assignment operator return a reference/const reference?。如果没有看到您在main()函数中引用的代码，我们只会猜测您的代码中发生了什么。

Answer 2

1-我没有得到“这个”指针。我应该超载赋值运算符返回* this或emp对象。因为看来对我来说，参数中的那个对象是右手对象分配。所以我不应该用左手对象返回 *这（如果这是*将要做的）？

您应该返回*this。

另请注意，首先应检查自我分配并返回引用：

Employee& Employee::operator = (const Employee &emp) //Overloading the assignment operator.
{
    if ( this != &emp)
    {
        name = emp.name;
        title = emp.title;
        salary = emp.salary;
    }
    cout << "\nOverloaded assignment operator called\n";
    return *this;
}

Here you can find how to write assignment operator.

2-在主要功能，我试图打电话给作业先运算符，然后是复制构造函数。所以我在期待我会看到我在那之后包含的cout语句其他

请从main()插入此代码，以便我们为您提供更多信息。

Answer 3

编辑：让我们复制构造函数为你做一切

Employee::Employee(const Employee &emp)
{
   if(this != &emp)
   {
       name = emp.name;
       title = emp.title;
       salary = emp.salary;
   }

  cout << "\nOverloaded copy constructor called\n";
}
Employee Employee::operator = (const Employee &emp) //Overloading the assignment   .
{       
   return emp;    //Having common assignment operations similar to copy constructor
}

Answer 4

重载赋值运算符应返回* this指针。您可以实现一个私有方法，该方法将复制类成员，并从复制构造函数/赋值运算符中调用它，这将导致更少的重复代码。

void Employee::copy(const A& rhs) throw() 
{
  name = rhs.name;
  title = rhs.title;
  salary = rhs.salary;
}

Employee::Employee(const Employee& rhs)
{
  copy(rhs);
  //  cout << "copy constructor" << endl;
}

Employee& operator=(const Employee& rhs)
{
  copy(rhs);
  //  cout << "assignment operator" << endl;
  return *this;
}

您的赋值运算符的实现确实按值返回Employee对象（它应该通过引用），因此通过调用copy c-tor创建临时对象的效果。

Overloaded copy constructor called // don't know where this printout comes from as I don't know what happens with 'another' object
Overloaded assignment operator called // x = another; call
Overloaded copy constructor called // the temporary object created by assignment operator's return statement (*this should be returned by reference not by value)
Overloaded copy constructor called // Employee y = x; cal

当您不打算修改输入参数时，应始终将输入参数作为const传递。我假设您要求在赋值运算符等中将Employee对象作为const引用传递？ - 它们应该是const引用

更多关于赋值运算符中的异常安全和自赋值处理： GotW 59和GotW 23

复制构造函数或赋值运算符？我错过了什么？

4 个答案: