Конструктор копирования

[Сtrl]

Поля класса:

int field1;
int field2;

В конструктор копирования объект передается по ссылке:

CMyClass::CMyClass(const CMyClass& value)
{
	field1 = value.field1;
	field2 = value.field2;
}

Помимо этого, для класса перегружена операция сложения:

CMyClass& CMyClass::operator * (const CMyClass& value)
{
	CMyClass result(*this); /* тут конструктор копирования работает хорошо */
	result.field1 += value.field1;
	result.field2 += value.field2;
	return
		result;
}

Но когда я пытаюсь использовать это вместе:

CMyClass Object1(2,5); /* создан первый объект, поля класса проинициализированы */
CMyClass Object2 (Oblect1 + Object1); /*тут конструктор дает сбой. Потому что в итоге в Object2 поля равны -858993460, то есть как-будто их не инициализировали */

Почему я уверен, что лажает именно конструктор копирования? Дело в том, что если его убрать, то создаваемый автоматически конструктор копирования исправно работает!
Путем пошагового исполнения я отследил, что оператор сложения корректно возвращает значение.
Какие будут мысли по этому поводу?

[netrino]

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

CMyClass& CMyClass::operator * (const CMyClass& value)
{
	CMyClass result(*this); /* тут конструктор копирования работает хорошо */
	result.field1 += value.field1;
	result.field2 += value.field2;
	return
		result;
}

Красный амперсанд нужно убрать, нельзя возвращать ссылки на локальные переменные

[Сtrl]

Ох, заработало. Спасибо!
Остается только вопрос: почему тогда создаваемый автоматически конструктор невозбранно работал со ссылкой на локальную переменную?

[netrino]

Вероятнее всего, компилятор производил некие оптимизации при неопределённом конструкторе копирования, либо же встраивал его в код, и, таким образом избавившись от вызова, не затиралось значение result из стека. Чтобы сказать конкретнее нужно смотреть под отладчиком, но особого смысла в этом не вижу, так как в любом случае, возвращать ссылки/указатели на локальные автоматические переменные функции нельзя.

[Сtrl]

netrino, весьма исчерпывающее объяснение. Благодарю.

[zeroname]

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

Ох, заработало. Спасибо!
Остается только вопрос: почему тогда создаваемый автоматически конструктор невозбранно работал со ссылкой на локальную переменную?

А почему вы решили, что конструктор не может получать в качестве аргумента ссылку на локальную переменную? Как я понимаю, во время работы конструктора переданная ему по ссылке локальная переменная еще существует, и соответственно проблем не возникает...

[zeroname]

Что-то я стормозил и не въехал сразу в суть вопроса. Прошлый пост я сделал не подумав. Сорри...