Rechnen mit Brüchen, ist meine Lösung soweit richtig?

Cookiieee

Mitglied
Die Rechnung mit endlichen (Gleit-)Kommazahlen kann zu Ungenauigkeiten führen, es kann daher wünschenswert sein, ganzzahlige Brüche zu verwenden.
In dieser Aufgabe soll eine Objektklasse zur Repräsentation von und Rechnung mit Brüchen erzeugt werden.
Die Köpfe der geforderten Methoden sind bereits vorgegeben und dürfen nicht verändert werden. Die Methoden sollen entsprechend der Kommentare vervollständigt werden.
Achten sie darauf diese Methoden, wann immer möglich, zu verwenden und so Duplikationen zu vermeiden.
Alle Methoden deren Rückgabetyp Fraction ist, sollen ein neues Objekt erzeugen und zurückgeben und die zur Berechnung verwendeten Objekte unverändert lassen!

Die Java-Standardbibliotheken Math und Integer dürfen (müssen aber nicht) verwendet werden. Um eine Gleitkommazahl bei der Rechnung mit Ganzzahlen zu erhalten muss ggf. Typecasting verwendet werden!
Es stehen außerdem die folgenden zwei Methoden schon zur Verfügung:
public static int GCD(int x, int y): Gibt den größten gemeinsamen Teiler der beiden Argumente (Greatest Common Divider) zurück.

public static int LCM(int x, int y): Gibt das kleinste gemeinsame Vielfache der beiden Argumente (Lowest Common Multiple) zurück.


Java:
public class Fraction {

    private int numerator, denominator;
    
    /**
     * Gibt den größten gemeinsamen Teiler der beiden Argumente (Greatest Common Divider) zurück.
     */
    public static int GCD(int x, int y) {
        if (y == 0) return x;
        return GCD(y, x % y);
    }

    /**
     * Gibt das kleinste gemeinsame Vielfache der beiden Argumente (Lowest Common Multiple) zurück.
     */
    public static int LCM(int x, int y) {
        return (x * y) / GCD(x, y);
    }
 //Beginn der Aufgabe 
   /**
     * Vollstaendig parametrisierter Konstruktor der Zaehler und Nenner
     * uebergeben bekommt und die entsprechenden Attribute setzt.
     * Negative Vorzeichen (Zahlen kleiner als Null) duerfen nur im Zaehler
     * auftreten (nicht im "denominator"-Attribut).
     * Die Uebergabe eines negativen Nenners ("denominator"-Argument) an den Konstruktor ist jedoch zulaessig.
     * Der Konstruktor muss also den uebergebenen Nenner pruefen und sein Vorzeichen so behandeln,
     * dass der resultierende Bruch (die Attribute) die genannte Restriktion erfüllt
     * und der Wert des Bruchs (die Argumente) unverändert bleibt
     * (ein negatives Vorzeichen im Nenner muss also methematisch korrekt beseitigt werden).
     * Wird eine Null als Nenner uebergeben, so wird das entsprechende Attribut
     * auf Eins gesetzt.
     * Jeder erzeugte Bruch wird gekuerzt (dazu soll die entsprechende Mehode s.u. verwendet werden).
     */
    public Fraction(int numerator, int denominator) {
        if(pDenominator<0){
            numerator = -pNumerator;
            denominator = -pDenominator;
        }
        if(pDenominator == 0){
            denominator = 1;
        }
        
    }

    /**
     * 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 = numerator / denominator;
        return fraction;
    }

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

    /**
     * Kuerzt (vereinfacht) den Bruch.
     */
    public void shorten() {
        int factorC = GCD(numerator, denominator);
        numerator = numerator/factorC;
        denominator = denominator/factorC;
        
    }

    /**
     * Erweitert (macht gleichnamig), addiert dann den uebergebenen Bruch.
     */
    public Fraction add(Fraction f) {
        numerator = numerator * f.denominator + f.numerator * denominator;
        denominator = denominator * f.denominator;
        cancel();
      
    }

    /**
     * Multipliziert mit dem uebergebenen Bruch.
     */
    public Fraction multiply(Fraction f) {
        numerator = numerator * f.numerator;
        denominator = denominator * f.denominator;
        cancel();
    }

    /**
     * Bildet den Kehrwert, wenn der Zaehler ungleich Null ist.
     * Sonst wird der Bruch unveraendert zurueckgegeben.
     */
    public Fraction reciprocal() {
      
    }
    
