Taschenrechner mit switch und while funktioniert noch nicht richtig

[L.E.O.C.Y.]

Hallo Leute,
unten seht ihr meinen bisherigen Code. Meine Aufgabe da zu lautet einen Taschenrechner den wir schon programmiert haben so umzuschreiben, dass er mit einer while schleife erweitert wird und zusätzlich beim Operator die Möglichkeit b für das Beenden des Programms dazukommt.
Ich hab es mittlerweile geschafft, dass die while schleife funktioniert. Allerdings beendet es nicht das Programm, wenn man bei Operator b eingibt.
Vllt kann mir ja jemand helfen. Bin echt am verzweifeln. Und ich brauche die Lösung leider schon bis morgen abend.
Danke schon mal im Voraus
LG Melanie

public class Rechnen {
  
    public static double rechner( double zahl1, char operator, double zahl2 )
    {
        double result = 0.0d;
        boolean Result;
        
        switch( operator )
            {
            case '+':
                result = zahl1 + zahl2;
            break;

            case '-':
                result = zahl1 - zahl2;
            break;

            case '/':
            case ':':
                result = zahl1 / zahl2;
            break;
              
            case '*':
            case 'x':
                result = zahl1 * zahl2;
                    
            case 'b':
                Result = false;
                break;
                          
            default:
                throw new IllegalArgumentException("F");
            }
      
        return(result);
      
    }
        public static void main( String[] args)
        {
            boolean Result;
            while (Result = true) {          
            char operator = javax.swing.JOptionPane.showInputDialog( "Operator eingeben (+ , - , / , : , * , x, b (für beenden))").charAt(0);
            double zahl1 = Double.parseDouble(javax.swing.JOptionPane.showInputDialog( "Erste Zahl eingeben" ) );
            double zahl2 = Double.parseDouble(javax.swing.JOptionPane.showInputDialog( "Zweite Zahl eingeben" ) );
            System.out.println( rechner( zahl1, operator, zahl2 ) );
            }
        }
        }

 

[Robat]

