1

У меня есть некоторый класс item, который имеет указатель в своём теле.

class item{
protected:
    int id;
    std::string name;
    int price;
    std::string description;
    QPixmap* pic=nullptr;
    string picpath;

Нужно ли мне делать свой конструктор копирования в этом случае и как он должен выглядеть? Как первый вариант или второй? 1 Вариант:

item::item(const item& x):id(x.id),name(x.name),price(x.price),description(x.description),pic(x.pic),picpath(x.picpath)
{
}

Второй:

item::item(const item& x):id(x.id),name(x.name),price(x.price),description(x.description),picpath(x.picpath)
{
    pic=new QPixmap(*x.pic);
}
7
  • В чем смысл делать pic указателем? Если сделать его просто QPixmap pic;, то самописный конструктор копирования не потребуется. 16 дек 2018 в 12:15
  • 1
    Если чесно, я сам не знаю, в С++ вообще не понятно, как правильно что делать, когда работают оба варианта, то вообще не ясно(( 16 дек 2018 в 12:18
  • первый вариант равносилен конструктору копирования по умолчанию.... какой использовать — зависит от того, что тебе нужно.... ЗЫ: третий вариант — использовать std::shared_ptr
    – Fat-Zer
    16 дек 2018 в 12:20
  • @Fat-Zer мне просто нужен правильный конструктор копирования. Чтоб использовать shared ptr мне нужно переписать пол программы(( 16 дек 2018 в 12:26
  • @andybelous2, они оба «правильные», просто разные... какой использовать — зависит в частности от того, кто отвечает за время жизни pic; если item сам, то вероятней всего нужен второй вариант и лучше использовать std::unique_ptr или std::shared_ptr; если какая-то внешняя сущность, то вероятней всего нужен первый вариант.
    – Fat-Zer
    16 дек 2018 в 12:34

0

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Посмотрите другие вопросы с метками или задайте свой вопрос.