Fehlerhafte Switch

[lulas[]args]

heyho!
immer wenn ich mich an einer einfachen switch versuche, habe ich das Problem, dass ich nach case1 immer fehler angezeigt bekomme. kann mir vllt jemand helfen, diesen basisfehler zu erkennen und abzustellen?
danke! :bahnhof:

public class Rechenmaschine {

  
    public static void main(String[] args) {
        double a;
        double b;
        int auswahl;
        double ergebnis;
        
        public double rechnen () {
           System.out.println ("Zu welcher Zahl willst du etwas addieren?");
           double a = new java.util.Scanner(System.in).nextDouble();
           System.out.println ("Welche soll dazu gezählt werden?");
           double b = new java.util.Scanner(System.in).nextDouble();
           int auswahl = new java.util.Scanner(System.in).nextInt();
           
           switch (auswahl) {
               case 1:
                   double ergebnis = a + b;
                   return ergebnis;
                   
                   
               case 2:
                   double ergebnis = a - b;
                   return ergebnis;
                   
                   
               case 3:
                   double ergebnis = a * b;
                   return ergebnis;
                   
                   
               case 4:
                   double ergebnis = a / b;
                   return ergebnis;
                   
                   
               default:
                   System.out.println ("Eingabefehler");
                   break;
                   
                   }
           System.out.println ("Das Ergebnis ist " + ergebnis);
           
         }
       } 
    }

 

[Gast2]

1) Die Methode rechnen() ist IN der main, das geht nicht.
2) Du hast die Variablen a, b und ergebnis doppelt bis öfter deklariert, das geht auch nicht.

 

[lulas[]args]

Methode hab ich verbessert, aber das mit dem deklarieren ist mir nicht so klar. Ich habe es jetzt oben nicht mehr deklariert, aber ich möchte es ja je nach Fall deklarieren. Warum nimmt mir die switch nicht die Auswahl an und führt die Deklaration demenstprechend durch? danke!

 

[Gast2]

Du legst in deinen cases jeweils ne neue Variable ergbnis an. Die wäre dann auch nur in dem case sichtbar, das willst du ja nicht.

Außerdem brauchst du auch nicht drei verschiedene Scanner Instanzen erzeugen, eine reicht völlig.

Wenn du noch Probleme mit dem Code hast, dann poste nochmal deine aktuelle Version.

 

[lulas[]args]

GESCHAFFT, es läuft :applaus:

public class Rechenmaschine {
    
    public static void main(String[] args) {
        
         double ergebnis = 0;
         
           System.out.println ("Deine erste Zahl?");
           double a = new java.util.Scanner(System.in).nextDouble();
           System.out.println ("Deine zweite Zahl?");
           double b = new java.util.Scanner(System.in).nextDouble();
           
        System.out.println ("Wähle die gewünschte Operation und drücke ENTER");
        System.out.println ("1: Addition");
        System.out.println ("2: Subtraktion");
        System.out.println ("3: Multiplikation");
        System.out.println ("4: Division");
        int auswahl = new java.util.Scanner(System.in).nextInt();
           
           switch (auswahl) {
               case 1:
                   ergebnis = a + b;
                   break;
                                                  
               case 2:
                   ergebnis = a - b;
                   break;
                                    
               case 3:
                   ergebnis = a * b;
                   break;
                                                         
               case 4:
                   ergebnis = a / b;
                   break;
                                                        
               default:
                   System.out.println ("Eingabefehler");
                   break;
                   
                   }
           System.out.println ("Das Ergebnis ist " + ergebnis);
           
        }
        
    }

Wie könnte ich denn jetzt noch zur Verbesserung die Instanzen des Scanners erzeugen?

 

[Gast2]

Naja, aktuell erzeugst du drei Instanzen. Erstell dir am Anfang eine und nutz die in deinem gesamten Programm.

 

[selim]

Ich hätte es etwas anders geschrieben

[Java]
import static javax.swing.JOptionPane.showInputDialog;
import static java.lang.Integer.parseInt;

public class Rechenmaschine {

/**
* @param args
*/
public static void main(String[] args) {
// TODO Auto-generated method stub
double a1 = parseInt( showInputDialog( "Erste Zahl" ) );
double b1 = parseInt( showInputDialog( "Zweite Zahl" ) );
int auswahl = parseInt( showInputDialog( "Wähle die gewünschte Operation und drücke ENTER\n 1: Addition\n 2: Subtraktion\n 3: Multiplikation\n 4: Division" ) );

System.out.println( rechnen (a1, b1)[auswahl] );
}

private static double[] rechnen(double a1, double b1) {
// TODO Auto-generated method stub
return new double[]{ a1 + b1, a1 - b1, a1 * b1, a1 / b1 };
}
}
[/Java]

Hier fehlt noch eins zwei dinge aber denke sollte passen.