Bruch Problem (int/double)

[xavie33]

Kann mir jemand erklären warum bei Eingabe eines double Wertes keine Ausgabe erscheint und bei einer int Eingabe schon?

bruch1.rational(14, 2) --> Richtig
bruch1.rational(14.2, 2.2) --> Fehler

public class Rational {
	
	double zaehler;
	double nenner;

	// Konstruktor
	
	void rational(double z, double n) throws Exception{
		zaehler = z;
		if (n <= 0) {
			throw new Exception("Nenner mit 0 ist nicht erlaubt!");
			
		} else {
			nenner = n;
		}
	}
	// Subtraktion

	public Rational subtraktion(Rational bruch2) {
		Rational ergebnis = new Rational();
		ergebnis.zaehler = zaehler * bruch2.nenner - bruch2.zaehler * nenner;
		ergebnis.nenner = nenner * bruch2.nenner;
		return ergebnis.kuerzen();
	}
	// Ausgabe
	
	public String toString() {
		String Ausgabe = new String();
		Ausgabe = zaehler + "/" + nenner;
		return Ausgabe;
	}

	// Kürzen

	private Rational kuerzen() {
		double iGrößer, iKleiner, iErgebnis = 0;
		boolean bStart = true;

		Rational ergebnis = new Rational();

		iGrößer = (nenner < zaehler ? zaehler : nenner);
		if (iGrößer == nenner) {
			iKleiner = zaehler;
		} else {
			iKleiner = nenner;
		}
		while (iErgebnis != iKleiner) {
			if (!bStart) {
				if (iKleiner < iErgebnis) {
					iGrößer = iErgebnis;
				} else {
					iGrößer = iKleiner;
					iKleiner = iErgebnis;
				}
			} else {
				bStart = false;
			}
			iErgebnis = iGrößer - iKleiner;
		}

		ergebnis.zaehler = zaehler / iErgebnis;
		ergebnis.nenner = nenner / iErgebnis;

		return ergebnis;
	}
}

	public static void main(String[] args) throws Exception {
		
		Rational bruch1 = new Rational();
		Rational bruch2 = new Rational();
		bruch1.rational(14, 2);
		bruch2.rational(4, 2);
		
		System.out.println("Ergebnis der Subtraktion beträgt: "
				+ bruch1.subtraktion(bruch2));

	}

 

[Campino]

1. Warum verwendest du keinen richtigen Konstruktor? Du musst nur das "void" weglassen oder durch ein "public" ersetzen und Rational groß schreiben. das Problem, dass du dann in der substraktionsmethode bekommst, kannst du umgehen, indem du einen zusätzlichen, leeren Konstruktor einfügst oder den neuen Rational erst unten erstellst, wenn die Werte bekannt sind, Dann musst du natürlich zusätzliche Variablen zum zwischenspeichern verwenden.

2. Ich habe das mal ausprobiert. Anscheinend ergibt sich in der Funktion kuerzen eine Endlosschleife. Versuch mal im Kopf nachzuvollziehen, was genau hier bei einer Kommazahl passiert. Ich vermute, dass sich einfach keine Kommazahl für den Nenner bildet, bzw. dies endlos lange dauert.

 

[Leroy42]

Ich verstehe deine Frage nicht: Die Methode rational besitzt doch
2 double-Argumente, also kannst du ihr auch doubles übergeben.

 

[Leroy42]

Vermeide diese Schwierigkeiten doch einfach dadurch,
das du Zähler und Nenner als int deklarierst.

Meine eigene Rational-Klasse verwendet auch
nur ints (genauer: BigInteger)

Edit: Auch das Kürzen vereinfacht sich dann zu einer simplen
Implementation des Gauß'schen GGT-Verfahrens.