wieso geht das nicht.

[programmiene]

hi !
habe folgendes programm...

import java.io.*;
import java.nio.charset.*;

public class a2 {
	public static void main(String[] args){

		InputStreamReader isr=new InputStreamReader(System.in);
		BufferedReader br=new BufferedReader(isr);
		
		double zahl1=0.0, zahl2=0.0, ergebnis=0.0;
		String input=null;
		
		
		// Erste Zahl einlesen
		System.out.println("Bitte geben Sie die erste Zahl ein:");
		try{
	   		input=br.readLine();
	   	}
		catch(IOException e){}
		//Eingabe von String nach double wandeln
		zahl1=Double.parseDouble(input);
	 
		
		// Zweite Zahl einlesen
		System.out.println("Bitte geben Sie die zweite Zahl ein:");
		try{
	   		input=br.readLine();
	   	}
		catch(IOException e){}
		//Eingabe von String nach double wandeln  
		zahl2=Double.parseDouble(input);
		
		
		//Operator eingeben
		System.out.println("Bitte geben Sie den Rechenoperator an:");
		try{
	   		char operator = br.readLine().trim().charAt(0);	// trim() entfernt Leerzeichen  														// charAt(0) = erstes Zeichen des Strings 
	   	}
		catch(IOException e){} 
	
		//Eingegebenen Zahlen berechnen
		switch (operator) {
			case '+':   // addieren
						ergebnis = zahl1 + zahl2;
						System.out.println(zahl1 + " + " + zahl2 + " = " + ergebnis);
						break;
						
			case '-':   // subtrahieren
						break;
						
			case '*':   // multiplizieren
						break;
						
			case '/':   // dividieren
						break;
						
			default:
			System.out.println( "Operand nicht definiert!" );
		}
   
   }
   
}

und bekomme die fehlermeldung

a2.java:44: cannot resolve symbol
symbol  : variable operator
location: class a2
                switch (operator) {
                        ^
1 error

aber was mache ich falsch... das müsste doch eigentlich gehen...
wer weiss rat!?

danke für jede hilfe...

 

[mic_checker]

      try{
            char operator = br.readLine().trim().charAt(0);   // trim() entfernt Leerzeichen                                            // charAt(0) = erstes Zeichen des Strings
         }
      catch(IOException e){}
   
      //Eingegebenen Zahlen berechnen
      switch (operator) {

Du deklarierst operator innerhalb vom try..catch Block, außerhalb dieses Blocks ist die Variable nicht sichtbar. Deklarier sie außerhalb des Blocks und er sollte diesbezüglich nicht mehr meckern...

 

[Guest]

danke !!! es tut... wie blöd auch...

 

[mic_checker]

Noch was anderes:
- Nächstens bitte einen aussagekräftigeren Namen für den Thread wählen
- Klassennamen werden groß geschrieben

zahl1=Double.parseDouble(input); 
...
zahl2=Double.parseDouble(input);

- hier solltest du noch die NumberFormatException abfangen....

 

[Guest]

werde ich mache... wie fange ich denn diese NumberExeptions ab!? und wo zu ist das gut.

 

[mic_checker]

Na so wie jede andere Exception auch:

try {
zahl1=Double.parseDouble(input); 
} catch(NumberFormatException nfe) {
nfe.printStackTrace();
}

Du könntest das ganze auch gleich in die try...catch vom vorherigen Block schreiben und zusammen mit dem User Input behandeln (also neben IOException noch die NumberFormatException abfangen).

Wozu das gut ist?

Falls der User was eingibt was der Compiler nicht parsen kann, z.B. er gibt für die erste Zahl "a,e2" ein oder so was....

 

[Guest]

ok, ich denke ich habe diese NumberFormatExeption jetzt gemacht. aber mal eine andere frage, ich habe jetzt ganz viele try-blöcke hinter einander... kann ich diese vielleicht auch irgendwie zusammenfassen um code zu sparen, oder um es auch übersichtlicher zumachen?

 

[mic_checker]

DU kannst, wenn du willst, einen großen machen, der IOException und NumberFormatException abfängt, in dem Block dann readLine() und parseDouble() entsprechend aufrufen (wie bisher)...

Btw. du solltest zumindest nen StackTrace ausgeben lassen (in catch), also z.B.

try {
...
} catch(IOException ioe) {
ioe.printStackTrace();
}

 

[programmiene]

wow... ich komme grad nicht ganz mit...
das müsste dann in etwa so aussehen, oder

		// Erste Zahl einlesen
		System.out.println("\n\n\nBitte geben Sie die erste Zahl ein:");
		try{
	   		input=br.readLine();
	   		zahl1=Double.parseDouble(input);
	   	}
		catch(IOException ioe){}
	 	catch(NumberFormatException e){}
		
		// Zweite Zahl einlesen
		System.out.println("\nBitte geben Sie die zweite Zahl ein:");
		try{
	   		input=br.readLine();
	   		zahl2=Double.parseDouble(input);
	   	}
		catch(IOException ioe){}
		catch(NumberFormatException e){}

oder?

ist es egal, welchen namen ich den exception gebe?

wenn ich das programm ausführe und dann einen buchstaben statt einer zahl eingebe, dann wird die exception ausgegeben?! bei mir reagiert der gar nicht darauf...

 

[mic_checker]

Du solltest dir noch den StackTrace ausgeben lassen:

try {
..
      } catch(IOException ioe) {
	         ioe.printStackTrace();
	   } catch(NumberFormatException nfe) {
		   	nfe.printStackTrace();
		   }

Dann sollte er im Fall der Fälle die Fehlermeldung ausgeben

 

[programmiene]

cool... danke...
ist das jetzt nur für eine zahl, oder kann ich das auch gleich für beide benutzen?!

 

[mic_checker]

Wenn du einen großen try..catch Block machst, dann reicht es im entsprechenden catch Block den StackTrace auszugeben. Ansonsten musst du halt in deinem zweiten try...catch Block entsprechendes ausgeben.

Du musst ja selbstverständlich das ganze auf beide Zahlen anwenden, da es keinen Sinn macht die eine Zahl zu überprüfen, die andere aber net