[C++] Przeciążenie operatora + problem.

misse OP · 29-03-2014 10:43

Witam,
Mam problem bo muszę napisac w C++ klasę liczb wymiernych, i wszystko pięknie tylko że nie działa mi przeciążenie operatora+. Dla ułatwienia na razie uprościłem kod żeby dodawał do siebie stopnie mianowników 2 funkcji wymiernych. Dla wartości 1+1 daje wynik 10679817 (około.) xD także coś jest nie tak..

Kod:

int rational::operator+(const rational& B)
{ 
int a=0;
int b=0;
if (B.stm+stl > B.stl+stm) {a=B.stm+stl;} else {a=B.stl+stm;}
b = B.stm + stm;
    return b;
}

I wogóle nie rozumiem samej teorii dlaczego przeciazenie operatora + ma być jednoargumentowe..? jak robie dwuargumentowe to kompilator wywala błąd że może mieć tylko zero albo jeden argument. Troche to dla mnie nielogiczne.

**zakius** · 29-03-2014 10:56

Operator jest wykonywny na obiekcie klasy rational, to jest ta pierwsza wartosc, a po drugiej stronie masz argument. OOP tak ma, przejście nie zawsze jest bezbolesne. w kod nie zerkne, bo na telefonie jestem

misse OP · 29-03-2014 11:10

No ja mam tak że mam 2 obiekty rational A i B, każdy z nich ma w sobie zmienną stm (stopien mianownika). I chcę zapisac sumę tych stopni w nowej zmiennej int c.
Dlaczego nie działa: c = stm + B.stm? B.stm = 1 to wiadomo jest ok, ale dla drugiego argumentu (samo stm) daje jakas liczbe z kosmosu jakby nie wiedzial o co chodzi.

Ewande · 29-03-2014 11:25

Musisz mieć błąd w innej części programu, bo nie widzę powodu, by miało tu zwracać coś dziwnego.
Co do przeciążania operatorów, to są dwa sposoby, które podałem poniżej.
Mój przykład:

main.cpp

Kod:

#include <iostream>

using std::cout;


class Rational
{
public:
    Rational(int iNum = 0, int iDenom = 1) : i_num(iNum), i_denom(iDenom) {}


    int operator+(const Rational& other) { return i_denom + other.i_denom; }


    friend int operator+(const Rational& fst, const Rational& snd) { return fst.i_denom + snd.i_denom; }


private:
    int i_num;
    int i_denom;
};


int main()
{
    Rational a(5, 12), b(10, 20);
    cout << a + b;
    getchar();
}

**zakius** · 29-03-2014 13:41

dla absolutnej pewności polecam do odwolywania się do własnych ... zapomnialem słowa, powiedzmy składników wskaźnikiem na siebie this, nie wiem, czy pomoże, ale jest bardziej jednoznacznie

misse OP · 29-03-2014 17:49

Dobra ruszyłem się dosyć dużo naprzód. mam już odejmowanie, dodawanie, mnozenie, i rational*double.
Teraz mam problem taki, że oprócz mnożenia rational*double mam też zrobić double*rational. Wiem że to głupie pytanie ale jak to zrobić?
Tak wygląda rational*double, ale co w tym zmienić aby mnożenie było przemienne?

Kod:

rational rational::operator*(const double& B) const{
	rational sum(stl,stm);
	for(int i=0;i<51;i++){sum.licznik[i]=licznik[i]*B;}
	for(int i=0;i<51;i++){sum.mianownik[i]=mianownik[i];}
	return sum;
}

**zakius** · 29-03-2014 17:56

przeciąż op * doubla, nie wiem jeszcze jak co prawda, ale może się udać :D
ewentualnie zrób nową klasę z doublem :f

spróbuj operator*(double, rational) czy coś?
bo double::operator* raczej nie

Ewande · 29-03-2014 22:26

Użyj sposobu drugiego, który podałem (z funkcją zaprzyjaźnioną). Tam możesz przeciążyć zarówno double*rational, jak i rational*double.

Temat: [C++] Przeciążenie operatora + problem.

Narzędzia tematu

Oceń ten temat

[C++] Przeciążenie operatora + problem.

Informacje o temacie

Użytkownicy przeglądający temat

Podobne tematy

Jakiego operatora wybrac?

zmiana operatora android

Zmiana operatora

Telefony od operatora

Jak ustawić na stałe operatora?

Zakładki

Zakładki

Zasady postowania