Interpreter-Fehler Ausgabe Bummelexpress

[kokosmilch]

Hallo!
Ich habe eine Schulaufgabe bekommen und habe die auch recht zufriedenstellen gelöst (dachte ich zumindest). Der Compiler gibt keine Fehlermeldung mehr aus, aber der Interpreter macht etwas völlig falsch. Hier einmal der Quelltext:

import java.io.*;
public class Bummelexpress
{
	public static void main (String args[]) throws java.io.IOException
	{
		//Deklaration der Variablen
		double iAnzahl, iEntfernung, iFahrpreis, iMWSt, iGesFahrpreis, iBrutto;
		String sSchnellexpress, sPlatzreservierung, sNachtfahrt;

		//Start des Programms
		System.out.println("Willkommen beim Assistenten für die Fahrpreisberechnung!");

		//Wertzuweisung
		Double iNetto = 0.00;

		//Eingabe der Fahrgastanzahl
		System.out.println("Bitte geben sie die Anzahl der Fahrgäste ein.");
		BufferedReader input = new BufferedReader (
       		new InputStreamReader(System.in));
       	iAnzahl = Integer.parseInt(input.readLine());

       	//Eingabe der Entfernung
       	System.out.println("Bitte geben sie die Entfernung in Kilometer an.");
		BufferedReader input2 = new BufferedReader (
       		new InputStreamReader(System.in));
       	iEntfernung = Integer.parseInt(input2.readLine());

		//Eingabe: Schnellexpress? Ja oder Nein (J/N)
		System.out.println("Soll der Schnellexpress benutzt werden? Hinweis: Eingabe Ja oder Nein mit J oder N.");
		BufferedReader input3 = new BufferedReader (
       		new InputStreamReader(System.in));
       	sSchnellexpress = input3.readLine();

       	//Eingabe: Platzreservierung? Ja oder Nein (J/N)
		System.out.println("Soll die Platzreservierung gewählt werden? Hinweis: Eingabe Ja oder Nein mit J oder N.");
		BufferedReader input4 = new BufferedReader (
       		new InputStreamReader(System.in));
       	sPlatzreservierung = input4.readLine();

       	//Eingabe: Nachtfahrt? Ja oder Nein (J/N)
       	System.out.println("Wird die Fahrt eine Nachtfahrt sein? Hinweis: Eingabe Ja oder Nein mit J oder N.");
		BufferedReader input5 = new BufferedReader (
       		new InputStreamReader(System.in));
       	sNachtfahrt = input5.readLine();


       	//Überprüfung der Entfernung
       	if (iEntfernung<100)
       	iFahrpreis=0.25;
       	else
       	if (iEntfernung<= 500)
       	iFahrpreis=0.20;
       	else
       	iFahrpreis=0.15;

       	//Brechnung Preis mal Entfernung (Weil Preis gilt ja nur pro km, darum muss preis mal km gerechnet werden)
       	iGesFahrpreis=iFahrpreis*iEntfernung;

       	//Überprüfung der Frage: Schnellexpress?
       	if (sSchnellexpress == "J")
        iGesFahrpreis=iGesFahrpreis+5.00;
        else
        if ((sSchnellexpress != "N") || (sSchnellexpress != "J"))
        System.out.println("Ungültige Eingabe, bitte widerholen sie den Versuch mit einer gültigen Eingabe (in diesem Fall bitte J oder N für Ja bzw. Nein)");
        else

        //Überprüfung der Frage: Platzreservierung?
        if (sPlatzreservierung == "J")
        iGesFahrpreis=iGesFahrpreis+3.50;
        else
        if ((sPlatzreservierung != "N") || (sPlatzreservierung != "J"))
        System.out.println("Ungültige Eingabe, bitte widerholen sie den Versuch mit einer gültigen Eingabe (in diesem Fall bitte J oder N für Ja bzw. Nein)");
        else

        //Überpreüfung der Frage: Nachtfahrt?
        if (sNachtfahrt == "J")
        iGesFahrpreis=iGesFahrpreis*0.85;
        else
        if ((sNachtfahrt != "N") || (sNachtfahrt != "J"))
        System.out.println("Ungültige Eingabe, bitte widerholen sie den Versuch mit einer gültigen Eingabe (in diesem Fall bitte J oder N für Ja bzw. Nein)");
        else




        //Berechnungen


        //Berechnung des Nettopreises
        iNetto=iGesFahrpreis*iAnzahl;

        //Berechnung der Mehrwertsteuer
        iMWSt=iNetto*0.19;

        //Berechnung des Bruttopreises
        iBrutto=iNetto+iMWSt;



        //Ausgabe


        //Ausgabe des Nettopreises
        System.out.println("Der Nettofharpreis beträgt "+iNetto+" Euro");

        //Ausgabe der Mehwertsteuer
        System.out.println("Die Mehwertsteuer (19%) beträgt "+iMWSt+" Euro");

        //Ausgabe des Bruttopreises (Gesamtpreises)
        System.out.println("Der Bruttopreis (tatsächlicher Gesamtpreis) beträgt "+iBrutto+" Euro");




        //Programmende
        System.out.println("Vielen Dank dass sie diesen Assistenten für die Fahrpreisberechnung benutzt haben!");


     }

}

