Brüche kürzen

[thefastandcurious]

Ist es so richtig?

 

[mihe7]

Nein, das ist kein gültiger Code. Einfach so:

    public Fraction(int numerator, int denominator) {
        this.numerator = numerator;
        this.denominator = denominator;
       
        if(denominator<0){
            this.numerator = -numerator;
            this.denominator = -denominator;
        }
        if(denominator == 0){
            this.denominator = 1;
        }
        shorten();
    }

 

[thefastandcurious]

Muss ich noch in den multiply und reciprocal Methoden was ändern?

 

[mihe7]

Die beiden sehen in Ordnung aus, add() und divide() sind noch falsch, wobei Du divide() jetzt unter Verwendung von multiply() und reciprocal() schreiben kannst.

 

[thefastandcurious]

Ist es so richtig?

    public Fraction divide(Fraction f) {
        return multiply(this.reciprocal());
    }
    
    public Fraction add(Fraction f) {
        return new Fraction(numerator * f.denominator + denominator * f.numerator, denominator * f.denominator);
    }

 

[mihe7]

add ja, divide nicht. Wofür bekommst du denn f übergeben, wenn Du es nicht verwendest?

Außerdem: warum probierst Du es nicht einfach aus?

 

[thefastandcurious]

public Fraction divide(Fraction f) {
return new Fraction(numerator * f.denominator, denominator * f.numerator);
}

Bei mir kam kein Fehler raus.

 

[mihe7]

return multiply(f.reciprocal());

 

[thefastandcurious]

Das hatte ich gelöscht und stattdessen geschrieben.

 

[thefastandcurious]

Soll ich stattdessen
return multiply(f.reciprocal());
schreiben?

 

[thefastandcurious]

Wegen dem Kommentar darüber?

 

[temi]

Bei mir kam kein Fehler raus.

Aber kam auch das korrekte Ergebnis raus?

 

[thefastandcurious]

Was soll denn dort in der Ausgabe stehen?

 

[KonradN]

Was soll denn dort in der Ausgabe stehen?

Du kannst doch Code schreiben, der es testet. Du gibst also immer etwas vor, berechnest es mit Deinem Code und schaust dann, ob das Ergebnis richtig ist.

Beispiel: Du erstellst den Bruch 2/-4 und gibst ihn aus: was wäre das erwartete Ergebnis? Was muss ausgegeben werden?

(Der erste wichtige Punkt ist immer, dass die Anforderungen verstanden wurden. Hast du das? Kannst du sagen, was da im Bruch gespeichert sein muss, was danach ausgegeben werden muss?)

 

[thefastandcurious]

Sind meine Methoden über shorten() in Ordnung:

    /**
     * Gibt den Nenner zurueck.
     */
    public int getDenominator() {
        return denominator;
    }

    /**
     * Gibt den Zaehler zurueck.
     */
    public int getNumerator() {
        return numerator;
    }

    /**
     * Gibt den Bruch als Gleitkommazahl zurueck.
     */
    public double toDouble() {
        double fraction = (double)numerator / denominator;
        return fraction;
    }

    /**
     * Gibt einen String im Format
     * "Zaehler/Nenner" zurueck.
     */
    public String toString() {
        String fraction = numerator + "/" + denominator;
        return fraction;
    }

Ich bin mir nicht sicher, weil ich glaube, dass ich bei toDouble() Typcasting verwenden muss, um eine Gleitkommazahl bei der Rechnung mit Ganzzahlen zu erhalten. Kann jemand darüberschauen? Ich habe es mit 2/-4 kontrolliert, aber dann kam diese Fehlermeldung:

Main.java:1: error: class Fraction is public, should be declared in a file named Fraction.java
public class Fraction {
^
1 error

Ich hatte eine Datei namens Fraction.java schon erstellt, aber dieser Fehler kommt trotzdem auf. Deshlab weiß ich nicht, ob es jetzt mit den Brüchen klappen wird oder nicht.

 

[temi]

Du kannst doch Code schreiben, der es testet.

Aber aufpassen, dass du in deinem Test auch unterschiedliche Fälle und ggf. Grenzfälle testest, sonst kannst du leicht zu einem falschem positivem Ergebnis kommen:

public class Adder {

    // addiert zwei Werte
    public int add(int a, int b) {
        return a + 2;
    }

    public static void main(String[] args) {
        Adder adder = new Adder();
        // Test der Addition
        System.out.println(adder.add(2,2)); // ergibt 4, scheint zu passen, oder?
        System.out.println(adder.add(2,3)); // ergibt 4, oh - doch nicht...
    }
}

 

[KonradN]

Ich hatte eine Datei namens Fraction.java schon erstellt, aber dieser Fehler kommt trotzdem auf.

Diese Klasse muss auch in genau dieser Datei sein. Und wenn Du den Inhalt nur kopierst, dann muss die alte Datei gelöscht werden.

Generell angewöhnen: Packe Klassen immer in eine Datei, die genau so heißt, wie die Klasse (mit .java als Endung).

 

[thefastandcurious]

Okay, danke. Entsprechen meine Methoden über der shorten()-Methode den Kommentaren?

 

[thefastandcurious]

Muss ich bei toDouble() typecasten?

 

[KonradN]

Muss ich bei toDouble() typecasten?

Ja, und das machst du in dem Code doch auch. Du kannst es ja mal ohne den cast probieren und schauen, was dann anders ist.

 

[thefastandcurious]

Vielen, vielen Dank für all Eure Hilfe. Es hat mir sehr viel bedeutet.