2012-03-29 3 views
2

Я представил приведенный ниже код. Компилятор выдает ошибку при перегрузке перегруженного оператора постфикса. Он отлично работает на перегруженном префиксном операторе. ОшибкаПерегрузка оператором ostream на перегруженном операторе приращения/уменьшения постфикса

error: no match for ‘operator<<’ in ‘std::cout << cDigit.Digit::operator++(0)’ 

Код

#include <iostream> 

using namespace std; 

class Digit 
{ 
private: 
    int m_nDigit; 
public: 
    Digit(int nDigit=0) 
    { 
     m_nDigit = nDigit; 
    } 

    Digit& operator++(); // prefix 
    Digit& operator--(); // prefix 

    Digit operator++(int); // postfix 
    Digit operator--(int); // postfix 

    friend ostream& operator<< (ostream &out, Digit &digit); 

    int GetDigit() const { return m_nDigit; } 
}; 

Digit& Digit::operator++() 
{ 
    // If our number is already at 9, wrap around to 0 
    if (m_nDigit == 9) 
     m_nDigit = 0; 
    // otherwise just increment to next number 
    else 
     ++m_nDigit; 

    return *this; 
} 

Digit& Digit::operator--() 
{ 
    // If our number is already at 0, wrap around to 9 
    if (m_nDigit == 0) 
     m_nDigit = 9; 
    // otherwise just decrement to next number 
    else 
     --m_nDigit; 

    return *this; 
} 

Digit Digit::operator++(int) 
{ 
    // Create a temporary variable with our current digit 
    Digit cResult(m_nDigit); 

    // Use prefix operator to increment this digit 
    ++(*this);    // apply operator 

    // return temporary result 
    return cResult;  // return saved state 
} 

Digit Digit::operator--(int) 
{ 
    // Create a temporary variable with our current digit 
    Digit cResult(m_nDigit); 

    // Use prefix operator to increment this digit 
    --(*this);    // apply operator 

    // return temporary result 
    return cResult;  // return saved state 
} 

ostream& operator<< (ostream &out, Digit &digit) 
{ 
    out << digit.m_nDigit; 
    return out; 
} 

int main() 
{ 
    Digit cDigit(5); 
    cout << ++cDigit << endl; // calls Digit::operator++(); 
    cout << --cDigit << endl; // calls Digit::operator--(); 
    cout << cDigit++ << endl; // calls Digit::operator++(int); //<- Error here?? 
return 0; 
} 

ответ

6

Ваш operator<< должен принять его параметр Digit по константной ссылке:

ostream& operator<< (ostream &out, const Digit &digit) 

Это необходимо потому, что здесь Digit::operator++(int) возвращает временный объект, который не может быть передается функции, которая принимает неконстантную ссылку.

+0

+1 Да, это было! Я удивлен, что большинство руководств по C++ не упоминают, что при перегрузке оператора. – enthusiasticgeek

+0

Хм, может быть, нам нужно написать несколько лучших уроков :-) Хорошее эмпирическое правило - если вы не намерены его модифицировать, сделайте его const. – bstamour

+0

Мне бы очень понравились компиляторы, которые были бы разумными для начинающих/программистов на промежуточном уровне с ошибкой и предложением, вроде «sir/ma'am, вы забыли спецификатор const?» а не какая-то мешанина необоснованной сложности (к счастью, не так много в этом случае). Надеюсь, мы увидим их в будущем. :) – enthusiasticgeek