while (Result = true) {

Das hier ist eine Zuweisung und kein Vergleich

 

[Javinner]

Bitte den Code in Code-Tabs setzen [code=Java]Dein Code[/code]

 

[L.E.O.C.Y.]

ok

 

[L.E.O.C.Y.]

Bitte den Code in Code-Tabs setzen [code=Java]Dein Code[/code]

erledigt

 

[L.E.O.C.Y.]

Das hier ist eine Zuweisung und kein Vergleich

ich hab das nur gemacht weil bei while ja unbedingt ein boolean wert da sein muss...
was heißt das jetzt genau?

 

[Robat]

Result ist doch schon ein Boolean.
Solange Result true ist soll deine schleife weiterlaufen.
Vergleiche erledigst du mit == und nicht mit =

 

[L.E.O.C.Y.]

Result ist doch schon ein Boolean.
Solange Result true ist soll deine schleife weiterlaufen.
Vergleiche erledigst du mit == und nicht mit =

JA genau - ich glaub ich steh auf dem Schlauch - wo genau soll das == bei while?
und wie mache ich es, dass es stoppt

 

[Tarrew]

while(result) ist das gleiche wie while(result==true).

Schreibs wie du willst in deiner Schleife. Nur nicht mit einem einfachen = wie Robat schon sagte.

 

[L.E.O.C.Y.]

while(result) ist das gleiche wie while(result==true).

Schreibs wie du willst in deiner Schleife. Nur nicht mit einem einfachen = wie Robat schon sagte.

Ok das hab ich soweit verstanden.
Und was muss ich jetzt umschreiben, dass das Programm sich beendet, wenn bei Operator ein b eingetragen wird?

 

[Robat]

Die Methode rechner liefert einen Boolean zurück den du momentan vernachlässigst.

Verstehst du deinen Code überhaupt?

 

[L.E.O.C.Y.]

Naja sagen wirs so: Den Code verstehe ich solange es die alte Variante ist, wo nur gefragt wird nach Operator - Zahl1 -Zahl2 und dann das Ergebnis rauskommt - Ende des Programms.
Mit der while schleife habe ich etwas rumprobiert, da ich mit der noch nie gearbeitet habe. Leider wurde uns auch nicht so ganz gut erklärt wie man die bedingung bei while eig genau betitelt. Und da ja jetzt die Aufgabe ist, dass man eben eine schleife einbauen soll die solange das schema operator - zahl1 - zahl2 - ergebnis wiederholt bis bei Operator ein b geschrieben wird, tue ich mir da etwas schwer, da ich nicht so genau verstehe wie ich das mit der bedingung bei while umsetzen kann.
Mein erster schritt war, bei case den fall 'b' miteinzubringen.
Danach hab ich erstmal die while Schleife gesetzt - nur halt eben ohne Bedinung.
Da hab ich dann ewig rumprobiert und bin halt immer noch nicht vorwärtsgekommen was bei der Klammer genau reinmuss, damit eben im fall 'b' das programm beendet wird.

 

[L.E.O.C.Y.]

das ist der code ohne rumprobieren (habe aber die while schleife mal drinnen gelassen) und ab hier scheitert es:
mir fehlt halt bei case 'b': etwas und unten bei while

public class Rechnen {
  
    public static double rechner( double zahl1, char operator, double zahl2 )
    {
        double result = 0.0d;
      
        
        switch( operator )
            {
            case '+':
                result = zahl1 + zahl2;
            break;

            case '-':
                result = zahl1 - zahl2;
            break;

            case '/':
            case ':':
                result = zahl1 / zahl2;
            break;
              
            case '*':
            case 'x':
                result = zahl1 * zahl2;
                    
            case 'b':
                ;
                break;
                          
            default:
                throw new IllegalArgumentException("F");
            }
      
        return(result);
      
    }
        public static void main( String[] args)
        {
          
            while () {          
            char operator = javax.swing.JOptionPane.showInputDialog( "Operator eingeben (+ , - , / , : , * , x, b (für beenden))").charAt(0);
            double zahl1 = Double.parseDouble(javax.swing.JOptionPane.showInputDialog( "Erste Zahl eingeben" ) );
            double zahl2 = Double.parseDouble(javax.swing.JOptionPane.showInputDialog( "Zweite Zahl eingeben" ) );
            System.out.println( rechner( zahl1, operator, zahl2 ) );
            }
        }
        }

 

[Robat]

Verstehst du das?
Solange die Bedingung in der Schleife wahr ist wird sie noch mal ausgeführt.

public static boolean rechne(..) {
       boolean result;
      // mache was


       return result;
}

public static void main(String[] arg) {
      boolean result = true;

       while(result) {
              // mache was
              result = rechne(...);
        }
}

Ps: sorry für die Formatierung. Bin am Handy

 

[L.E.O.C.Y.]

ja das verstehe ich
aber ich weiß halt nicht was ich bei case 'b' und while schreiben muss damit er dann erkennt: oh da muss jetzt das programm beendet werden

 

[Meniskusschaden]

Die Methode rechner liefert einen Boolean zurück den du momentan vernachlässigst.

Nein, sie liefert die double-Variable result zurück und der Wert wird auch für die Ausgabe verwendet. Die lokale Boolean-Variable Result wird nicht verwendet. In der main-Methode gibt es auch eine lokale Boolean-Variable Result. Aber die bekommt natürlich nichts davon mit, was mit Result in der Methode rechne() passiert.
Meines Erachtens sollte rechne() gar nicht erst aufgerufen werden, wenn ein b eingegeben wurde. Dementsprechend gehört das auch nicht in das Switch-Statement, sondern sollte in der Hauptschleife abgewickelt werden.

 

[L.E.O.C.Y.]

Nein, sie liefert die double-Variable result zurück und der Wert wird auch für die Ausgabe verwendet. Die lokale Boolean-Variable Result wird nicht verwendet. In der main-Methode gibt es auch eine lokale Boolean-Variable Result. Aber die bekommt natürlich nichts davon mit, was mit Result in der Methode rechne() passiert.
Meines Erachtens sollte rechne() gar nicht erst aufgerufen werden, wenn ein b eingegeben wurde. Dementsprechend gehört das auch nicht in das Switch-Statement, sondern sollte in der Hauptschleife abgewickelt werden.

Ok und das heißt? wie muss man das dann umschreiben?

 

[Meniskusschaden]

Sorry, aber das ist mir dann doch zu wenig Eigenleistung, wenn du nach einer Minute schon weißt, dass du du den Vorschlag nicht selbst umsetzen kannst.

 

[Robat]

Stimmt da hast du recht.
Keine Ahnung wie ich darauf gekommen bin.

 

[L.E.O.C.Y.]

Sorry, aber das ist mir dann doch zu wenig Eigenleistung, wenn du nach einer Minute schon weißt, dass du du den Vorschlag nicht selbst umsetzen kannst.

Du hast einen Vorschlag verbessert und das was du geschrieben hast tut mir leid verstehe ich nicht was du meinst. Ich habe mit Java jetzt seit 4 Wochen zu tun.

 

[alexanderbehrens]

Ich würde die Beendung mit b im Hauptprogramm per while überprüfen.

 

[Xyz1]

Leichen ausbuddeln du hast. @alexanderbehrens

Bearbeitung:
* ausgebuddelt

 

[truesoul]

Hallo L.E.O.C.Y

was Meniskusschaden meint ist das du in dieser

while(wo ist die Bedingung?){         
   char operator = javax.swing.JOptionPane.showInputDialog( "Operator eingeben (+ , - , / , : , * , x, b (für beenden))").charAt(0);
  
}

abfragen solltest ob 'b' eingegeben wurde und nicht in der Methode rechner

Grüße