Es scheint alles zu funktionieren, bis auf dass der Interpreter einmal ausgibt, ich hätte eine ungültige Eingabe getätigt, obwohl ich immer mit J oder N geantwortet habe.
Ausserdem meint er, alle Preise würden bei 0,00 Euro liegen.
Wenn ich den Code folgendermaßen verändere (also ohne das mit der Überprüfung, wenn der Benutzer etwas ausser J oder N eingegeben hat und ohne Fehlermeldung):

import java.io.*;
public class Bummelexpress
{
	public static void main (String args[]) throws java.io.IOException
	{
		//Deklaration der Variablen
		double iAnzahl, iEntfernung, iFahrpreis, iMWSt, iGesFahrpreis, iBrutto;
		String sSchnellexpress, sPlatzreservierung, sNachtfahrt;

		//Start des Programms
		System.out.println("Willkommen beim Assistenten für die Fahrpreisberechnung!");

		//Wertzuweisung
		Double iNetto = 0.00;

		//Eingabe der Fahrgastanzahl
		System.out.println("Bitte geben sie die Anzahl der Fahrgäste ein.");
		BufferedReader input = new BufferedReader (
       		new InputStreamReader(System.in));
       	iAnzahl = Integer.parseInt(input.readLine());

       	//Eingabe der Entfernung
       	System.out.println("Bitte geben sie die Entfernung in Kilometer an.");
		BufferedReader input2 = new BufferedReader (
       		new InputStreamReader(System.in));
       	iEntfernung = Integer.parseInt(input2.readLine());

		//Eingabe: Schnellexpress? Ja oder Nein (J/N)
		System.out.println("Soll der Schnellexpress benutzt werden? Hinweis: Eingabe Ja oder Nein mit J oder N.");
		BufferedReader input3 = new BufferedReader (
       		new InputStreamReader(System.in));
       	sSchnellexpress = input3.readLine();

       	//Eingabe: Platzreservierung? Ja oder Nein (J/N)
		System.out.println("Soll die Platzreservierung gewählt werden? Hinweis: Eingabe Ja oder Nein mit J oder N.");
		BufferedReader input4 = new BufferedReader (
       		new InputStreamReader(System.in));
       	sPlatzreservierung = input4.readLine();

       	//Eingabe: Nachtfahrt? Ja oder Nein (J/N)
       	System.out.println("Wird die Fahrt eine Nachtfahrt sein? Hinweis: Eingabe Ja oder Nein mit J oder N.");
		BufferedReader input5 = new BufferedReader (
       		new InputStreamReader(System.in));
       	sNachtfahrt = input5.readLine();


       	//Überprüfung der Entfernung
       	if (iEntfernung<100)
       	iFahrpreis=0.25;
       	else
       	if (iEntfernung<= 500)
       	iFahrpreis=0.20;
       	else
       	iFahrpreis=0.15;

       	//Brechnung Preis mal Entfernung (Weil Preis gilt ja nur pro km, darum muss preis mal km gerechnet werden)
       	iGesFahrpreis=iFahrpreis*iEntfernung;

       	//Überprüfung der Frage: Schnellexpress?
       	if (sSchnellexpress == "J")
        iGesFahrpreis=iGesFahrpreis+5.00;
        else

        //Überprüfung der Frage: Platzreservierung?
        if (sPlatzreservierung == "J")
        iGesFahrpreis=iGesFahrpreis+3.50;
        else

        //Überpreüfung der Frage: Nachtfahrt?
        if (sNachtfahrt == "J")
        iGesFahrpreis=iGesFahrpreis*0.85;
        else




        //Berechnungen


        //Berechnung des Nettopreises
        iNetto=iGesFahrpreis*iAnzahl;

        //Berechnung der Mehrwertsteuer
        iMWSt=iNetto*0.19;

        //Berechnung des Bruttopreises
        iBrutto=iNetto+iMWSt;



        //Ausgabe


        //Ausgabe des Nettopreises
        System.out.println("Der Nettofharpreis beträgt "+iNetto+" Euro");

        //Ausgabe der Mehwertsteuer
        System.out.println("Die Mehwertsteuer (19%) beträgt "+iMWSt+" Euro");

        //Ausgabe des Bruttopreises (Gesamtpreises)
        System.out.println("Der Bruttopreis (tatsächlicher Gesamtpreis) beträgt "+iBrutto+" Euro");




        //Programmende
        System.out.println("Vielen Dank dass sie diesen Assistenten für die Fahrpreisberechnung benutzt haben!");


     }

}

