Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

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?

like image 800
Eliad Avatar asked Oct 20 '22 10:10

Eliad


2 Answers

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.

like image 145
sirgeorge Avatar answered Oct 21 '22 23:10

sirgeorge


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.

like image 38
Zereges Avatar answered Oct 22 '22 00:10

Zereges