Methoden JUnit 4 - Test

[PoroBobRoss]

Hallo,
ich muss ein Programm schreiben, dass mir mit der Übergabe eines Calendar-Objekt an die Methode
[CODE lang="java" title="berechneBruttoPreis(Calendar cal)"] public double berechneBruttoPreis (Calendar cal) {
//Compare Calendar
//Jul
Calendar jul = new GregorianCalendar(2020, 6, 1);
//Dez
Calendar dez = new GregorianCalendar(2021, 0, 1);

//Nicht Ermaessigt
if (ermaessigterSteuersatz == false) {
// innerhalb Zeitraum
if (cal.after(jul) && cal.before(dez)) {
return Math.round((nettoPreis * 1.17) * 100) / 100.0;
}
// außerhalb Zeitraum
else {
return Math.round((nettoPreis * 1.19) * 100) / 100.0;
}

}

//Ermaessigt
else {
// innerhalb Zeitraum
if (cal.after(jul) && cal.before(dez)) {
return Math.round((nettoPreis * 1.05) * 100) / 100.0;
}
// außerhalb Zeitraum
else {
return Math.round((nettoPreis * 1.07) * 100) / 100.0;
}

}

}[/CODE]
übergibt und so den Preis eines Produktes mit den jeweiligen MwSt zu berechnen (je nach Datum eben 19/16% bzw 7/5%).
Beim Testen dieser Methode tritt aber nun folgendes Problem auf:
[CODE lang="java" title="JUnit4 Test Klasse"]import static org.junit.Assert.*;

import java.util.Calendar;
import java.util.GregorianCalendar;

import org.junit.Before;
import org.junit.Test;

public class ProduktTest {
private Produkt prod1;

private Calendar march;

@Before
public void setUp() {
//arrange
//Produkte
prod1 = new Produkt("Katzenbrunnen", false, 25.20);
//Calendar
march = new GregorianCalendar(2020, 2, 18, 13, 0);
}

@Test
public void keineReduzierteSteuer() {
//act
double erg1 = prod1.berechneBruttoPreis(march);
System.out.println(erg1);
//assert
assertEquals(29.99, erg1);


}

}[/CODE]
Der Test schlägt fehl. Das sysout hab ich mal zur Überprüfung hingeschrieben, damit ich mir auch selbst sicher bin, dass das das richtige Ergebnis ist.
Ich weiß nicht ganz genau warum der Test fehlschlägt. Es könnte sich ja mal einer anschauen (vlt Stimmt ja auch was an der Preisberechnung nicht so ganz)
Vielen Dank im voraus, egal ob wir zu einer Lösung kommen oder nicht.

Edit:
Mit "NotSame" funktioniert der Test zwar, aber das irritiert nicht etwas, da dort ja mit == verglichen wird.
Die Ergebnisse werden auch auf 2 Nachkomma gerundet. Hab auch schonmal versucht das ganze zu debuggen, jedoch finde ich dort auch nicht den Fehler.

Grüße Julian

 

[PinkMuffin]

Was kommt denn als Meldung. Normal steht da "excpeted... but was ..."
Hat es einen Grund, dass du bei deinem Testdatum 5 Parameter mitgibst?
Eigentlich sollte Jahr, Monat und Tag ausreichen

 

[PoroBobRoss]

Nein, es gibt da keinen Grund für. Hatte nur gedacht das es nicht so ganz "dumm" aussähe wenn ich noch die Uhrzeit mitnehme.
Also wenn du den FailureTrace meinst dann:
java.lang.AssertionError: Use assertEquals(expected, actual, delta) to compare floating-point numbers
Da hab ich wohl vergessen draufzuschauen.
Delta ist ja normal die Differenz? Für was wird die denn hier benötigt? (Ich hoffe ich liege nicht falsch ^^)

Ja hab es grad in der API gesehen. Ahje, manchmal hat man das Brett vor dem Kopf

 

[PinkMuffin]

Delta hab ich zumindest gebraucht, weil die normale Methode deprecated ist. Da es genau übereinstimmen soll, kannst du an der Stelle 0 einsetzen. Wenn ich die Fehlermeldung richtig verstehe, ist das auch das Problem.

 

[LimDul]

Das Problem ist, das Floating Point Operationen in der Regel nicht exakt sind. Wenn man - als Beispiel - 100 mal 0.01 aufaddiert, hat man nicht 1 sondern vermutlich 0.99999999999999999999999999999999999999999999 oder 1.0000000000000000000000000000000001 in Art. Dementsprechend sollte man bei Vergleichen auch immer ein Delta mit angeben, inwieweit die gleich sind. Gut hier rundest du, daher geht ein Delta von 0 gut, aber da läuft man halt leicht in Probleme rein.

 

[kneitzel]

Nur als kleine Anmerkung und zur besseren Einordnung:
So viele 0er bzw. 9er braucht man nicht einmal. Das sind ja über 30 Stück. Im Dezimalsystem ... Mit wie viel bit willst Du die Mantisse denn speichern?

Aber das kann man ja schnell austesten:

        double summe = 0.0;
        double inc = 0.01;
        for (int i = 0; i<100; i++) summe += inc;
        System.out.println(summe);

1.0000000000000007

Ist immer noch eine gute Genauigkeit. Aber wichtig ist dann halt, dass man sich Gedanken macht über die benötigte Genauigkeit. Und dann braucht man einen Datentyp mit höherer Genauigkeit.
Und dann muss man bei Berechnungen ggf regelmäßig "runden", so dass Abweichungen sich nicht aufaddieren!

 

[httpdigest]

if (ermaessigterSteuersatz == false) {
// innerhalb Zeitraum
if (cal.after(jul) && cal.before(dez)) {
return Math.round((nettoPreis * 1.17) * 100) / 100.0;
}

Kleiner Hinweis noch: Der durch Corona gesunkene Umsatzsteuersatz betrug 16%, nicht 17%.
Und: Wenn du jegliche Rundungsfehler vermeiden willst, rechne einfach immer mit Ganzzahlen in der kleinsten Einheit, die in deinem Anwendungsbereich Sinn macht, also zum Beispiel 1 = 1 Eurocent. €19,95 wäre dann also einfach der Integer 1995.