Factory Method-Muster mit unique_ptr Mitglied einer abstrakten Klasse und Copy-Konstruktor

Question

Angenommen, ich habe eine Einstellung mit Abstrakte Klasse ApplicatonBase, die eine unique_ptr auf eine andere abstrakte Klasse Document hat:

class ApplicationBase 
{ 
public: 

    void factoryMethod() 
    { 
     _doc->doSomething(); 
    }; 

protected: 
    std::unique_ptr<DocumentBase> _doc; 

}; 

class DocumentBase 
{ 
public: 
    virtual void doSomething() = 0; 
}; 

Nun entwickle ich konkrete Klassen RichDocument und RichApplication wie folgt:

class RichDocument : public DocumentBase 
{ 
public: 
    void doSomething() override 
    { 
     std::cout << "I'm rich document\n"; 
    } 
}; 

class RichApplication : public ApplicationBase 
{ 
public: 
    RichApplication() 
    { 
     _doc = std::unique_ptr<RichDocument>(new RichDocument()); 
    } 

    RichApplication(const RichApplication& other) 
    { 
     /* 
     * TODO: I want to copy the data from other document 
     *  unique_ptr and give it to the current document 
     *  unique_ptr 
     */ 

     //Following crashes as the copy constructor is not known: 
     // error: no matching function for call to ‘RichDocument::RichDocument(DocumentBase&) 
     //_doc.reset(new RichDocument(*other._doc)); 
    } 
}; 

Das Problem liegt im Kopierkonstruktor. Ich möchte die Daten im Copy-Konstruktor im unique_ptr tiefkopieren. Gibt es eine Möglichkeit, das zu tun? Oder sollte ich meine Struktur ändern? Jede Hilfe wäre willkommen.

Answer 1

Da Sie RichDocument und RichApplication in der Klassenhierarchie Paarung sind, sollten Sie eine der folgenden Möglichkeiten zu tun, darauf vorbereitet sein:

• jede Basisdokument übernehmen, wenn ein reiches Dokument oder
• alle Ablehnen Konstruktion Dokumente außer RichDocument.

Unter der Annahme, dass der einzige Weg _doc ist im Konstruktor von RichApplication, der zweite Ansatz initialisieren sollte gut genug sein:

RichDocument *otherDoc = dynamic_cast<RichDocument*>(other._doc.get()); 
// We created RichDocument in the constructor, so the cast must not fail: 
assert(otherDoc); 
_doc.reset(new RichDocument(*otherDoc));