OOP Brüche nicht richtig berechnen

[morry329]

Hallo, ich habe den folgende Code, um arithmetische Operation durchzuführen. Die Berechnungen sind falsch außer addition:

9/7
1/5
addition:
52/35 //passt
subtraction:
8855/6125 //sollte 38/35 sein
multiply:
61985/1071875 //sollte 9/35 sein
divide:
1549625/42875 //sollte 45/7 sein

Ich habe eine Stunde gekämpft, aber ich bin nicht weitergekommen. Vielleicht kann jemand mir eine Hilfsstellung geben? Hier ist den Code

package F;

public class FractionTypVorlesung {
    int numer;
    int denom;

    public FractionTypVorlesung(int n, int d){
        numer = n;
        denom = d;
        //reduce();
    }

    public void reduce(int numer, int denom){
        int GCD = gcd(numer, denom);
        numer /= GCD;
        denom /= GCD;
    }

    public void add (FractionTypVorlesung other){
        numer *= other.denom;
        other.numer *= denom;
        denom *= other.denom;
        other.denom *= denom;
        numer+= other.numer;
        reduce(numer, denom);
        System.out.println(numer + "/" + denom);
    }

    public void sub(FractionTypVorlesung other) {

        int newNumer = this.numer * other.denom - other.numer * this.denom;
        int newDenom = this.denom * other.denom;
        reduce(newNumer, newDenom);
        System.out.println(newNumer + "/" + newDenom);
    }

    public void multiply(FractionTypVorlesung other) {
        int newNumer = this.numer * other.numer;
        int newDenom = this.denom * other.denom;
        reduce(newNumer, newDenom);
        System.out.println(newNumer + "/" + newDenom);
    }

    public void div(FractionTypVorlesung other) {
        int newNumer = this.numer * other.denom;
        int newDenom = this.denom * other.numer;
        reduce(newNumer, newDenom);
        System.out.println(newNumer + "/" + newDenom);
    }

    public int gcd(int a, int b){
         if(b==0){
             return a;
         }
         return gcd(b, a%b);
    }

    public String toString(){
        return Integer.toString(numer) + "/" + Integer.toString(denom);
    }

    public static void main(String[] args) {
        FractionTypVorlesung a = new FractionTypVorlesung(9,7);
        FractionTypVorlesung b = new FractionTypVorlesung(1, 5);

        System.out.println(a.toString());
        System.out.println(b.toString());
        System.out.println("addition: ");
         a.add(b);
        System.out.println("subtraction: ");
        a.sub(b);
        System.out.println("multiply: ");
        a.multiply(b);
        System.out.println("divide: ");
        a.div(b);

    }
}

 

[Robert Zenz]

    public void reduce(int numer, int denom){
        int GCD = gcd(numer, denom);
        numer /= GCD;
        denom /= GCD;
    }

Also, du rechnest Dinge, dann rechnest du das nochmal und weist das Ergebnis dann der Variable numer zu. Wo ist numer definiert? In welchem Scope ist numer definiert?

Dein nicht ganz so konkretes Problem ist dass du sehr viel mischt was deine Zustaende angeht. Manchmal sind es Parameter, manchmal Instanzvariablen. Du solltest dich versuchen auf eines zu einigen, entweder du verwendest den aktuellen Zustand der Klasse, oder zu uebergibt den Zustand welcher gebraucht wird.

 

[KonradN]

Das, was @Robert Zenz geschrieben hat noch einmal in ganz klaren Worten:

a) Bei reduce kommen zwei wichtige Aspekte zum tragen:
Zum einen werden die Instanzvariablen durch die Parameter versteckt. Du rechnest also nur auf den Parametern
Zum Anderen sind Parameter Call by Value, d.h. es wird bei einem Aufruf von reduce nur der Wert der Variablen übergeben. Die Variablen vom Aufruf selbst ändern sich also nicht, wenn die Parameter geändert werden!

b) Es ist ganz wichtig, dass Du Dir genau überlegst, was die einzelnen Methoden machen sollen:

• Sollen sie was berechnen? Wenn ja: Was genau? Was soll sich ändern?
• Sollen sie etwas ausgeben?

Jede Method macht einfach irgendwas. add / sub ändern die Instanzen wild. Also beide! Sowohl die, auf der die Methode aufgerufen wurde aber auch den Parameter. multiply und div ändern keine Instanz sondern berechnen nur etwas in lokalen Variablen um dann etwas auszugeben. Dazu kann ich nur die absoluten basics anmerken:

1. Überlege Dir genau, was jede Methode machen soll! Was genau erwartest Du? Das musst Du ordentlich in Worte fassen! Bei add könnte es z.B. sein, dass du bei a.add(b) haben willst, dass b zu a addiert wurde. Dann ist in a das Ergebnis und b ist unverändert. Das sollte dann aber auch ebenso bei den anderen Methoden sein. reduce hätte dann keine Parameter und würde die Instanz selbst kürzen.
(Hier der Hinweis: Es ist bei so APIs üblich oder sehr oft zu finden, dass keine Instanz verändert wird. a.add(b) würde also weder a noch b verändern sondern einfach nur einen neuen Bruch zurück geben.)

2. Schreibe es am Besten wirklich auf! Du kannst also z.B. als erstes einen JavaDoc Kommentar schreiben, ehe Du die Methode schreibst! Dann kann man Dir auch am ehesten helfen, weil wir im Code sehen, was Du überhaupt machen willst. Das ist so bei dem Code derzeit schlicht nicht möglich.

 

[berndoa]

Ich sehe, wie ja auch größtenteils shcon gesagt wurde, Probleme:
1. deine add Methode würde ich mal genauso aufbauen wie die sub() methode.
Aktuell ist sie unübersichtlich und es mahct mehr sinn , für den neuen numerator und denominator eigene variabeln zu definieren.
womit wir zu 2. kommen:
wie offensichtlich durch die definition mittels

 int newNumer =...

ist die Variable newNumer und die zum FractionTypVorlesung Objekt gehörende numer Variable NICHT dasselbe!

Du müsstest dann natürlich auch numer mit dem in newNumer berehcneten Wert auch überschrieben.
FGleiches für denominator.


Was jetzt eher optional ist aber du auch prüfen solltest:
Dass Alles auch bei negativen Werten klappt.
Sowohl bezüglich der +-*/ Operationen als auch bzgl der gcd methode.