Suche Fehler bei Programm

[Guest]

Habe ein Problem mit folgendem Code. Ich bekomme immer folgende Fehlermeldung, obwohl, die entsprechenden Variablen deklariert und initialisiert sind (zumindest in meinen Augen).

import java.io.*;

class K13A1_Internet_Delikatessen
{
public static void main(String[]args) throws IOException
{
BufferedReader buff = new BufferedReader(new InputStreamReader(System.in));


String ArtikelSt, PreisSt, ExpressversandtSt;
Double PreisDb, ExpressversandtDb, VersandtDb, ZuschlagDb;

//------------------------------------------------

System.out.println("Internet Delikatessen v.01");
System.out.println("");

System.out.print("Geben Sie den Artikel ein: ");
ArtikelSt = buff.readLine();

System.out.print("Geben Sie den Preis ein: ");
PreisSt = buff.readLine();
PreisDb = Double.parseDouble(PreisSt);


System.out.print("Expressversandt (0==nein, 1==ja): ");
ExpressversandtSt = buff.readLine();
ExpressversandtDb = Double.parseDouble(ExpressversandtSt);



System.out.println("");

//------------------------------------------------

if (PreisDb < 10)
    {
     PreisDb = PreisDb+2;
     VersandtDb = 2.0;
    }

if (PreisDb >= 10)
    {
     PreisDb = PreisDb+3;
     VersandtDb = 3.0;
    }

if (ExpressversandtDb == 1)
    {
     PreisDb = PreisDb+5;
     ZuschlagDb = 5.0;
    }
//------------------------------------------------

System.out.println("Rechnung");


System.out.println(ArtikelSt+": $"+PreisDb +" ($" +VersandtDb +" Versandt)");

if (ExpressversandtDb == 1)
System.out.print("Mit $" +ZuschlagDb +"Expresszuschlag");


}
}

C:\Dokumente und Einstellungen\Christopher\Java\1T>javac K13A1_Internet_Delikate
ssen.java
K13A1_Internet_Delikatessen.java:58: variable VersandtDb might not have been ini
tialized
System.out.println(ArtikelSt+": $"+PreisDb +" ($" +VersandtDb +" Versandt)");
^
K13A1_Internet_Delikatessen.java:61: variable ZuschlagDb might not have been ini
tialized
System.out.print("Mit $" +ZuschlagDb +"Expresszuschlag");
^
2 errors

 

[sliwalker]

Hi,

ganz klar.
Eigentlich meine ich, dass es nur eine Warnung sein sollte, aber egal. In Deinen if-Abfragen weist Du die Double-Variablen sich selbst, plus einen Wert zu. Es könnte aber sein, dass der User gar nichts eingegeben hat. Damit würden Deine Werte sinnfrei werden.

Punkt 1:
Initialisiere Deine Variablen in der korrekten Weise.

Punkt 2:
Wenn Du mit Double arbeiten willst. Dann verwende bei allen Zuweisungen, Rechnungen usw. immer das Format:

	double d1 = 0.0d;
	double d2 = 1.5d;
		
	System.out.println("Erg: " + (d1+d2));

Punkt 3:
Der richtige Datentyp für Deine Variablen heißt "double" nicht "Double". "Double" sind Wrapperklassen, die Dir zB erweiterte Funktionen mit dem handling von "double"-Variablen bieten. Zum rechnen nimm lieber Kleindouble
(Deine parsen ist aber trotzdem richtig, denn hier nutzt Du tatsächlich die Wrapperklasse "Double")

greetz
SLi
[/code]

 

[Leroy42]

In Deinen if-Abfragen weist Du die Double-Variablen sich selbst, plus einen Wert zu. Es könnte aber sein, dass der User gar nichts eingegeben hat. Damit würden Deine Werte sinnfrei werden.

Das stimmt nicht: Er erhöht nur PreisDb und das war bereits vor den if-Abfragen initialisiert.

Abgesehen davon, daß sliwalker natürlich Recht hat was Double==>double betrifft sind es
zwei Probleme

1. ZuschlagDb wird tatsächlich nur gesetzt, wenn die dritte if-Abfrage true ergibt. Es muß
vorher mit 0 initialisiert werden.
2. VersandtDb (Wo kommt eigentlich das klein t her? Süss ) wird zwar innerhalb
einer der beiden ersten if-Abfragen immer gesetzt, der Compiler ist aber nicht smart
genug, daß zu erkennen. Ich vermute mal sowieso, daß die ersten beiden Abfragen nicht
richtig sind. Ist z.B. PreisDb gleich 9 dann wird es in der ersten Abfrage um 2 erhöht und, da
es jetzt mit 11 auch größergleich 10 ist, wird es nocheinmal um 3 erhöht ???:L
Ersetze

if (PreisDb < 10)
    {
     PreisDb = PreisDb+2;
     VersandtDb = 2.0;
    }

if (PreisDb >= 10)
    {
     PreisDb = PreisDb+3;
     VersandtDb = 3.0;
    }

einfach durch

if (PreisDb < 10) {
    PreisDb += 2;
    VersandtDb = 2;
} else {
    PreisDb += 3;
    VersandtDb = 3.0;
}

Noch ein Tip(p): Gewöhn dir langsam an, Bezeichner für Variablen mit kleinem Anfangsbuchstaben zu schreiben[/i]