>Видимо ты хотел сказать конструктор копирования. Кода, который ты привел, недостаточно, чтобы дать конкретный >совет. Пиши объявление класса полностью. Например, неясно, есть ли у тебя указатели в классе и если есть, то как >они используются.

Я имел ввиду конструктор копирования. Мой класс picture имеет два private members

...
std::vector<const Shape*> sContainer;
Graphics *grDevice;
....

Вектор содежит указатели на виртуальный Shape, от которого происходит наследование. Эти указатели будут использоваться для вызова виртуальных методов наследников Shape.

Copy constructor должен скопировать все поля из объекта переданного в качестве параметра в конструктор копирования. Я могу добавлять любые методы, но не думаю что корректно делат что-то вроде

....
std::vector<const Shape*> Picture::getContainer() return sContainer;
...
Graphics Picture::getGR() return grDevice;
......
.....
}