kleines c++ problem mit ostream << operator

Hier die 3 Files die ich verwendet habe (MacOSX 10.6.8. + XCode4). Ich hab jetzt die Funktionen der Klasse Kreis auf die Schnelle gleich im h. File implementiert, wär aber besser im .cpp aufgehoben.

Kreis.h:

Code: CPP

#ifndef RECHNEN_H
#define RECHNEN_H

#define PI 3.14159256

#include <iostream>
#include <math.h>

class Kreis
{
public:
    Kreis(double radius) : r(radius) {};

    double getRadius() const { return r; }
    double getDurchmesser() const { return r * 2.0; }

    double getFlaeche() const { return PI * pow(getRadius(), 2.0); }

private:
    const double r;
};

std::ostream& operator << (std::ostream &os, const Kreis& kreis);

#endif // RECHNEN_H

Kreis.cpp:

Code: CPP

#include "Kreis.h"
#include <stdio.h>

using namespace std;

ostream& operator << (ostream &os, const Kreis& kreis)
{
    os << "Radius: " << kreis.getRadius() << endl;
    os << "Durchmesser: " << kreis.getDurchmesser() << endl;
    os << "Flaeche: " << kreis.getFlaeche() << endl;

    return os;
}

main.cpp:

Code: CPP

#include "Kreis.h"
#include <stdio.h>

using namespace std;

int main (int argc, const char * argv[])
{
    Kreis kreis(5);
    cout << kreis;

    return 0;
}

Das "using namespace std" aber bitte rausnehmen aus dem .h File

Dann verstehe ich aber nicht, welche Lösung du haben wolltest? :-)

Meinst du den stream Operator?

Den kannst eh wieder als friend in die Klasse ziehen. Allerdings bringt es halt nicht viel, weil du sowieso nicht direkt auf die private Member zugreifst vom stream Operator aus.

Den friend brauchst ja nur dann, wenn du auf private Details der Klasse ohne Getter-Funktionen zugreifen willst.

Spikx hat die Fehler eh schön zusammengefasst oben (bis auf das "using namespace std" halt, das gehört mMn nicht in ein .h File) . Den Rest deiner Lösung kannst/sollst du eh selber schreiben :-)

Zitat von PuhBär

(bis auf das "using namespace std" halt, das gehört mMn nicht in ein .h File)

Würd ich eh auch net machen

Es seit übrigens noch gesagt dass:
using namespace std;
nicht unbedingt der sauberste Stil ist weil es in einigen Fällen (vorallem größeren Projekten) auch vorkommt, dass man mit eigenen oder weiteren Namensräumen arbeitet.

Daher entweder immer den namespace davor schreiben:
zb.: std:: cout << ....

oder so:
using std::cout;
using std::cin; ....

cout << ...
cin >> ...