Перегрузка оператора (-) устанавливает переменные в 0

Question

Когда я вызываю r--; Мой объект сбрасывает значения до 0. Любые идеи?

class MyClass : Superclass { 
private: 
    int length; 
    int width; 

public: 
    MyClass() { 
     length = 0; 
     width = 0; 
    } 

    MyClass (int x, int y):Superclass(x/2,y/2){ 
     length = x; 
     width = y; 

    } 

    MyClass operator--(int) { 
     MyClass temp = *this; 
     temp.length --; 
     temp.width --; 
     return temp; 
    }; 
}; 

Создание и пытается класс:

MyClass *r = new MyClass(2,3); 
r--; // now length and width = 0 (should be 1,2) 

Answer 1

Во-первых, оператор не уменьшаем объект это называется, но копия он собирается вернуться. Он должен оставить это в покое (чтобы вернуть предыдущее значение) и уменьшаем объект:

MyClass temp = *this; 
this->length--; // this-> is optional 
this->width--; 
return temp; 

Во-вторых, r является указателем. r-- уменьшает указатель, а не объект, на который указывает, оставляя его указаным на недопустимое расположение памяти. Разделение его впоследствии дает неопределенное поведение.

Я не знаю, почему вы используете new здесь; вы почти наверняка просто хотите переменную:

MyClass r(2,3); 
r--; // should behave as expected. 

Если вы действительно хотите указатель на какой-то причине вам придется разыменования его, чтобы получить объект:

(*r)--; 

и не забыть для удаления объекта после его завершения. И не раньше.