Datentypen Fehler bei Berechnung mit Double und Int-Werten

[Jo.Mo.]

Hallo,

ich sitz grad an einem Programm zur Berechnung von Abschreibungswerten, jetzt wollte ich als Anfangswert double nehmen, um nicht nur "glatte" Zahlen zur berechnung zuzulassen, ich habe auch deswegen extra als Sicherung hasNext verwendet (was ich noch nicht kannte, hoffe/denke es is so richtig), leider kommt dann beim comp. mit JCreator "possible loss in precision" wegen der Multiplikation in der Berechnung...

...merkwürdigerweise, kann ich den Anfangswert nicht auch als Int deklarieren, dann heisst es plötzlich "Variable might not have been initialized" in Bezg. auf alles ausser den Restwert. Es ist mir zwar klar, dass die variablen nicht global gültig sind, aber dass er plötzlich vergisst dass sie initialisiert wurden verstehe ich dabei auch nicht....

jemand eine Lösung dafür...?

import java.io.*;
import java.util.Scanner;

public class Restwert {

    public static void main(String[] args) {

	    // Variablendeklarationen
		int Anfangswert;
 	    int Nutzungsdauer;
	    int Jahre;
	    int Restwert;

		// Werte eingeben
	    System.out.println("Geben sie die Anschaffungskosten ein! "); // Anfangswert eingeben
        Scanner in = new Scanner( System.in);
        if (in.hasNextInt()) Anfangswert = in.nextInt();
		System.out.println("Geben sie die Nutzungsdauer in Jahren an!"); // Nutzungsdauer eingeben
	    if (in.hasNextInt()) Nutzungsdauer = in.nextInt();
	    System.out.println("Geben sie das Jahr an!"); // Jahr eingeben
		if (in.hasNextInt()) Jahre = in.nextInt();

		// (3) Anweisungen
	    Restwert = (1 - (((2 * Nutzungsdauer + 1 - Jahre) * Jahre) / (Nutzungsdauer * Nutzungsdauer + Nutzungsdauer))) * Anfangswert;
	    Restwert = Math.round(Restwert);

	    // (4) Ausgabe
        System.out.println ("Der Restwert beträgt nach "+ Jahre +" Jahren rund "+ Restwert +" Euro.");

    }

}

 

[nrg]

du musst explizit casten

 

[diggaa1984]

int / int bleibt int ... 1/2 = 0 wenn ich nicht irre

du musst daher, wie mein vorredner bereits sagte .. bei der division explizit mit zB. double-typen arbeiten, damit auch das ergebnis double wird.

 

[Zeeu]

als erstes solltest du wie variablen initialisieren.

und warum rundest du einen integer wert ? was soll denn da raus kommen ?

ich habe jetzt mal den anfangswert und den restwert als double deklariert, da kommt denke ich nen sinniger wert raus.

 

[Jo.Mo.]

@Zeeu: Das hatte ich in der Zwischenzeit auch gemacht, war mein Fehler, klar dass der Restwert ebenfalls double sein muss (wie eigentlich auch der anfangswert sein sollte), nur beschwert er sich dann über die variablen...?

import java.io.*;
import java.util.Scanner;

public class Restwert {

    public static void main(String[] args) {

	    // Variablendeklarationen
		double Anfangswert;
 	    int Nutzungsdauer;
	    int Jahre;
	    double Restwert;

		// Werte eingeben
	    System.out.println("Geben sie die Anschaffungskosten ein! "); // Anfangswert eingeben
        Scanner in = new Scanner( System.in);
        if (in.hasNextDouble()) Anfangswert = in.nextDouble();
		System.out.println("Geben sie die Nutzungsdauer in Jahren an!"); // Nutzungsdauer eingeben
	    if (in.hasNextInt()) Nutzungsdauer = in.nextInt();
	    System.out.println("Geben sie das Jahr an!"); // Jahr eingeben
		if (in.hasNextInt()) Jahre = in.nextInt();

		// Anweisungen
	    Restwert = (1 - (((2 * Nutzungsdauer + 1 - Jahre) * Jahre) / (Nutzungsdauer * Nutzungsdauer + Nutzungsdauer))) * Anfangswert;
	    Restwert = Math.round(Restwert);

	    // Ausgabe
        System.out.println ("Der Restwert beträgt nach "+ Jahre +" Jahren rund "+ Restwert +" Euro.");

    }

}

 

