-1

我使用下面的代码来测试复制赋值运算符的行为:

#include <iostream>
using namespace std;

int group_number = 10; // Global

class Player {
public:
  explicit Player(const int &g): group(g)
  {
  }

  Player& operator=(const Player& rhs)
  {
      cout << "Copy assignment operator called " << endl;
      return *this;
  }

  void set_stamina(int i) {stamina = i; }
  int  get_stamina()      {return stamina; }
  const int &get_group()  const { return group; }

protected:
  const int &group;

private:
  int stamina;
}; // End of Player class

class Round {
public:
  Round(const Player &p)
  {
    main_player = &p;
  }

  const Player &get_main_player() const
  {
    return *main_player;
  }

protected:
  const Player *main_player;
}; // End of Round class

// Test code
int main()
{
  Player p1(group_number);
  Round r(p1);
  p1.set_stamina(100);

  // Add player2 in the same group as p1
  Player p2(group_number);
  p2 = r.get_main_player();
  cout << "This is p1's stamina: "    << p1.get_stamina()
       << ", and p1's group number: " << p1.get_group() << endl;;
  cout << "This is p2's stamina: "    << p2.get_stamina()
       << ", and p2's group number: " << p2.get_group() << endl;;

  return 0;
}

我预计 p1 和 p2 的值相同stamina。但输出显示 p2stamina与 p1 不同

Copy assignment operator called
This is p1's stamina: 100, and p1's group number: 10
This is p2's stamina: 241098768, and p2's group number: 10

为什么复制赋值运算符不将 p1 的值复制stamina到 p2?

4

2 回答 2

2

您尚未在复制赋值运算符中复制数据成员。stamina因此,只需stamina = rhs.stamina;在复制赋值运算符中添加即可,您将获得预期的输出。所以修改后的定义operator=如下:

Player& operator=(const Player& rhs)
  {
      cout << "Copy assignment operator called " << endl;
      stamina = rhs.stamina; //ADDED THIS 
      return *this;
  }

另外,请注意,在您的原始代码片段中,由于数据成员stamina未初始化,因此它具有不确定的值。使用这个未初始化的变量(您在编写时所做p2.get_stamina();的)是未定义的行为

未定义的行为意味着任何事情1都可能发生,包括但不限于给出预期输出的程序。但永远不要依赖(或基于)具有未定义行为的程序的输出。

因此,您看到的输出是未定义行为的结果。正如我所说,不要依赖具有 UB 的程序的输出。

因此,使程序正确的第一步是删除 UB。只有这样,您才能开始推理程序的输出。

解决这个更严重的问题,只需对数据成员使用类内初始化stamina程序。

class Player {
    //other members here
private:
  int stamina = 0; //USE IN-CLASS INITIALIZER
};

1有关未定义行为的更技术上准确的定义,请参见此处提到:对程序的行为没有限制

于 2022-01-18T15:25:34.540 回答
1

为什么复制赋值运算符不将 p1 的耐力值复制到 p2?

您定义了运算符。您不会在运算符的定义中复制任何值,因此不会复制任何值。

您可以像这样分配成员:

stamina = rhs.stamina;
于 2022-01-18T15:24:51.517 回答