Комментарии на: О копировании объектов в C++

От: sba

sba — Wed, 18 May 2011 13:17:26 +0000

2picania: А если добрый дядя из соседнего отдела в своем cpp файле предусмотрит определение и копирующего конструктора и оператора присваивания для Вашего класса? Вобщем это извечный вопрос определять или не определять (только объявлять). Лично я тоже не определяю, но доброго дядю остерегаюсь.

От: Максим Тремпольцев

Максим Тремпольцев — Thu, 03 Feb 2011 17:09:55 +0000

Согласен

От: picania

picania — Wed, 02 Feb 2011 19:12:27 +0000

Цитата: Для этого надо перенести объявления копирующего конструктора и оператора присваивания в защищенную секцию (определять их необязательно)…
Лично я склоняюсь к тому, что их нужно ОБЯЗАТЕЛЬНО НЕ определять… Поскольку, не дай Бог внутри класса Port вы присвоите один порт другому и это присваивание прокатит, так как имеется их реализация. Но если реализация отсутствует, ошибка будет отловлена на стадии линковки.

От: Максим Тремпольцев

Максим Тремпольцев — Thu, 20 Jan 2011 17:09:55 +0000

Работать компилятором – дело неблагодарное :)

От: xrnd

xrnd — Thu, 20 Jan 2011 04:04:45 +0000

Очень интересная статья.

А я всегда пишу в копируемых классах конструктор копирования и оператор присваивания.
Боюсь, вдруг компилятор неправильно скопирует ;)

От: Vernat

Vernat — Mon, 06 Sep 2010 12:59:25 +0000

еще про запрет копирования, в Qt есть такой простой макрос
#define Q_DISABLE_COPY(Class) \
Class(const Class &); \
Class &operator=(const Class &);

просто и работает, меньше возьни с ручным описанием

От: Максим Тремпольцев

Максим Тремпольцев — Fri, 30 Jul 2010 08:09:53 +0000

ilnar, в целом справедливо, но:
1) Никаких локальных переменных до блока проверки быть не должно по определению
2) Вопрос вкуса, но мне кажутся наглядней короткие блоки if

От: ilnar

ilnar — Fri, 30 Jul 2010 07:42:22 +0000

Пример с присваиванием плох наличием двх return.
DataMessage& operator=(const DataMessage& from)
{
if (this == &from)
{
return *this;
}
Message::operator=(from);
Data_ = from.Data_;
return *this;
}
Это заставляет компилятора вставлять 2 раза код возврата, того хуже, вызова деструкторов локальных переменных, если они будут.
Предпочтительный вариант, который можно найти у Майерса:
DataMessage& operator=(const DataMessage& from)
{
if (this != &from)
{
Message::operator=(from);
Data_ = from.Data_;
}
return *this;
}