使用复制构造函数的深度复制

Question

我有一项任务需要执行以下操作。

我有 3 个课程，其中 A 链接到许多 B 关联A->*B->*C

A: Contains only -iA_ : int
B: Contains only -iB_ : int
C: Contains only -IC_ : int

任务是：

为所有类实现必要的代码，以便A包含用于深拷贝的拷贝结构。 容器类型列表。 不需要设置器和获取器

不需要更多的构造函数和析构函数

如果您使用默认实现，则必须证明这一点。

所以我已经实现了这是正确的吗？

class C
{
   private:
   int iC_;
}
class B
{
   private:
      int iB_;
      std::list<C*> CList_;
   public:
   B(std::list<C*> CList) : CList_(CList)
   {
   }
}
class A
{
   private:
      int iA_;
      std::list<B*> BList_;
   public:
   A(std::list<B*> BList) : BList_(BList)
  {
  }
  A(const A& crA)
 {
   iA_ = crA.iA_;
 }

}

Answer 1

不幸的是，您的代码还不正确。 为了创建包含指针的 class 的深层副本，您还需要复制这些指针指向的对象。 这是一个只有 A 和 B 的简单示例：

class B
{
 private:
    int iB_;
 public:
    // you will not be able to default this in your code
    B(const B& other) = default;
};

class A
{
 private:
    int iA_;
    vector<B*> BList_;
 public:
    A(const A& other) : iA_{other.iA_}, BList_{}
    {
        // fill up the BList_ with copies of other's BList_ items
        for (auto b : other.BList_) {
            // call B's copy constructor
            B* copyPtr = new B{*b};
            BList_.push_back(copyPtr);
        }
    }

    A& operator=(const A& other)
    {
        iA_ = other.iA_;

        // destroy all the current items in BList_
        while (!BList_.empty()) { delete BList_.pop_back(); }

        // fill up the BList_ with copies of other's BList_ items
        for (auto b : other.BList_) {
            // call B's copy constructor
            B* copyPtr = new B{*b};
            BList_.push_back(copyPtr);
        }
    }

    ~A()
    {
        // destroy all the current items in BList_
        while (!BList_.empty()) { delete BList_.pop_back(); }
    }
};

使用您的代码，您还需要使用 B 的复制构造函数实现类似的东西，以确保它复制CList_中的所有内容，只有这样您才能拥有真正的深度复制。

编辑：正如@Alan Birtles 所指出的，在这样的实现中，您还应该定义复制赋值运算符和析构函数以防止 memory 泄漏。