Что должен содержать класс по-умолчанию?

@anapshy · Регистрация: 14.11.2016

Если пишется класс, то обязательно должны быть реализованы следующие операции:
1) Конструктор по-умолчанию
2) Перегрузка оператора '='
3) Конструктор копирования
4) Деструктор

Как вы считаете? Является ли это "правилом хорошего тона"?

И как должен по-умолчанию работать конструктор копирования, вызывать оператор '=' для двух объектов (*this = other;), при условии что оператор '=' уже перегружен?

@John Prick · 19.07.2017, 10:56

Сообщение от anapshy

Если пишется класс, то обязательно должны быть реализованы следующие операции:
1) Конструктор по-умолчанию
2) Перегрузка оператора '='
3) Конструктор копирования
4) Деструктор
Как вы считаете?

Вовсе не обязательно. Очень часто то, что генерирует компилятор, именно то, что нужно от этих функций.

@anapshy · 19.07.2017, 11:09 **[ТС]**

John Prick, да, это знаю.

Добавлено через 7 минут
MrGlurk, zss, Croessmah, ildwine, как вы считаете?

@rikimaru2013 · 19.07.2017, 11:38

anapshy, думаете красиво звать людей, которые не знают о вашей теме, получат уведомления и какую реакцию вы ожидаете? Не пойму откуда у вас позитивная репутация с таким отношениям к людям.

А по делу, нету обязательств у программистов делать перегрузку, если она не необходима

@TRam_ · 19.07.2017, 12:00

rikimaru2013, вопрос был чуть в другом - можно ли использовать в конструкторе копирования оператор присваивания. Лично моё мнение - можно, но это будет менее производительно. В конструкторе копирования проверять this == &other не требуется (т.к. это конструктор, его объект ранее не существовал, а значит не мог быть передан по ссылке), ну и для полей вызывать копирующий конструктор вместо оператора присваивания тоже будет рациональнее.

Добавлено через 2 минуты
Потому что иначе поля вначале инициализируются по-умолчанию, а только потом скопируются.

Croessmah · 19.07.2017, 12:15

Сообщение от anapshy

Если пишется класс, то обязательно должны быть реализованы следующие операции

Нет. Есть еще POD и другие приколюхи.

Добавлено через 49 секунд

Сообщение от TRam_

т.к. это конструктор, его объект ранее не существовал, а значит не мог быть передан по ссылке

Глупости. В конструктор копирования может быть передан и сам конструирующийся объект.

C++

1	Some a = a;//Валидный код. В конструктор копирования придет не инициализированный объект.

@zss · 19.07.2017, 12:16

Мое мнение о том, что должен содержать класс, я изложил создав образец:
Образцы (шаблоны) программ для типовых задач

@TRam_ · 19.07.2017, 12:21

Сообщение от Croessmah

В конструктор копирования может быть передан и сам конструирующийся объект.

Да? Не знал...
Вообще речь о разнице

C++

Foo::Foo(int q) : value()
{
    value = q;
}

и

C++

1
2
3

Foo::Foo(int q) : value(q)
{
}

@inovarka · 19.07.2017, 12:24

3) Конструктор копирования
Разве его стоит писать если нету динамических полей?

Croessmah · 19.07.2017, 12:25

Сообщение от TRam_

Речь о различии

Причем здесь это?
Вот цитаты:

Сообщение от TRam_

конструкторе копирования проверять this == &other не требуется

Сообщение от TRam_

т.к. это конструктор, его объект ранее не существовал, а значит не мог быть передан по ссылке

В конструктор копирования может быть передан тот же объект, который конструируется.
Не проверив, получим невалидный объект. правда, делать так никто не будет.
Я об этом уже писал на форуме с примером, можете поискать.

@anapshy · 19.07.2017, 12:25 **[ТС]**

rikimaru2013, а что не так-то? Что по вашему плохого в том, что я хочу услышать совета от людей, которые уже не 2, и не 5 лет в данной сфере (не считая времени отведенного на изучение)?

Croessmah · 19.07.2017, 12:36

Сообщение от Croessmah

Я об этом уже писал на форуме с примером, можете поискать.

Во: Отличие перегрузки operator() от конструктора

@anapshy · 19.07.2017, 13:09 **[ТС]**

Croessmah, будет же верным следующий код? Только теперь тут выводится operator= cctor. Логику почему так происходит - понимаю.
Переменные объекта v остаются теми же, что и в вашем бы коде (т.е. строка пустая).

C++

#include <iostream>
#include <string>
 
struct Test
{
    Test();
    Test(const Test&);
    Test& operator=(const Test&);
    std::string str;
};
 
Test::Test() : str("Hi, hoggy!")
{
    std::cout << "ctor" << std::endl;
}
 
Test& Test::operator=(const Test& rhv)
{
    if (this != &rhv)
        this->str = rhv.str;
    std::cout << "operator=" << std::endl;
    return *this;
}
 
Test::Test(const Test& src)
{
    *this = src;
    std::cout << "cctor" << std::endl;
}
 
int main()
{
    Test v = v; // operator=    cctor
    std::cout << v.str << std::endl;
}

Добавлено через 6 минут
Уже привык сначала перегружать =, а потом использовать его(=) в копирующем конструкторе.

@TRam_ · 19.07.2017, 13:14

anapshy, только тогда уже

C++

Test::Test(const Test& src) : str("Hi, hoggy 2 !")
{
    *this = src;
    std::cout << "cctor" << std::endl;
}

@anapshy · 19.07.2017, 13:18 **[ТС]**

TRam_, как я понимаю все способы верные, спасибо

@rikimaru2013 · 19.07.2017, 13:57

Не по теме:

Сообщение от anapshy

