while Schleife Taschenrechner

[salem7]

Hallo, bin relativ neu und habe folgendes Problem. Ich möchte es schaffen, dass wenn man einen anderen Operator als +,-,*,/,% oder X eingibt man direkt wieder aufgefordert wird einen Operator einzugeben. Wie könnte ich das lösen?

import java.util.Scanner;
public class TaschenrechnerV3 {
    public static void main(String[] args) {
        Scanner scannernew = new Scanner(System.in);
        char a = '*';

        while (a == '+' || a == '-' || a == '*' || a == '/' || a == '%') {
            System.out.println("Enter operating symbol (+,-,*,/,%)");
            a = scannernew.nextLine().charAt(0);
            if (a== 'X') {
                break;


                }

            }
            System.out.println("Enter first number:");
            int b = scannernew.nextInt();
            System.out.println("Enter second number:");
            int c = scannernew.nextInt();

            switch (a) {
                case '+':
                    System.out.println(b + " + " + c + " = "  + (b + c));
                    break;
                case '-':
                    System.out.println(b + " - " + c + " = "  + (b - c));
                    break;
                case '*':
                    System.out.println(b + " * " + c + " = "  + (b * c));
                    break;
                case '/':
                    System.out.println(b + " / " + c + " = "  + (b / c));
                    break;
                case '%':
                    System.out.println(b + " % " + c + " = "  + (b % c));
                    break;
            }
            scannernew.nextLine();
        }

    }

 

[KonradN]

Die Problematik hier ist doch eigentlich nur, dass hier so eine Funktionalität innerhalb eines vorhandenen Konstrukts erfolgen soll.

Die einfachste Lösung ist immer ein Teilen und Herrschen: Teil die Problematik auf, so dass diese separat zu lösen ist.

Also aus dem Code:

            System.out.println("Enter operating symbol (+,-,*,/,%)");
            a = scannernew.nextLine().charAt(0);

macht man einfach ein: a = getCharFromUser("Enter operating symbol (+,-,*,/,%)");

Und die Ausgabe packt man dann in die neue Methode:

public static char getCharFromUser(String message) {
    char result;
    System.out.println("Enter operating symbol (+,-,*,/,%)");
    result = scannernew.nextLine().charAt(0);
    return result;
}

Das hat jetzt noch keine weitere Funktionalität - aber ist es jetzt einfacher, da eine Schleife einzubauen, die weiter läuft, wenn das Zeichen ungültig ist?