[Zeeu]

bei mir beschwert er sich nicht, und kleine probleme wie casts etc kann der Compiler dir anzeigen und mit nem doppelklick selbstständig lösen.

Edit: deine Rechnung sieht sehr komisch aus, sicher dass die stimmt ?

 

[Jo.Mo.]

...die sollte stimmen, da ich das Programm vor einiger Zeit mit JavaScript geschrieben hatte, da ging es (sort war nur wichtig dass man die Eingabe parsen musste damit er richtig rechnet...)...

...womit compilierst du denn...? Ich arbeite erst seit kurzem (wieder) mit Java, und muss mich auch erstmal wieder mit JCreator (oder was anderem, sofern es da was besseres gibt) auseinander setzen...

 

[Gast2]

(((2 * Nutzungsdauer + 1 - Jahre) * Jahre) / (Nutzungsdauer * Nutzungsdauer + Nutzungsdauer)))

In der Rechnung verwendest du ausschließlich ints, d.h. das ergebnis wird auch immer ein int sein. Falls du da einen Wert von bspw. 0,5 erwartest kommt da 0 raus.

 

[Zeeu]

mache in java alles mit Eclipse

Edit:
wenn das ergebnis in double gespeichert wird, macht java da nicht einen automatischen Typecast ?

 

[Gast2]

double a = 1 / 2

=>

a = 0.0

 

[hierUndDa]

Wie bereits gesagt, musst du explizit casten, sonst bekommst du immer falsche Werte.

[JAVA=24]Restwert = (1 - (((2 * Nutzungsdauer + 1 - Jahre) * (double) Jahre) / (Nutzungsdauer * Nutzungsdauer + Nutzungsdauer))) * Anfangswert;[/code]

Die if-Bedingungen kannst du, so wie du es jetzt gemacht hast gleich weglassen, dann beschwert sich der Compiler auch nicht mehr. Das Problem ist, dass der Compiler nicht feststellen kann, ob die Bedingung immer erfüllt wird, deshalb beschwert er sich, dass die variable eventuell nicht intialisiert wird.

[JAVA=19]
if (in.hasNextInt())
[/code]

Ein bisschen exception-handling wäre natürlich auch noch schön, dann könntest du falsche eingaben des users abfangen. Die Nutzungsdauer in deinem Beispiel darf z.B. nie 0 sein, weil du ansonsten eine division durch 0 hast.

Edit: Viel zu spät... sorry!

 

[Jo.Mo.]

Klasse, jetzt läufts zumindest.

...das exception handling wollte ich ja mittels des if lösen...gibts da noch was einfacheres...?

Achso...wenn ich die Berechnung jetzt in einer eigenen Klasse vornehmen will (also quasi "objektorientiert"), kann ich dann einfach die berechnung in eine eigene Klasse auslagern (also zB class Restwert) und auf die dann mittels Restwert a = new Restwert (double, int, int, double) und dann a.getRestwert() zurückgreifen oder wie müsste das aussehen...?

Ich habe bisher wirklich nur step-by-step programmiert, das auslagern der einzelnen Schritte in Klassen ist mir noch etwas fremd...

 

[hierUndDa]

Zur ersten Frage:

Ich würde wohl den Input des Users mittels

String input = in.nextLine();

abfangen. Und danach mit einem try and catch block versuchen zu parsen.

try {
  Anfangswert = Double.parseDouble(wert);
}
catch (Exception e) {
  System.out.println("Keine gültige Eingabe!");
}

