Оптимизировать перегрузку операторов

[Иллидан]

Перегружаю в классе операторы '+','*','/','-'. Сталкнулся с такой проблемой: для всех опреторов у меня идет один и тот же код. Кроме одного действия: для оператора +, это действие '+', для оператора '-', это - и т.д. Получается, в программе будет много повторяющихся фрагметов, что не желательно. Можно ли как-то это усовершенствовать?

[B_N]

Как это так "один и тот же"? Покажите код, что гадать-то.

[Иллидан]

Ну что-то типа такого.
Перегрузка плюса.

TMoney TMoney::operator +(TMoney A)
{

float a,temp;
TMoney ftemp; 
// тут идет одинаковый код 
//------------------------- 
temp=temp+a;// отличие только в этой строчке
//------------------------
//тут идет одинаковый код

return ftemp;
}

Перегрузка минуса.

TMoney TMoney::operator -(TMoney A)
{

float a,temp;
TMoney ftemp; 
// тут идет одинаковый код 
//------------------------- 
temp=temp-a;// отличие только в этой строчке
//------------------------
//тут идет одинаковый код
return ftemp;
}

[B_N]

А почему вы минус перегружаете плюсом??? И зачем возвращаете какой-то ftemp, когда, по идее, нужно возвращать *this?

[Иллидан]

почему вы минус перегружаете плюсом???

В каком смысле '+', '-' ?

И зачем возвращаете какой-то ftemp, когда, по идее, нужно возвращать *this?

Не очень понял, на сколько я знаю, обычно *this возвращают, когда перегружают унарные операторы(например, ++). Тут же у меня идет обычная перегрузка '+'. То есть, в основной программе, у меня написанно, например:
TMoney A,B,C;
C=A+B;//равенство, я тоже перегружаю, разумеется

[B_N]

Сообщение от Иллидан

В каком смысле '+', '-' ?

Да то ли я посмотрел не туда, то ли солнышко отсвечивает , в общем показалось, что у Вас '+' в обоих случаях, так что на это можно не обращать внимания.

Сообщение от Иллидан

Не очень понял, на сколько я знаю, обычно *this возвращают, когда перегружают унарные операторы(например, ++). Тут же у меня идет обычная перегрузка '+'. То есть, в основной программе, у меня написанно, например:
TMoney A,B,C;
C=A+B;//равенство, я тоже перегружаю, разумеется

Тоже, видимо, не разглядел, где temp превращается в ftemp. А в целом, что такого страшного в одинаковых участках, если там кода всё равно 3-4 строчки. В крайнем случае можно в минусе сделать "return operator+( - A );"....

[Иллидан]

А в целом, что такого страшного в одинаковых участках, если там кода всё равно 3-4 строчки.

Там у меня не три строчки кода, а гораздо больше, просто, я не стал их приводить, что-бы захламлять не нужным код. И к тому же аналогично с операторами делить и умножить.

В крайнем случае можно в минусе сделать "return operator+( - A );"....

Не совсем понял, это что?

[B_N]

Сообщение от Иллидан

Там у меня не три строчки кода, а гораздо больше, просто, я не стал их приводить, что-бы захламлять не нужным код. И к тому же аналогично с операторами делить и умножить.

Хм... странно, в этих "деньгах" обычно кода кот наплакал, я из этого и исходил.

Сообщение от Иллидан

Не совсем понял, это что?

Сложение и унарный минус...

[Иллидан]

Ну такие у меня уж "деньги", специфические...

Сложение и унарный минус...

В смысле вы предлагаете так оформить перегрузку минуса?

TMoney TMoney::operator -(TMoney A)   
{  

return operator+( - A );

}

Это будет работать?

[B_N]

Сообщение от Иллидан

Ну такие у меня уж "деньги", специфические...

Сообщение от Иллидан

В смысле вы предлагаете так оформить перегрузку минуса?

TMoney TMoney::operator -(TMoney A)   
{  
	return operator+( - A );
}

В качестве варианта, если Вы так против большого количества кода.

Сообщение от Иллидан

Это будет работать?

Не просто будет, а работает. Не забудьте только определить унарный минус. Если опустить ненужные детали, это выглядит примерно вот так:

class CMoney
{
public:
	CMoney(void);
//.........................
	~CMoney(void);

	const CMoney operator-();
	const CMoney CMoney::operator-(CMoney arg);
	const CMoney CMoney::operator+(CMoney arg);
//.........................
//.........................
}

//----------------------------------

const CMoney CMoney::operator-(CMoney arg)
{
	return operator +(-arg);
}

const CMoney CMoney::operator-()
{
	CMoney tmp = *this;
	tmp.nRub = - tmp.nRub;
	return tmp;
}

//----------------------------------

int main()
{
	CMoney arg1, arg2;

	std :: cin >> arg1;
	std :: cin >> arg2;

	std :: cout << arg1 - arg2;

	return 0;
}