c++ - 复制构造函数被隐式删除，因为默认定义的格式不正确

Question

我有一个 A 类(来 self 无法控制的库)，它有一个私有(private)复制构造函数和一个 clone 方法，还有一个类 B 派生自 A。我也想为 B 实现 clone。

天真的方法

#include <memory>

class A { // I have no control here
  public:
    A(int a) {};

    std::shared_ptr<A>
      clone() const
      {
        return std::shared_ptr<A>(new A(*this));
      }

  private:
    A(const A & a) {};
};

class B: public A {
  public:
    B(int data, int extraData):
      A(data),
      extraData_(extraData)
    {
    }

    std::shared_ptr<B>
    clone() const
    {
      return std::shared_ptr<B>(new B(*this));
    }

  private:
    int extraData_;
};

int main() {
  A a(1);
}

然而，失败了，因为 A 的复制构造函数是私有(private)的:

main.cpp: In member function ‘std::shared_ptr<B> B::clone() const’:
main.cpp:27:42: error: use of deleted function ‘B::B(const B&)’
     return std::shared_ptr<B>(new B(*this));
                                      ^
main.cpp:17:7: note: ‘B::B(const B&)’ is implicitly deleted because the default definition would be ill-formed:
 class B: public A {
       ^
main.cpp:14:5: error: ‘A::A(const A&)’ is private
     A(const A & a) {};
     ^
main.cpp:17:7: error: within this context
 class B: public A {

可能有一种方法可以将 A::clone() 用于 B::clone()，但我不确定这将如何工作.有什么提示吗？

Answer 1

我认为您的 B 根本没有公共(public)成员是一个拼写错误， 并且您在 B::B(int,int) 的定义之前缺少一个 public:。

你的 A 所代表的类的作者显然希望它是 可克隆但不可复制构造。这表明他或她想要所有 存在于堆上的实例。但相反，有公众 构造函数A::A(int)。你确定你是对的吗？

假设类可以揭示足够的信息是合理的 关于一个给定的实例构成另一个实例。例如，放 A 上的肉多一点:

class A {
public:
    A(int a) 
    : data_(a){};

    std::shared_ptr<A>
    clone() const
    {
        return std::shared_ptr<A>(new A(*this));
    }

    int data() const {
        return data_;
    }

private:
    A(const A & a) {};
    int data_;
};

如果这是真的，那么公共(public)构造函数只会渲染它 不方便规避私有(private)的、未定义的复制构造函数:

A a0(1);
A a1{a0.data()};     // Inconvenient copy construction

所以我不太相信 A 忠实地代表了问题 类(class)。然而，从表面上看，你需要回答的问题 是:你甚至可以不方便地复制构造一个A吗？

如果不是，那么你就卡住了。如果是这样，那么你可以使用不方便的拷贝 A 的构造，以明确定义 B 的常规复制构造函数， 这就是你所需要的。例如

class B: public A {
public:
    B(B const & other)
    : A(other.data()),extraData_(other.extraData_){}    

    B(int data, int extraData):
    A(data),
    extraData_(extraData)
    {
    }

    std::shared_ptr<B>
    clone() const
    {
        return std::shared_ptr<B>(new B(*this));
    }

    int extradata() const {
        return extraData_;
    }

private:
    int extraData_;
};

#include <iostream>

int main()
{
    B b(1,2);
    std::shared_ptr<B> pb = b.clone();
    std::cout << pb->data() << std::endl;
    std::cout << pb->extradata() << std::endl;
    return 0;
}