8

プライベート コピーコンストラクターAとcloneメソッド、および. にも実装したいと思います。BAcloneB

素朴なアプローチ

#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がプライベートであるため、失敗します。

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()forを利用する方法があるかもしれませんが、B::clone()これが正確にどのように機能するかはわかりません。ヒントはありますか？

4

3 に答える 3

4

Bpublic メンバーがまったくなくpublic:、 の定義の前にa がないのはタイプミスだと思いますB::B(int,int)。

your で表されるクラスの作成者は、A明らかにそれを複製可能にすることを望んでいますが、コピー構築可能にすることは望んでいません。これは、すべてのインスタンスがヒープ上に存在することを望んでいることを示唆しています。しかし逆に、 public コンストラクターがあり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したがって、問題のクラスを忠実に表しているとは言えません。ただし、額面どおりに考えると、答える必要がある質問は次のとおりです 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;
}
于 2015-06-05T19:34:16.647 に答える
2

A派生クラスがそれを使用できるように、保護されたコピー コンストラクターを作成する必要があります。

protected:
    A(const A & a) { /*...*/ }

それが役立つことを願っています。

于 2015-06-05T11:25:05.297 に答える