去除 C++ 中类之间的循环依赖

Question

假设我们正在制作一个两人纸牌游戏，我们有名为 Game、Player 和 Card 的类。游戏包含指向两个玩家的指针并为玩家提供接口。玩家包括玩家的健康和他们的魔法以及他们的手牌向量。 Card 是一个抽象类。每张牌都需要消耗魔法才能打出，并且可以打出。

问题在于，每张牌在使用时都可以通过多种方式改变游戏状态。例如，我们可以有一张使玩家生命值翻倍的卡片，一张使敌方玩家中毒两回合的卡片，一张摧毁板上所有随从的卡片，一张为已经在棋盘上的每个随从创建副本的卡片，等等. 可能性真的是无穷无尽的。我能看到的唯一解决方案是在每张卡片中都有一个指向游戏的指针，但这似乎相当不雅。

还有其他解决方案吗？

Answer 1

只要你有交叉依赖，比如：

class Card
{
   Game* game;
   void f() {
     game->method1();
   }
}; 

class Game
{
   std::vector<Card> cards;
 public:
   void method1();
};

其中一个依赖项应该实现如下接口：

class IGame
{
 public:
   virtual void method1()=0;
};

class Card
{
  IGame* game;
  void f() {
     game->method1();
   }
}; 

class Game : public IGame
{
   std::vector<Card> cards;
 public:
   virtual void method1();
};

你可以看到没有更多的交叉依赖。

Answer 2

您的基本想法是正确的，但您应该进一步增强它以保持高水平的抽象和封装。

因此，在您的示例中，假设您有一个 Card 类，它是现有类的一个实例。

你可以做的第一件事就是将卡牌的效果与卡牌本身分开，例如：

class CardEffect {
 // TODO ...
};

class Card {
private:
  const CardEffect* effect;
};

所以现在卡本身不需要知道任何关于游戏的信息。但是让我们更深入一点：CardEffect 不需要知道Game 类的每一个细节，它需要能够将效果应用到游戏中。这可以通过为效果提供一个单独的接口来完成，该接口只公开应用效果所需的内容，所以就像这样。

class GameInterface {
public:
  virtual const std::vector<Player*>& getPlayers() = 0;

  virtual damagePlayer(Player* player, int amount) = 0;
  virtual applyPeriodicDamage(Player* player, int turns, int amount) = 0

  ..
};

class CardEffect {
  virtual applyEffect(GameInterface* interface) = 0;
};

class Card {
private:
  const CardEffect* effect;
};

现在这只是一个示例，您的问题没有明确的解决方案，因为每个特定场景都不同并且有不同的要求，这只是为了让您了解如何在保持代码优雅和封装的同时保持它描述性强，易于管理。