Ułamki - Klasy C++

Question

Cześć, do klasy Fraction dopisałem przeciążenia operatorów umożliwiające operacje na ułamkach (dodawanie, mnożenie, dzielenie, odejmowanie, rzutowanie na double, porównywanie), lecz niestety jeszcze do końca nie orientuję się w tym zagadnieniu i program nie działa. Czy mógłby ktoś rzucić okiem na kod (przeciążenie operatorów), aby znaleźć ewentualne błędy? Z góry dzięki!

#include <iostream>

class Fraction {
  int licznik;
  int mianownik;

  public:
    Fraction(int licz, int mian) {
      if (mian != 0) {
        licznik = licz;
        mianownik = mian;
        if (mianownik < 0) {
          licznik *= (-1);
          mianownik *= (-1);
        }
      } else {
        std::cout << "Mianownik nie moze byc rowny 0 !!";
        exit(0);
      }
    }

  Fraction(const Fraction & f1) {
    licznik = f1.licznik;
    mianownik = f1.mianownik;
  }

  int getLicznik() {
    return licznik;
  }

  int getMianownik() {
    return mianownik;
  }

  void setLicznik(int x) {
    licznik = x;

  }

  void setMianownik(int y) {
    if (y != 0) {
      mianownik = y;
    } else {
      std::cout << "0 nie moze byc w mianowniku!" << std::endl;
      exit(0);
    }
  }

  void simplify(int a, int b) {
    while (a != b)
      if (a > b) {
        a -= b;
      }
    else {
      b -= a;
    }
    licznik /= a;
    mianownik /= b;
  }

  void wypisz() {
    std::cout << "Podany ulamek to: " << licznik << "/" << mianownik << std::endl;
  }

  float toFloat() {
    float dziesietny;
    dziesietny = float(licznik) / float(mianownik);
    return dziesietny;
  }
  
//  Dodawanie
  Fraction operator+(Fraction fraction) {
	Fraction resultFraction;
	if (this->mianownik == fraction.getMianownik()) {
		resultFraction.setLicznik(this->licznik + fraction.getLicznik());
		resultFraction.setMianownik(this->mianownik);
	} else {
		resultFraction.setLicznik((this->licznik * fraction.getMianownik()) + (fraction.getLicznik() * this->mianownik));
		resultFraction.setMianownik(this->mianownik * fraction.getMianownik());
	}
	return resultFraction;
}

// Odejmowanie
	Fraction operator-(Fraction fraction) {
	Fraction resultFraction;

	if (this->mianownik == fraction.getMianownik()) {
		resultFraction.setLicznik(this->licznik - fraction.getLicznik());
		resultFraction.setMianownik(this->mianownik);
	} else {
		resultFraction.setLicznik((this->licznik * fraction.getMianownik()) - (fraction.getLicznik() * this->mianownik));
		resultFraction.setMianownik(this->mianownik * fraction.getMianownik());
	}

	return resultFraction;
}

// Mnozenie
Fraction operator*(Fraction fraction) {
	Fraction resultFraction;
	resultFraction.setLicznik(this->licznik * fraction.getLicznik());
	resultFraction.setMianownik(this->mianownik * fraction.getMianownik());
	return resultFraction;
}

// Dzielenie
Fraction operator/(Fraction fraction) {
	Fraction resultFraction;
	resultFraction.setMianownik(this->mianownik * fraction.getLicznik());
	resultFraction.setLicznik(this->licznik * fraction.getMianownik());
	return resultFraction;
}
	
	
//	Rzutowanie na double
	Fraction(int licznik,int mianownik)
{
	this.licznik = (double)licznik;
	this.mianownik = (double)mianownik;
	}
	
//	Porownywanie ulamkow
	bool Fraction operator==(Fraction fraction) {
	return (this->licznik * fraction.getMianownik()) == (fraction.getLicznik() * this->mianownik);
}

};

int main() {
  return 0;
}

 

Answer 1

i program nie działa.

Rozwiń to "nie działa".

• Operatory, które nie zmieniają stanu obiektu this, powinny być const.
• Obiekty przekazuj w parametrach przez const referencję. Nie ma potrzeby robić kopii obiektów, skoro chcesz je tylko odczytać.

Comments

Tworzę obiekt i wtedy wtedy przy kompilacji wyskakują błędy.

---

No to zacytuj te błędy.

Ale najpierw zastosuj się do tego, co napisałem w punktach.

Answer 2

(kopia odpowiedzi z duplikatu pytania, który to został zamknięty)

Przede wszystkim skompiluj sobie ten kod i rozwiązuj błędy jeden po drugim ZACZYNAJĄC OD GÓRY. Tj. po każdej poprawce skompiluj ponownie i dopiero wtedy rozwiązuj kolejny problem (kompilatory C++ mają trochę efekt kuli śnieżnej, gdzie jeden błąd potrafi spowodować milion błędów w kolejnych linia; a jak się go poprawi, to one magicznie znikają).

Tak na szybko to:

• Brakuje mi tam jakiegos include<iostream>
• Masz dwa razy konstruktor (na górze i na dole klasy)
• Brakuje konstruktora który nie przyjmuje żadnych argumentów (zauważ, że masz w kodzie trochę miejsc gdzie tworzysz ułamek bez podania parametrów, np w metodach operator-, operator*, etc).

Po tych poprawkach przynajmniej się kompiluje - i jak do tego momentu dojdziesz, to musisz zacząć to porządnie testować :)

Comments

Yup, pisalem o tym ;)

• Masz dwa razy konstruktor (na górze i na dole klasy)

---

fakt pisałeś.

---

@Gynvael Coldwind, Dziękuję!! :)

---

@Gynvael Coldwind, A wie ktoś jak zrobić wypisywanie ułamka na konsole / do pliku za pomocą przeciążenia operatorów?

---

https://learn.microsoft.com/en-us/cpp/standard-library/overloading-the-output-operator-for-your-own-classes?view=msvc-170