...dann funktioniert´s. Also ne Fehlermeldung kann er ja nicht mehr ausgeben von wegen ungültige Angabe. Und die richtigen Preise zeigt er dann auch an.
Ich stehe jetzt mal so richtig auf dem Schlauch, wär nett wenn ihr mir helfen könntet.


Gruß

 

[faetzminator]

Also auf die schnelle sehe ich zwei grobe Fehler:
- Strings sind Objekte und vergleicht man deshalb mit [c]equals()[/c], nicht mit [c]==[/c]!
- Du musst nicht für jede Zeile einen neuen BufferedReader erzeugen. Einfach einen erstellen und immer wieder [c]readLine()[/c] aufrufen.

 

[nillehammer]

Du darfst Strings nicht mit == vergleichen. Ein String ist ein komplexes Objekt und muss deswegen mit equals verglichen werden.

 

[kokosmilch]

//Überprüfung der Frage: Platzreservierung?
        if (sPlatzreservierung equals ("J"))
        iGesFahrpreis=iGesFahrpreis+3.50;
        else

Sollte das so aussehen? Oder eher so:

//Überprüfung der Frage: Platzreservierung?
        sPlatzreservierung equals ("J")
        iGesFahrpreis=iGesFahrpreis+3.50;

??????????

Und sonst keine Fehler?
Ich habe zwar gesagt, dass bei der 2. version alles funzt, das stimmt aber nicht ganz! Denn er rechnet zwar mit den ersten werten, aber die die man mit Ja oder Nein bestätigen muss rechnet er nicht hinzu... liegt das nur an dem equals? Und das mit dem Buffered Reader ist ja denke ich kein fehler an sich der jetzt dazu beiträgt, dass es nicht funktioniert, oder???


Danke und MfG.
Kokosmilch

[TIPP]PS: Sorry dass ich erst jetzt antworte...[/TIPP]

 