rikimaru2013, а что не так-то? Что по вашему плохого в том, что я хочу услышать совета от людей, которые уже не 2, и не 5 лет в данной сфере (не считая времени отведенного на изучение)?

так выделите все ники в разделе С++. Так ведь больше профита будет - вы согласны?

@anapshy · 19.07.2017, 14:02 **[ТС]**

rikimaru2013,

прислушаюсь к вашему совету

@anapshy 237 / 263 / 218 Регистрация: 14.11.2016 Сообщений: 988
		1
	Что должен содержать класс по-умолчанию? 19.07.2017, 10:52. Показов 1906. Ответов 16 Метки нет (Все метки) Если пишется класс, то обязательно должны быть реализованы следующие операции: 1) Конструктор по-умолчанию 2) Перегрузка оператора '=' 3) Конструктор копирования 4) Деструктор Как вы считаете? Является ли это "правилом хорошего тона"? И как должен по-умолчанию работать конструктор копирования, вызывать оператор '=' для двух объектов (`*this = other;)`, при условии что оператор '=' уже перегружен? 0

@John Prick 2098 / 1619 / 690 Регистрация: 27.07.2012 Сообщений: 4,853
	19.07.2017, 10:56	2
	Сообщение от anapshy Если пишется класс, то обязательно должны быть реализованы следующие операции: 1) Конструктор по-умолчанию 2) Перегрузка оператора '=' 3) Конструктор копирования 4) Деструктор Как вы считаете? Вовсе не обязательно. Очень часто то, что генерирует компилятор, именно то, что нужно от этих функций. 0

@anapshy 237 / 263 / 218 Регистрация: 14.11.2016 Сообщений: 988
	19.07.2017, 11:09 [ТС]	3
	John Prick, да, это знаю. Добавлено через 7 минут MrGlurk, zss, Croessmah, ildwine, как вы считаете? 0

@rikimaru2013 2549 / 1208 / 358 Регистрация: 30.11.2013 Сообщений: 3,826
	19.07.2017, 11:38	4
	anapshy, думаете красиво звать людей, которые не знают о вашей теме, получат уведомления и какую реакцию вы ожидаете? Не пойму откуда у вас позитивная репутация с таким отношениям к людям. А по делу, нету обязательств у программистов делать перегрузку, если она не необходима 1

@TRam_ зомбяк 1580 / 1214 / 345 Регистрация: 14.05.2017 Сообщений: 3,936
	19.07.2017, 12:00	5
	rikimaru2013, вопрос был чуть в другом - можно ли использовать в конструкторе копирования оператор присваивания. Лично моё мнение - можно, но это будет менее производительно. В конструкторе копирования проверять `this == &other` не требуется (т.к. это конструктор, его объект ранее не существовал, а значит не мог быть передан по ссылке), ну и для полей вызывать копирующий конструктор вместо оператора присваивания тоже будет рациональнее. Добавлено через 2 минуты Потому что иначе поля вначале инициализируются по-умолчанию, а только потом скопируются. 1

@zss Модератор 12789 / 10243 / 6152 Регистрация: 18.12.2011 Сообщений: 27,433
	19.07.2017, 12:16	7
	Сообщение было отмечено anapshy как решение Решение Мое мнение о том, что должен содержать класс, я изложил создав образец: Образцы (шаблоны) программ для типовых задач 1

@inovarka 0 / 0 / 0 Регистрация: 01.04.2017 Сообщений: 15
	19.07.2017, 12:24	9
	3) Конструктор копирования Разве его стоит писать если нету динамических полей? 0

Croessmah Неэпический 17807 / 10579 / 2042 Регистрация: 27.09.2012 Сообщений: 26,609 Записей в блоге: 1
	19.07.2017, 12:25	10
	Сообщение от TRam_ Речь о различии Причем здесь это? Вот цитаты: Сообщение от TRam_ конструкторе копирования проверять this == &other не требуется Сообщение от TRam_ т.к. это конструктор, его объект ранее не существовал, а значит не мог быть передан по ссылке В конструктор копирования может быть передан тот же объект, который конструируется. Не проверив, получим невалидный объект. правда, делать так никто не будет. Я об этом уже писал на форуме с примером, можете поискать. 2

@anapshy 237 / 263 / 218 Регистрация: 14.11.2016 Сообщений: 988
	19.07.2017, 12:25 [ТС]	11
	rikimaru2013, а что не так-то? Что по вашему плохого в том, что я хочу услышать совета от людей, которые уже не 2, и не 5 лет в данной сфере (не считая времени отведенного на изучение)? 0

Croessmah Неэпический 17807 / 10579 / 2042 Регистрация: 27.09.2012 Сообщений: 26,609 Записей в блоге: 1
	19.07.2017, 12:36	12
	Сообщение от Croessmah Я об этом уже писал на форуме с примером, можете поискать. Во: Отличие перегрузки operator() от конструктора 1

@anapshy 237 / 263 / 218 Регистрация: 14.11.2016 Сообщений: 988
	19.07.2017, 13:18 [ТС]	15
	TRam_, как я понимаю все способы верные, спасибо 0

@rikimaru2013
	19.07.2017, 13:57 #16
	Не по теме: Сообщение от anapshy rikimaru2013, а что не так-то? Что по вашему плохого в том, что я хочу услышать совета от людей, которые уже не 2, и не 5 лет в данной сфере (не считая времени отведенного на изучение)? так выделите все ники в разделе С++. Так ведь больше профита будет - вы согласны? 0

@anapshy 237 / 263 / 218 Регистрация: 14.11.2016 Сообщений: 988
	19.07.2017, 14:02 [ТС]	17
	rikimaru2013, прислушаюсь к вашему совету 0

Что должен содержать класс по-умолчанию?

Решение