CODEWITHSUNDEEP
c++classc++11copy-constructor
Eliad
Eliad

Reputation: 926

Copy constructor for a class with unique_ptr to an abstract class as a member

I have a class (C) with a vector of unique_ptrs to an abstract class (A) as a member. This is because C must work with all classes of type A, i.e. its children.

The problem is that I cannot figure out how to write a copy constructor for C, since the type of the objects that the pointers are pointing to are not known at compile time. It actually seems impossible to me. Can anyone confirm that it is impossible? Do you have any suggestions on how to solve the problem? Is it too awful to have a class without a copy constructor?

Upvotes: 7

Views: 1603

Answers (2)

Zereges
Zereges

Reputation: 5209

Well, since std::unique_ptr<T> is not copyable, and thus std::vector<std::unique_ptr<T>> is not copyable, and thus C which has std::vector<std::unique_ptr<T>> as a member should not be default-copyable.

You can, of course, implement copy-constructor that makes deep copy of T, but it depends on what T actually is.

Upvotes: 1

sirgeorge
sirgeorge

Reputation: 6541

You did not say whether you have control of the code for the abstract class and the classes derived from it. If you do, then the easiest way is to provide a pure virtual method Clone in the abstract class and implement it in derived classes. This method should handle creating the right copies. Unfortunately, because unique_ptr is not copyable you need to iterate through your vector and create copies by calling Clone.

Upvotes: 2

Related Questions