[HimBromBeere]

if (sPlatzreservierung.equals("J"))

So vlcht.?

 

[askk]

if(a.equals(b)) {

}

So sieht equals aus.

 

[bone2]

bei deinen if-else herrscht einfach das totale chaos.

änder die bitte alle in

if (bedingung) {
    doA();
}
else if (bedingung2) {
    doB();
}
else {
    doC();
}

es ist bei dir völlig unklar was wann warum aufgerufen wird und was nicht.

 

[kokosmilch]

@askk
wirklich so?
ich probiere es mal aus...
@bone2
ähhh okayy, kann ich machen...

moment bitte...

 

[jwiesmann]

Dazu kommt noch, dass deine else evtl. verschoben sind.

        if (sSchnellexpress == "J") {
            iGesFahrpreis = iGesFahrpreis + 5.00;
        } else //Überprüfung der Frage: Platzreservierung?
        if (sPlatzreservierung == "J") {
            iGesFahrpreis = iGesFahrpreis + 3.50;
        }

prüft z.B. nur dann auf Platzreservierung, wenn sSchnellexpress nicht "J" ist.
richtiger wäre es so:

if (sSchnellexpress.equals("J")) {
            iGesFahrpreis = iGesFahrpreis + 5.00;
        }
        if (sPlatzreservierung.equals("J")) {
            iGesFahrpreis = iGesFahrpreis + 3.50;
        }
        if (sNachtfahrt.equals("J")) {
            iGesFahrpreis = iGesFahrpreis * 0.85;
        }
        //Berechnung des Nettopreises
        iNetto = iGesFahrpreis * iAnzahl;
//usw...
// wenn du ganz cool bist, schreibst du es sogar so :
if ("J".equals(sSchnellexpress)) {
            iGesFahrpreis += 5.00;
        }
// dann bekommst du auf keinen Fall eine nullpointer, sollte sSchn.. mal null sein .. aber das nur am rande

Edit fragt: Hab ich echt so lang geschrieben? .. soviele Antworten

 

[bone2]

Dazu kommt noch, dass deine else evtl. verschoben sind.

wie gesagt, das totale logische chaos. unübersichtlich ohne ende.
mit den

{ }

macht man ganz genau klar welche befehle in welchen if und else block gehören. ein einsames else wo 5 zeilen später einfach ein if kommt, wird dieses if nur ausführen wenn die bedingung vor dem else falsch war.

 

[kokosmilch]

@jwiesmann
Danke, jetzt funktioniert es!!!
Evtl. habe ich gleich nochmal eine Frage, sonst würde ich jetzt schon auf "erledigt" machen.

MfG.

 

[kokosmilch]

So, ich wollte jetzt noch eine Fußgesteuerte Schleife mit einbinden, wo gefragt wird, ob das Programm nochmal durchlaufen werden soll.
Das Ganze habe ich (versucht) so realisiert:

import java.io.*;
public class Bummelexpress_javaforum
{
	public static void main (String args[]) throws java.io.IOException
	{
		do
		{

108[code=Java]
		}
		while (sProgWdh.equals("J"));
	}

}

[/code]


Die Fehlermeldung lautet:

[Pfad].java:110: cannot find Symbol
symbol : variable sProgWdh
location: class Bummelexpress
while (sProgWdh.equals("J"));
1 Error

Ich hatte schon gehofft, diesmal alles richtig gemacht zu haben
[EDIT]Ich habe das mit dem

nicht hinbekommen, es soll halt so sein, dass es da bei Zeile 108 weitergeht...[/EDIT]

 

[Gast2]

sProgWdh muss vor der Schleife deklariert werden damit die Variable in deiner while Bedingung sichtbar ist.

 

[kokosmilch]

Oh, danke, ich hatte es zwar deklariert, aber nicht davor feif:

ERLEDIGT

Vielen Dank an alle!!!
Gruß,
Kokosmilch