    /**
     * Dividiert durch den uebergebenen Bruch
     * (unter Verwendung von Kehrwert und Multiplikation).
     */
    public Fraction divide(Fraction f) {
        numerator = numerator * f.denominator;
        denominator = denominator * f.numerator;
        cancel();
    }
}
 
Neben der Tatsache, dass es nicht kompiliert - was passiert im Konstruktor, wenn ein Nenner > 0 übergeben wird?
Kommentar im Konstruktor:
* Jeder erzeugte Bruch wird gekuerzt (dazu soll die entsprechende Mehode s.u. verwendet werden).
Wo passiert das?

reciprocal - kein Code, kompiliert nicht.
divide - zwar mathematisch richtig, aber nicht gemäß Aufgabe, es sollen die Methoden für den Kehrwert und Multiplikation genutzt werden.

Alle Methoden verlezten folgende Aufgabestellung:

Alle Methoden deren Rückgabetyp Fraction ist, sollen ein neues Objekt erzeugen und zurückgeben und die zur Berechnung verwendeten Objekte unverändert lassen!

toDouble wird auch nicht funktionieren, da es eine Integer Division.

Als konstruktiver Vorschlag: Ersetz alle Methoden, die was zurückgeben mit folgendem Inhalt:
return null;

Dann fang an eine Methode nach der anderen zu implementieren und bau dir ein kleines Programm, was die Methoden testet. Und wenn eine Methode funktioniert (am besten mit dem Konstruktor anfangen), dann die nächste.
 
Java:
public class Fraction {

    private int numerator, denominator;

    public static int GCD(int x, int y) {
        if (y == 0) return x;
        return GCD(y, x % y);
    }

    public static int LCM(int x, int y) {
        return (x * y) / GCD(x, y);
    }

    public Fraction(int numerator, int denominator) {
        if (denominator == 0)
            throw new IllegalArgumentException("Division by zero");
        int factor = GCD(Math.abs(numerator), Math.abs(denominator));
        this.numerator = numerator / factor;
        this.denominator = denominator / factor;
        if (this.denominator < 0) {
            this.numerator = -this.numerator;
            this.denominator = -this.denominator;
        }
    }

    public int getDenominator() {
        return denominator;
    }

    public int getNumerator() {
        return numerator;
    }

    public double toDouble() {
        return (double) numerator / denominator;
    }

    public String toString() {
        return numerator + "/" + denominator;
    }

    public void shorten() {
        int factor = GCD(Math.abs(numerator), Math.abs(denominator));
        this.numerator = numerator / factor;
        this.denominator = denominator / factor;
        if (this.denominator < 0) {
            this.numerator = -this.numerator;
            this.denominator = -this.denominator;
        }
    }

    public Fraction add(Fraction f) {
        int commonDenominator = LCM(this.denominator, f.denominator);
        return new Fraction(this.numerator * (commonDenominator / this.denominator)
                + f.numerator * (commonDenominator / f.denominator), commonDenominator);
    }

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

    public Fraction reciprocal() {
        return numerator == 0 ? this : new Fraction(denominator, numerator);
    }

    public Fraction divide(Fraction f) {
        return this.multiply(f.reciprocal());
    }

    public static void main(String[] args) {
        // Getter Test
        Fraction twothird = new Fraction(2, 3);
        System.out.println("Fraction twothird: " + twothird.getNumerator() + "/" + twothird.getDenominator());
        // Erwartet: 2/3

        try {
            Fraction f = new Fraction(2, 0);
            System.out.println("Fraction f: " + f.getNumerator() + "/" + f.getDenominator());
        } catch (IllegalArgumentException e) {
            System.out.println("Exception caught: " + e.getMessage());
            // Erwartet: Exception mit "Division by zero"
        }
    }
}


Ich habe jetzt das, allerdings kommt, dass da noch etwas falsch ist, kann mir da jemand weiter helfen?
Siehe Bilder

Oder muss ich bei Zeile 77
Fraction f = new Fraction(2, 0);

Statt (2,0) einfach (2,1) hinschreiben?
 

Anhänge

  • Screenshot (111).png
    Screenshot (111).png
    12 KB · Aufrufe: 0
  • Screenshot (110).png
    Screenshot (110).png
    16,4 KB · Aufrufe: 0
Zuletzt bearbeitet:
In Zeile 77 übergibst Du einen Nenner = 0. Genau dieser Fall wird in Zeile 15 abgeprüft und in Zeile 16 wird eine Exception geworfen, denn Brüche mit einer 0 im Nenner (entspricht Division durch 0) sind mathematisch nicht definiert.
 

Zurück
Oben