нет матча для оператора *

Question

Читаем Эффективное использование C++ (Scott Meyers) и получает ошибку «не подходят для оператора *» при попытке компиляции следующего кода из этой книги:

rational.h

class rational 
{ 
    private: 
     int num; 
     int den; 
    public: 
     rational(int n = 0, int d = 1); 
     int getNum() const {return num;} 
     int getDen() const {return den;} 
}; 

rational.cpp

#include "rational.h" 

rational::rational(int n, 
        int d) 
    :num(n), 
    den(d) 
{} 

const rational operator*(const rational &lhs, 
         const rational &rhs) 
{ 
    return rational(lhs.getNum()*rhs.getNum(), 
         lhs.getDen()*rhs.getDen()); 
} 

main.cpp

#include "rational.h" 
int main() 
{ 
    rational r1(1,2); 
    rational r2; 
    r2 = 2*r1; 
    r2 = r1*3; 
    return 0; 
} 

Может кто-нибудь объяснить, почему это происходит?

Answer 1

Вы не объявили operator* в своем файле заголовка, поэтому он не отображается в main.cpp.

Answer 2

Вы забыли рассказать пользователям о существовании вашего оператора:

rational.h

... 
const rational operator*(const rational &lhs, 
         const rational &rhs); 
... 

Как правило, в C, а также в C++, мы говорим о "определений" и «декларация». Объявления - это аннотации, чтобы сделать что-то видимым для кого-то другого, но сами по себе они ничего не делают. Определения являются entititiess, что на самом деле сделать что-то:

int foo();    // <- we call it "declaration" 
int foo() { return 0; } // <- we call it foo's "definition" 

В коде нет декларации operator* видимого в main.cpp, так что вам нужно предоставить один где-то (в идеале в вашем рациональны в заголовке).

Как советы стиль: Почти во всех случаях, если конструктор принимает встроенные типы, вы хотите сделать его явным:

explicit rational (int, int); 

Это предотвращает иногда тонкие ошибки, потому что (в вашем случае) rational s может быть непреднамеренно созданный (см. Automatic Conversions).