Перегрузка ++

[counter]

нужно перегрузить оператор ++ , чтобы он работал в префиксной и постфиксной формах

вот класс для которого нужно сие сотворить

class My_Int {

	int num;

public:

	My_Int(int n=0) { num=n; }
		
	My_Int operator++();
	My_Int operator++(int notused);

	My_Int operator--();
	My_Int operator--(int notused);

	friend ostream& operator<<(ostream &stream,My_Int ob);
	friend istream& operator>>(istream &stream,My_Int &ob);
    
};

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

My_Int My_Int::operator ++()

{
	num++;
	return *this;
};

My_Int My_Int::operator ++(int notused)

{
	int tmp=this->num;
	++tmp;
	this->num=tmp;
	return *this;
};

правильно?
или есть недочеты?

[MaTBeu]

С постфиксным вроде все нормально, а вот префиксный чуть не так пишется. Префиксный

//прототип
My_Int &operator++();
//реализация
My_Int &operator++()
{
    this.num++;
    return *this;
}

Тоесть перегруженная функция должна возвращать ссылку на измененный обьект. С декрементом та же история.

[counter]

все понял.

спасибо за ответ!

[como]

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

My_Int My_Int::operator ++(int notused)

{
	int tmp=this->num;
	++tmp;
	this->num=tmp;
	return *this;
};

Так работать не будет. Ты возвращаешь объект My_Int с уже приращенным значением, а надо с предыдущим (старым) значением. Правильная реализация вот такая:

My_Int My_Int::operator++(int)
{
    My_Int old_value(*this);
    this->operator++();
    return old_value;
}

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

Еще один совет. Не пиши параметр notused. Если параметр не используется, то его имя нужно просто опускать, как в моем примере.