Перегруженный оператор ++ не работает в C++

Question

Может кто-нибудь объяснить мне, почему мой перегруженный ++ (предварительная версия) не обновляет значение? Отрывок, как это:

circle circle:: operator++() 
{ 
    Area = Area * 2.0; 
    return *this; 
} 
///////////////////////////// 

int main() 
{ 
    class circle c1(4, 1, -1), c2(12, 4, 6); 
    c1.output(); 
    c1++; 
    c1.output(); 

    system("pause"); 
    return 0; 
} 

Answer 1

signature вашего оператора перегрузки должны быть:

circle& operator++(); // return by reference and this is prefix. 

Но вы используете суффикса, так и должно быть:

circle operator++ (int); // int is unused 

Изменение подписи является недостаточно, поскольку вы реализуете префиксную логику, напрямую изменяя значение без сохранения начального значения. Поэтому, если вы используете оператор postfix с вашей реализацией в комбинированном выражении, например (c++).output(), он не будет уважать ожидаемую семантику.

Здесь implemetnation обоих версии:

circle& operator++() { // prefix 
    Area = Area * 2.0; // you can change directly the value 
    cout << "prefix"<<endl; 
    return *this;  // and return the object which contains new value 
} 

circle operator++ (int) { // postfix 
    circle c(*this);  // you must save current state 
    Area = Area * 2.0; // then you update the object 
    cout << "postfix"<<endl; 
    return c;   // then you have to return the value before the operation 
} 

а вот online demo, чтобы показать разницу между обоими.

Answer 2

Это потому, что вы перегружаете префикс и вызываете постфикс. Вам необходимо позвонить ++c1;. Для использования c1++; вам нужно перегружать суффикс, а также:

circle operator++ (int); 

Answer 3

Вот как версия префикса и после исправления. И вы можете добавить код в случае вызова типа c1 ++ (1); (конечно, если необходимо)

circle circle:: operator++() // prefix version 
{ 
    Area = Area * 2.0; 
    return *this; 
} 

circle& circle::operator++(int n) { //postfix version 
    if(n != 0)    // Handle case where an argument is passed. 
     //some code 
    else 
     Area = Area * 2.0;  // Handle case where no argument is passed. 
    return *this; 
} 


int main() 
{ 
    class circle c1(4, 1, -1), c2(12, 4, 6); 
    c1.output(); 
    c1++; 
    ++c1; 
    c1.output(); 

    system("pause"); 
    return 0; 
}