Normale Rechnung - komisches Ergebnis.

[Mujahiddin]

Folgender Code ist gegeben:

System.out.println(geld + " + (" + (double)verkaufteBecher + " * (" + (double)verkaufspreis/100 + " - " + Daten.SAFTBECHER_PREIS + "))");
geld = geld + ((double)verkaufteBecher*1.0 * ((double)(verkaufspreis/100) - Daten.SAFTBECHER_PREIS));
System.out.println(geld);

Wichtig ist natürlich die mittlere Zeile, in der der Gewinn vom heutigen Tag zusammengezählt und dem eigenen Geld zugefügt wird. Nur leider kommt ein total unsinniges Ergebnis raus.
Wenn ich das in meinen Taschenrechner eingebe, kommt 117.6 raus.
Ich dachte zuerst, es könnte an Double-Integer Mischmasch liegen, hab alles in doubles umgewandelt und trotzdem das gleiche Problem.
Die Konsole spuckt folgendes aus:

5.6 + (140.0 * (1.4 - 0.6000000000000001))
61.59999999999999

Ich bin am Ende mit meinem Latein!

Liebe Grüße,
Mujahiddin!

 

[XHelp]

Bei verkaufspreis/100 kommt nicht 1.4, sondern ein int wert (da du durch 100 und nicht durch 100.0 teilst), also 1
Dementsprechend lautet die Formel:
5.6 + (140.0 * (1.0 - 0.6000000000000001))

Teile also durch 100.0 und es müsste passen.

 

[Mujahiddin]

Okay, es klappt, danke dir!
Wieso kommt aber System.out.println() 1.4 und beim anderen 1.0?
Ergibt doch gar keinen Sinn....

 

[Gast2]

(double)verkaufspreis/100

Hier castet er zuerst den Verkaufspreis auf double => es kommt 1.4 raus.

(double)(verkaufspreis/100)

Da du hier geklammert hast rechnet java zuerst verkaufspreis / 100 => da kommt 1 raus, danach der cast auf double ergibt dann 1.0

 

[XHelp]

Ich nur eine Vermutung, aber:
in Sysout machst du

(double)verkaufspreis/100

und in der Formel

((double)(verkaufspreis/100))

Durch die Klammer bestimmst du ja was gecastet wird.

 

[Mujahiddin]

Ja, okay, stimmt.
Dann meine letzte Frage:
Wie ihr seht, ist bei Daten.SAFTBECHER_PREIS 0.6000000001. das 0000001 hinten ergibt 0 sinn, da sich der preis folgendermaßen zusammensetzt:

public static final double KONZENTRAT_PREIS = 11.00;
public static final double SUESSSTOFF_PREIS = 1.50;
public static final double BECHER_PREIS = 0.02;
public static final double KONZENTRAT_PRO_BECHER = 0.05;
public static final double SUESSSTOFF_PRO_BECHER = 0.02;
public static final double SAFTBECHER_PREIS = KONZENTRAT_PREIS * KONZENTRAT_PRO_BECHER + SUESSSTOFF_PREIS * SUESSSTOFF_PRO_BECHER + BECHER_PREIS;

also sprich 0.55 + 0.03 + 0.02.
es müsste zwar 0.60 rauskommen, wieso kommt aber 0.6000001 raus? Das ist bei doubles häufiger so. Wie kann man das vehrindern? (wenn mans nicht verhindern kann -> geht es irgendwie, dass ich es auf 2 dezimalstellen runde? also 0.60?)

 

[gman]

wieso kommt aber 0.6000001 raus? Das ist bei doubles häufiger so

Bei solchen Rechensachen empfiehlt es sich "BigDecimal" statt "double" zu verwenden. Da kannst du
dann auch bequem runden "setScale" und auch angeben wie gerundet werden soll "RoundingMode".

 

[Mujahiddin]

Okay, hört sich schon mal nicht schlecht an... Aber ich kann keine Mal-Zeichen dazwischen setzen, oder mach ich was falsch?

public static final BigDecimal KONZENTRAT_PREIS = new BigDecimal(11.00);
public static final BigDecimal SUESSSTOFF_PREIS = new BigDecimal(1.50);
public static final BigDecimal BECHER_PREIS = new BigDecimal(0.02);
public static final BigDecimal KONZENTRAT_PRO_BECHER = new BigDecimal(0.05);
public static final BigDecimal SUESSSTOFF_PRO_BECHER = new BigDecimal(0.02);
public static final BigDecimal SAFTBECHER_PREIS = KONZENTRAT_PREIS * KONZENTRAT_PRO_BECHER + SUESSSTOFF_PREIS * SUESSSTOFF_PRO_BECHER + BECHER_PREIS;

- Der Operator * ist für den/die Argumententyp(en) java.math.BigDecimal, java.math.BigDecimal nicht definiert.

 

[XHelp]

BigDecimal ist eine Klasse, wie du damit umgehst, steht in der Api:
BigDecimal (Java Platform SE 6)
z.B. gibt es die Methode

multiply

 

[Mujahiddin]

okay, und warum muss man alles, was mit double zu tun hat, in BigDecimal umrechnen, nur weil paar Java Programmierer so schlau waren, in irgendwelche normale Rechnungen völlig sinnlose Abweichungen reinzubauen?
Tut mir Leid, ich weiß, bisschen überreagiert, aber das nervt irgendwie - ich muss jetzt also das ganze Programm umschreiben, nur damit ich hinten kein 00001 stehen hab?

 

[hemeroc]

Der Grund warum das passiert liegt in der Natur der Binärdarstellung, genauer nachzulesen hier Gleitkommazahl ? Wikipedia.

Und natürlich lässt sich das Problem umgehen, beispielsweise mittels runden:

double precision = Math.pow(10, anzahlDerGewünschtenNachkommastellen);
deineGleitkommaZahl = Math.round(deineGleitkommaZahl * precision) / precision;

LG Hemeroc