JUnit Test Klasse Rational

[Barney0815]

Hallo,

ich habe da eine Aufgabe, wo ich nicht weiter komme.
Ich muss dabei zwei Brüche addieren. Aber zuvor probiere ich gerade eine einfache Sache aus.

Also die Klasse Rational:

public final class Rational implements Comparable {

    private final long zaehler;
    private final long nenner;

    public Rational(long number) {
        zaehler = number;
        nenner = 1;
    }

    public Rational(long numerator, long denominator) throws ArithmeticException {

        long g = gcd(numerator, denominator);
         
        numerator /= g ;
        denominator /= g;
        this.zaehler = numerator;
        this.nenner = denominator;
    }
}

Und meine Test.java

public void testAdd() {
Rational r1 = new Rational(1, 2);

assertEquals(new Rational(1, 2), r1);

}

Leider liefert JUnit kein postives ergebnis. Kann mir einer Helfen??

 

[Barney0815]

Keiner einer Idee!!!

 

[kama]

Hallo,

stelle Dir mal die Frage, was Du in dem JUnit Test tatsächlich miteinander vergleichst?

MfG
Karl Heinz Marbaise

 

[Barney0815]

Keine Ahnung! Kannst du mir nicht helfen?

Wenn ich sowas habe funkoniert es auc nicht

public void testAdd() { 
Rational r1 = new Rational(1, 2); 
Rational r2 = new Rational(1, 2); 

assertEquals(r1, r2); 

}

oder

public void testAdd() { 

assertEquals( new Rational(1, 2), new Rational(1, 2)); 

}

 

[kama]

Hi,

public void testAdd() { 
Rational r1 = new Rational(1, 2); 
Rational r2 = new Rational(1, 2); 

assertEquals(r1, r2); 

}

Also dass hier das assertEquals false liefert ist völlig in Ordnung. Du prüftst, ob die Objekte (r1, r2) identisch/gleich sind. Was sie nicht sind!

Es sind zwei unterschiedliche Objekte!

Was möchtest Du mit einem solchen Test erreichen?

MfG
Karl Heinz Marbaise

 

[Guest]

Nein das Teste ich auch nicht. Ich will ja das obere Testen.

public void testAdd() { 
Rational r1 = new Rational(1, 2); 

assertEquals(new Rational(1, 2), r1); 

}

Leider weiß ich nicht, was ich machen soll

 

[SnooP]

Och Leute... sagt ihm das doch mal ... - also du möchtest testen ob zwei Objekte inhaltlich gleich sind, sprich ob Zähler/Nenner identisch sind... - dazu musst du aber Java erstmal beibringen, dass er diese Attribute der Objekte miteinander vergleichen soll. Daher musst du schlicht die equals Methode in Rational vergleichen... diese muss dann true liefern, wenn Zähler/Nenner des übergebenen Objekts verglichen mit this gleich sind.

Wenn du das nicht willst, kannst du nur die konkreten Zahlenwerte des Objekts mit assertEquals vergleichen, dafür bräuchtest du aber konkrete get-Methoden für zaehler und nenner.

 

[Gast]

Also muss ich eine equals() Methode schreiben???

 

[Barney0815]

Ich habe nun eine equals()-Methode geschrieben. Kann ich diese so verwenden? oder sind da Fehler drin

public boolean equals( Object other ) {
        if (! (other instanceof Rational)) return false;
        Rational r = (Rational) other;

        if( other == null ) return false;
        
        if( this.nenner == r.nenner && this.zaehler == r.zaehler ) return true;
        else return false;
    }

 

[Der Müde Joe]

so ca ja

zähler und nenner sind nicht static
also sollte r.nenner nicht gehen
zudem sind sie noch private also
haste eh keinen zugriff drauf

--> getter machen

 

[Leroy42]

so ca ja

zähler und nenner sind nicht static
also sollte r.nenner nicht gehen
zudem sind sie noch private also
haste eh keinen zugriff drauf

--> getter machen

:autsch: :autsch: :autsch:

Ist doch vollkommen richtig, was Barney geschrieben hat.

Ich würde es allerdings etwas verkürzen:

public boolean equals(Object other) { 
  if (other instanceof Rational) {
    Rational r = (Rational) other;
    return r != null && nenner == r.nenner && zaehler == r.zaehler;
  } else
    return false,
}

 

[Der Müde Joe]

ach ja...sry...
muss aufhören dokus zu schreiben.
da wird mir ganz ander im kopf.. ;-)

 

[Guest]

OK Danke ihr habt mir weiter geholfen