Wenn du das ganze dann mit einer while-schlaufe umrandest, wird solange gefragt, bis der User eine gültige Antwort gibt.

boolean validInput = false;
while(! validInput) {
  System.out.println("Geben sie die Anschaffungskosten an!");
  String wert = in.nextLine();
  try {
    Anfangswert = Double.parseDouble(wert);
    validInput = true;
  }
  catch (Exception e) {
    System.out.println("Keine gültige Eingabe!");
  }
}

ACHTUNG: Wenn du das so machst, kann der compiler wiederum nicht feststellen, ob die variable Anfangswert auch intialisiert wird. Deshalb initialisiere sie bereits zu beginn mit double Anfangswert = 0;

 

[hierUndDa]

Zur zweiten Frage:

Ich würde wohl eher mit einer Klasse Gegenstand (oder wie auch immer du sie nennen willst) arbeiten, die einen Preis als Instanzvariable hat und eine Methode berechneRestwert(int, int), die dir dann den korrekten Restwert berechnet.

public class Gegenstand {
    double preis;
	public Gegenstand(double preis) {
		this.preis = preis;
	}
	public double berechneRestwert(int nutzungsdauer, int jahre) {
		double rest = (1 - (((2 * nutzungsdauer + 1 - jahre) * (double) jahre) 
				/ (nutzungsdauer * nutzungsdauer + nutzungsdauer))) * preis;
        return Math.round(rest);
	}
}

 

[Jo.Mo.]

...hey, vielen Dank für die Hinweise, genau darauf wollte ich hinaus! :toll:

 

[Andi_CH]

Schreib doch grundsätzlich 1.0, 2.0 etc. hin

denn

double d = 1.0 / 2.0;

gibt wunderbare 0.5 als Resultat ;-)

 

[Jo.Mo.]

...ich hab noch Problem mit dem Programm, ich versuche irgendwie die Berechnung in einer sepraraten Klasse zu machen. Muss ich dafür zwingend eine neue Java-Datei anlegen oder kann ich die Klasse auch im eigentlichen Java-programm lassen (siehe unten). Ausserdem weiss ich nicht, wie ich der Klasse zur Berechnung die Werte übergebe, denn die sind ja bei Java nur lokal gültig wenn ich das richtig in Erninnerung habe...

import java.io.*;
import java.util.Scanner;

public class Restwert {

    public static void main(String[] args) {

	    // Variablendeklarationen
		double Anfangswert = 0;
 	    int Nutzungsdauer;
	    int Jahre;
	    double Restwert;

		// Werte eingeben
		Scanner in = new Scanner( System.in);
		boolean validInput= false;
		while (! validInput) {
	        System.out.println("Geben sie die Anschaffungskosten ein! "); // Anfangswert eingeben
	        String kosten = in.nextLine();
            try {
        	    Anfangswert = Double.parseDouble(kosten);
        	    validInput = true;
            }
            catch (Exception e) {
        	    System.out.println("Keine gültige Eingabe!");
                }
		}
		while (! validInput) {
			System.out.println("Geben sie die Nutzungsdauer in Jahren an!"); // Nutzungsdauer eingeben
			String dauer = in.nextLine();
			try {
				Nutzungsdauer = Integer.parseInt(dauer);
				validInput = true;
			}
			catch (Exception e) {
				System.out.println("Keine gültige Eingabe!");
			}
		}
		while (! validInput){
			System.out.println("Geben sie das Jahr an!"); // Jahr eingeben
			String jahre = in.nextLine();
			try {
				Jahre = Integer.parseInt(jahre);
				validInput = true;
			}
			catch (Exception e) {
				System.out.println("Keine gültige Eingabe!");
			}
		}

		// Verarbeitung
		Rest r = new Rest(double Anfangswert,int Nutzungsdauer,int Jahre);


	    // Ausgabe
        System.out.println ("Der Restwert beträgt nach "+ Jahre +" Jahren rund "+ r.getRestwert +" Euro.");

    }

