Operator overloading für vektoren C++

[Ada (Gast)]

Ich versuche gerade den: << Operator, für Vektoren zu deklarieren und zu 
definieren, bekomme bei der Ausführung aber folgende Fehlermeldung:

error: no match for 'operator<<' (operand types are 'std::ostream {aka 
std::basic_ostream<char>}' and 'std::vector<Book>')




Auszug des Quellcodes:

// Library.h

class Library {
public:
    Library();
    void showBooks() {cout << mBook;}   // In dieser Zeile Fehlermeldung
    
private:
    vector<Book> mBook;
};

ostream& operator<<(ostream& os, const vector<Book>& b);

// Library.cpp

ostream& operator<<(ostream& os, const vector<Book>& b)
{
    for(auto& book : b)
        os << book;
    return os;
}

[Ada (Gast)]

Wo liegt mein Fehler?

[g457 (Gast)]

> Wo liegt mein Fehler?

- falsche Reihenfolge
- grund- und nutzlose, fehlerverursachende Implementierung im Headerfile

HTH

[Ada (Gast)]

g457 schrieb:
> - falsche Reihenfolge
> - grund- und nutzlose, fehlerverursachende Implementierung im Headerfile

Gehts auch genauer bitte

[tictactoe (Gast)]

Wäre

friend ostream& operator<<(ostream& os, const Library& b);

nicht viel logischer?

[Ada (Gast)]

tictactoe schrieb:
> Wärefriend ostream& operator<<(ostream& os, const Library& b);
> nicht viel logischer?

Wenn ich so drüber nachdenke, ja du hast Recht.

[Sebi (Gast)]

Ada schrieb:
> Gehts auch genauer bitte

Dein operator<< steht im Header nach der Definition deiner showBooks() 
Funktion. Pack die Deklaration deines Operators davor, dann kennt der 
Compiler den Operator auch.

[Ada (Gast)]

Habe es jetzt anders gelöst (in Anlehnung an den Vorschlag von 
tictactoe), dann spielt die Reihenfolge auch keine Rolle mehr (danke für 
den Hinweis Sebi):

// Library.h

class Library {
public:
    Library();
    void showBooks() {return mBook;}   // In dieser Zeile Fehlermeldung
    
private:
    vector<Book> mBook;
};

ostream& operator<<(ostream& os, const Library& l)

// Library.cpp

ostream& operator<<(ostream& os, const Library& l)
{
    for( auto& book : l.showBooks() )  // l.showBooks() entspricht dem Vektor mBook
        os << book;                
    return os;
}

g457 schrieb:
> - falsche Reihenfolge
> - grund- und nutzlose, fehlerverursachende Implementierung im Headerfile
>
> HTH

Hört sich so an als sei mein Programmierstil nicht der beste, vielleicht 
kann mir jemand ne gute Referenz nennen damit ich mich da verbessern 
kann?