Ich bin noch ein C++ Anfänger und beschäftige mich gerade mit dem Überladen von Operatoren. In meinem Programm handelt es sich um Brüche und für diese wollte ich einige Operatoren überladen. Nun bekomme ich jedoch folgende Fehlermeldung, mit der ich nichts anzufangen weiss:

Fraction.cpp:28:56: Fehler: Die �bergabe von �const Fraction� als �this�-Argument von �int Fraction::getNumerator()� streicht Qualifizierer [-fpermissive]

Fraction.cpp:29:60: Fehler: Die �bergabe von �const Fraction� als �this�-Argument von �int Fraction::getDenominator()� streicht Qualifizierer [-fpermissive]

Hier ist meine Header-Datei

/* 
 * File:   Fraction.h
 * Author: Xarsis
 *
 * Created on 24. Juni 2012, 15:48
 */

#ifndef FRACTION_H
#define	FRACTION_H

#include <iostream>

using namespace std;

class Fraction {
public:
    Fraction();
    Fraction(int numerator, int denominator);
    int getNumerator();
    int getDenominator();
    Fraction operator*(const Fraction& rhs);
    void operator<<(const Fraction& fraction);
private:
    int numerator;
    int denominator;
};

#endif	/* FRACTION_H */

und hier meine .cpp

/* 
 * File:   Fraction.cpp
 * Author: Xarsis
 *
 * Created on 24. Juni 2012, 15:48
 */

#include "Fraction.h"

Fraction::Fraction() {

}

Fraction::Fraction(int numerator, int denominator) {
    this->numerator = numerator;
    this->denominator = denominator;
}

int Fraction::getNumerator() {
    return this->numerator;
}

int Fraction::getDenominator() {
    return this->denominator;
}

Fraction Fraction::operator*( const Fraction& rhs) {
    Fraction result(this->numerator * rhs.getNumerator(),
                    this->denominator * rhs.getDenominator());
    return result;
}

void Fraction::operator <<(const Fraction& fraction) {
    cout << fraction.getNumerator() << " / " << fraction.getDenominator();
}

Vielen Dank schonmal für die Hilfe!

P.S.: Du weißt, dass du das this-> nicht brauchst? Bist du auch sicher, dass der leere Konstruktor tut, was du möchtest? Derzeit ermöglicht er das uninitialisierte erstellen von Objekten deiner Klasse, obwohl deine Klasse anscheindend doch initialisiert erstellt werden möchte.

int getNumerator() const {return numerator;}
int getDenominator() const {return denominator;}

ansonsten:
schau dir mal die Initialisierungsliste an.

PS: using namespace std; gehört nicht in den header.

gruß
syntax

@SeppJ
Ja der leere Konstruktor war in diesem Fall so gewollt und das mit dem this-> weiss ich zwar das ich es auch weg lassen könnte aber mir wurde geraten das so zu machen, damit ich auch gleich sehe das ich auf Sachen zugreiff in dem Objekt indem ich mich befinde. Aber mir fällt grad auf würde das denn in meinem überladenen Konstruktor funktionieren wenn man da schreibt: numerator = numerator ? Woher weiss denn dann der Compiler was gemeint ist? Auf jeden nochmal danke für die Hilfe!

@Syntax
Auch dir ein Dankeschön!! Wieso gehört der namespace std nicht in den Header? Hätte ich eine Methode die ich sag mal string oder ostream zurückgeben würde und ich hätte alles included dann brauch ich doch den namespace std sonst würde er meinen string oda ostream nicht erkennen oder sehe ich da was falsch?

MfG Xarsis

Aber mir fällt grad auf würde das denn in meinem überladenen Konstruktor funktionieren wenn man da schreibt: numerator = numerator ? Woher weiss denn dann der Compiler was gemeint ist?

Da hast du recht, darum macht man so was:

private:
    int a;        //auch schon gesehen: _a, _b
    int b;
public:
    Classenname(int aa, int bb) : a(aa), b(bb) {}

Ich mache eigentlich nit this->.

Xarsis schrieb:

Für den <<operator muss ich nochmal schauen, da hatte ich glaub schon einen total falschen Ansatz. Laut Internet wird das mit ostream gemacht.

Ja der ist doch schon überladen:

std::ostream& operator <<(std::ostream& rhs, T& lhs);  //T steht fuer deinen Datentyp

Xarsis schrieb:

Wieso gehört der namespace std nicht in den Header?

Weil es zu Problemen führen kann. Es gibt alternativen:

using std::cout;
using std::cin;
cout<<...;
cin>>...;

oder:

std::cout<<...;
std::cin>>...;

gruß
syntax

Der Fehler ist:

Fraction.cpp:33:63: Fehler: �std::ostream& Fraction::operator<<(std::ostream&, const Fraction&)� muss genau ein Argument nehmen

Header

/* 
 * File:   Fraction.h
 * Author: Xarsis
 *
 * Created on 24. Juni 2012, 15:48
 */

#ifndef FRACTION_H
#define	FRACTION_H

#include <iostream>
#include <ostream>

class Fraction {
public:
    Fraction();
    Fraction(int numerator, int denominator);
    int getNumerator() const;
    int getDenominator() const;
    Fraction operator*(const Fraction& rhs);
private:
    int numerator;
    int denominator;
};

std::ostream& operator<<(std::ostream &os, const Fraction &rhs);

#endif	/* FRACTION_H */

CPP

/* 
 * File:   Fraction.cpp
 * Author: Xarsis
 *
 * Created on 24. Juni 2012, 15:48
 */

#include "Fraction.h"

using namespace std;

Fraction::Fraction() {

}

Fraction::Fraction(int numerator, int denominator) {
    this->numerator = numerator;
    this->denominator = denominator;
}

int Fraction::getNumerator() const {
    return this->numerator;
}

int Fraction::getDenominator() const {
    return this->denominator;
}

Fraction Fraction::operator*(const Fraction& rhs) {
    Fraction result(this->numerator * rhs.getNumerator(),
            this->denominator * rhs.getDenominator());
    return result;
}

ostream& Fraction::operator<<(ostream &os, const Fraction& rhs) {
    return os << rhs.getNumerator() << " / " << rhs.getDenominator();
}

Du musst diese Funktion als friend deklarieren, wenn du sie in die Klasse packst.

friend std::ostream& operator<<(std::ostream& os, Fraction& rhs);

std::ostream& operator<<(std::ostream& os, Fraction% rhs){

//Das hier stimmt bei dir
}

Ansonste hier ein guter tut.
http://www.c-plusplus.net/forum/232010

gruß
syntax

MfG Xarsis