    class Rest {
        //this.Anfangswert = Anfangswert; //Konstruktor

    public double berechneRestwert(int Nutzungsdauer, int Jahre) {
        double Restwert = (1 - (((2 * Nutzungsdauer + 1 - Jahre) * (double) Jahre) / (Nutzungsdauer * Nutzungsdauer + Nutzungsdauer))) * Anfangswert; //Methode
        return Math.round(Restwert);
    }
    }


}

 

[Jo.Mo.]

...ich hab das Programm jetzt nochmal überarbeitet...es hakt nur noch an einem Punkt, an dem ich nochmal Hilfe benötige:

Der JCreator Compiler meldet "inner classes cannot have static declarations" - wenn ich die Methode in der Klasse "Rest" aber nur public nenne, meckert er auch ...

import java.io.*;
import java.util.Scanner;

public class Restwert {

    public static void main(String[] args) {

	    // Variablendeklarationen
		double Anfangswert = 0;
 	    int Nutzungsdauer;
	    int Jahre;
	    double Restwert;

		// Werte eingeben
		Scanner in = new Scanner( System.in);
		boolean validInput= false;
		while (! validInput) {
	        System.out.println("Geben sie die Anschaffungskosten ein! "); // Anfangswert eingeben
	        String kosten = in.nextLine();
            try {
        	    Anfangswert = Double.parseDouble(kosten);
        	    validInput = true;
            }
            catch (Exception e) {
        	    System.out.println("Keine gültige Eingabe!");
                }
		}
		while (! validInput) {
			System.out.println("Geben sie die Nutzungsdauer in Jahren an!"); // Nutzungsdauer eingeben
			String dauer = in.nextLine();
			try {
				Nutzungsdauer = Integer.parseInt(dauer);
				validInput = true;
			}
			catch (Exception e) {
				System.out.println("Keine gültige Eingabe!");
			}
		}
		while (! validInput){
			System.out.println("Geben sie das Jahr an!"); // Jahr eingeben
			String jahre = in.nextLine();
			try {
				Jahre = Integer.parseInt(jahre);
				validInput = true;
			}
			catch (Exception e) {
				System.out.println("Keine gültige Eingabe!");
			}
		}

		// Verarbeitung
		double r = Rest.berechneRestwert(Nutzungsdauer, Jahre, Anfangswert);


	    // Ausgabe
        System.out.println ("Der Restwert beträgt nach "+ Jahre +" Jahren rund "+ r +" Euro.");

    }

    public class Rest {

       public static double berechneRestwert(int Nutzungsdauer, int Jahre, double Anfangswert) {
           double Restwert = (1 - (((2 * Nutzungsdauer + 1 - Jahre) * (double) Jahre) / (Nutzungsdauer * Nutzungsdauer + Nutzungsdauer))) * Anfangswert; //Methode
           return Math.round(Restwert);
       }
    }


}

 

[Gast2]

Leg dir für die Rest Klasse einfach eine neue java Datei an

 

[Jo.Mo.]

...eine andere Möglichkeit, dass "ineinander" zu verpacken, also dass die Klasse Rest dort mit drin ist, gibts nicht..?

Ausgelagert hatte ich es eben probiert, wird zumindest ohne Fehler kompiliert...

 

[Gast2]

...eine andere Möglichkeit, dass "ineinander" zu verpacken, also dass die Klasse Rest dort mit drin ist, gibts nicht..?

Doch. Du kannst deine innere Klasse als static class definieren.

 

[nrg]

warum willst du das "ineiner" packen?

edit:

Doch. Du kannst deine innere Klasse als static class definieren.

muss nicht sein. nach zeile 69 als packageclass geht auch.

 

[diggaa1984]

mach die Klasse Rest selbst auch static .. dann sollte das gehen

... huch nicht gesehen, dass es hier noch ne seite gab ^^ .. sry für repeat