java division < Algorithmen < Schule < Informatik < Vorhilfe
|
Hallo !!
Hier ein script, dass einfach nur das Ergebnis der Division zweier Zahlen ausgeben soll Oo:
1: | public class tester {
| 2: |
| 3: | public double neu1;
| 4: |
| 5: |
| 6: | public void gibErgebnis(int ersteZahl, int zweiteZahl) {
| 7: |
| 8: | neu1 = ersteZahl / zweiteZahl;
| 9: | System.out.println(neu1);
| 10: |
| 11: | }
| 12: | } |
der code is vom programm bluJ, also nicht wundern. Aber das Problem hängt einfach nur an Java, bzw an mir ;)
die Funktion gibErgebnis() gibt immer die Division der zwei Zahlen auf die nächst niedrigere (?) natürliche Zahl aufgerundet aus...
Warum bloß gibt es keine Kommazahlen aus, neu1 ist doch oben als double definiert???
Vielen Dank für Eure Hilfe !
|
|
|
|
Status: |
(Antwort) fertig | Datum: | 00:46 Sa 14.10.2006 | Autor: | piet.t |
Hallo Richard,
dass neu1 ein double ist stimmt schon, allerdings bist in eine typische Falle getappt, die jedem java-Anfänger (und auch so manchem Erfahrenen) immer wieder über den Weg läuft.
Analysieren wir also mal, was in Zeile 8 passiert:
Zuerst wird der Quotient ersteZahl/zweiteZahl berechnet - und zwar ohne Rücksicht auf das, was ansonsten noch in dieser Zeile passiert. Erst danach wird das Ergebnis in das Feld neu1 übertragen.
Den Typ von Zwischenergebnissen bestimmt java nur in Abhängigkeit von den Typen der Ausgangsfelder - und dabei gilt eben, dass bei der Verknüpfung von zwei int-Feldern (egal ob durch Addition, Subtraktion, Multiplikation oder Division) das Ergebnis auch immer ein int ist.
Die Lösung: vor der Division musst Du eines der Felder (oder auch beide) auf den Typ double abändern (casten), dann wird auch das Ergebnis ein double:
1: | public class tester {
| 2: |
| 3: | public double neu1;
| 4: |
| 5: |
| 6: | public void gibErgebnis(int ersteZahl, int zweiteZahl) {
| 7: |
| 8: | neu1 = (double)ersteZahl / (double)zweiteZahl;
| 9: | System.out.println(neu1);
| 10: |
| 11: | }
| 12: | } |
Noch ein paar kleine stilistische Anmerkung (können wenn nötig ignoriert werden): neu1 als public-Attribut der ganzen Klasse zu definieren ist nicht besonders objektorientiert. Besser wäre es, neu1 im Innern der Methode zu deklarieren.
Ausserdem sollte eine Methode, die "gibErgebnis" heisst vielleicht auch etwas zurückgeben - nämlich das Ergebnis. Alles in allem sähe meine Klasse dann so aus:
1: | public class tester {
| 2: |
| 3: | public double neu1;
| 4: |
| 5: |
| 6: | public double gibErgebnis(int ersteZahl, int zweiteZahl) {
| 7: |
| 8: | double neu1; //speichert das Ergebnis
| 9: |
| 10: | neu1 = (double)ersteZahl / (double)zweiteZahl; //Die int-Argumente auf double casten, um auch ein double Ergebnis zu bekommen
| 11: | System.out.println(neu1);
| 12: |
| 13: | return neu1; //das Ergebnis an den Aufrufer zurückgeben
| 14: |
| 15: | }
| 16: | } |
Gruß
piet
|
|
|
|
|
Status: |
(Mitteilung) Reaktion unnötig | Datum: | 20:57 Mo 16.10.2006 | Autor: | Bit2_Gosu |
Ahso ;) Vielen Dank Piet !!!
|
|
|
|