Wie kann ich bei int-Variablen im exception handler auf bestimmte Strings reagieren?

[MarkusFromBerne]

Hallo zusammen

Ich habe ein try-Block, in dem der User per Scanner eine Int-Variable eingibt. Dazu habe ich einen catch-Block, um InputMismatchException abzufangen (alles, was nicht int ist). So weit, so gut.

Nun will ich aber bei bestimmten, eigentlich falschen, User-Eingaben, eine If-Schleife einbauen. Konkret; falls der User "quit" eingibt, soll abgebrochen werden.
Dazu 2 Fragen:

1. Ist es sinnvoller, diese IF-Schleife direkt nach der User-Eingabe einzubauen oder besser erst im catch-Block?
2. Ich habe eine funktionierende Lösung: Anstatt int-Variablen werden String-Variablen abgefragt. Die werden anschliessend in Ints umgewandelt. Das ist mir zu unelegant. Gibt es bessere Lösungen, mit denen ich int-Variablen behalten und trotzdem zusätzlich "quit" abfangen kann?

Der in der Aufgabenstellung enthaltene Anfangscode sowie die Aufgabenstellung selbst sind in meinem Code enthalten, zuunterst.
Danke für eure Tipps.
Gruss aus dem regnerischen Bern,
Markus

import java.util.*;

public class kapitel80a1 {
    public static void main(String[] args) {
        Scanner scan = new Scanner( System.in  );
        String num="", div="" ;
        int numInt=0, divInt=0;
        boolean allesOk=false;

        while (allesOk==false){
            try{
                System.out.print("Enter the numerator: ");
                num = scan.next();
                if (num.charAt(0)=='q'||num.charAt(1)=='Q'){
                    System.out.println("na dann tschuess");
                    System.exit(0);
                }
                numInt=Integer.parseInt (num);
                System.out.print("Enter the divisor  : ");
                div = scan.next();
                divInt=Integer.parseInt(div);
                System.out.println( numInt + " / " + divInt + " is " + (numInt/divInt) + " rem " + (numInt%divInt) );
                allesOk=true;
            } catch (ArithmeticException ex ){
                System.out.println("You can't divide " + num + " by " + div);
            } catch (InputMismatchException ie) {
                System.out.println("Hoppla, da ist wohl ein Buchstabe oder so reingerutscht.");
                System.out.println("Probiers nomau");
            } catch (NumberFormatException ne){
                System.out.println("Hoppla, da ist wohl ein Buchstabe oder so reingerutscht.");
                System.out.println("Probiers nomau");
            } catch (StringIndexOutOfBoundsException se){
                System.out.println("check mal deine Eingabe fuer den teiler");
            } finally {
                System.out.println("If something went wrong, you entered bad data." );
            }
            System.out.println("------------------------------------------------------");

        }
        System.out.println("I han fertig!");
        
    }

/*
//Original-Code:
import java.util.* ;

public class SquareUser
{

  public static void main ( String[] a )
  {
    Scanner scan = new Scanner( System.in  );
    int num = 0 ;
    boolean goodData = false;

    while ( !goodData )
    {
      System.out.print("Enter an integer: ");
      
      try
      {
        num = scan.nextInt();     
        goodData = true;
      }

      catch (InputMismatchException  ex )
      {
        System.out.println("You entered bad data." );
        System.out.println("Please try again.\n" );
        String flush = scan.next();
      }
    }

    System.out.println("The square of " + num + " is " + num*num );
}




Enhance the DivisionPractice program from the chapter:

Put in a loop so that the user is repeatedly asked for the numerator and the divisor.
For each set of data, the program prints out the result,
or an informative error message if there is a problem (division by zero or poor input data).
The program continues looping, even if there is a problem

Exit the loop when data entered for the numerator start with characters "q" or "Q".
Don't print out an error message in this case.
Don't ask for the divisor if the user just asked to quit.


Here is sample output from one run:
Enter the numerator: 12
Enter the divisor:  4
12 / 4 is 3

Enter the numerator: 12
Enter the divisor  : 0
You can't divide 12 by 0

Enter the numerator: glarch
You entered bad data.
Please try again.

Enter the numerator: quit
You will need to use the method charAt() from the String class.

Click here to go back to the main menu.
*/
}

 

[kneitzel]

Also erst einmal: "IF Schleife" - If ist natürlich keine Schleife! Es gibt lediglich eine if Bedingung und dann eine bedingte Ausführung eines Befehls oder Blocks.

Dann: Exception sind immer etwas, das eine unerwartete Sache ist. Die Festplatte ist kaputt, das Netzwerk geht plötzlich nicht mehr oder irgend was, was unerwartet ist. Eine Usereingabe fällt da nicht drunter. Erst recht nicht, wenn es eine gültige, erwünschte Eingabe ist. Daher ja: Eingabe ordentlich abfragen als String und dann entsprechend die Eingabe prüfen.

Und eleganter bekommst Du es über einen Objektorientierten Ansatz. Statt mit Basistypen kannst Du die Eingabe natürlich auch modellieren. Das kann - je nach Anforderung - etwas triviales sein oder eben etwas mit vielen Klassen...

Also man könnte eine Klasse Eingabe bauen, die dann eine Eingabe entgegen nimmt und zentral auswertet. Dann hättest Du im Code sowas wie:

Eingabe eingabe = new Eingabe(); // oder = Eingabe.next() oder so - da wäre z.B. ein Iterator Pattern denkbar - also mit hasNext und next Eingaben abarbeiten ...
switch(eingabe.getInputType()) {
    case QUIT_COMMAND: // quit ausführen.
        break;
    case INVALID_INPUT: // was auch immer du da machen möchtest ...
        break;
    case NUMBER:
        int number = eingabe.getNumber(); // Die Zahl wäre dann natürlich so abfragbar ...
        // weitere Nutzung der Zahl ...
        break;
    default:
        // Was auch immer ... z.B. ein throw new RuntimeException("New InputType defined without changing the use in the code!");
}

Die Frage ist immer, was man genau benötigt. Und bei ungültigen Eingaben erneut eine Eingabe abzufragen, wäre dann auch an einer zentralen Stelle.

 

[MarkusFromBerne]

Also erst einmal: "IF Schleife" - If ist natürlich keine Schleife! Es gibt lediglich eine if Bedingung und dann eine bedingte Ausführung eines Befehls oder Blocks.

Dann: Exception sind immer etwas, das eine unerwartete Sache ist. Die Festplatte ist kaputt, das Netzwerk geht plötzlich nicht mehr oder irgend was, was unerwartet ist. Eine Usereingabe fällt da nicht drunter. Erst recht nicht, wenn es eine gültige, erwünschte Eingabe ist. Daher ja: Eingabe ordentlich abfragen als String und dann entsprechend die Eingabe prüfen.

Und eleganter bekommst Du es über einen Objektorientierten Ansatz. Statt mit Basistypen kannst Du die Eingabe natürlich auch modellieren. Das kann - je nach Anforderung - etwas triviales sein oder eben etwas mit vielen Klassen...

Also man könnte eine Klasse Eingabe bauen, die dann eine Eingabe entgegen nimmt und zentral auswertet. Dann hättest Du im Code sowas wie:

Eingabe eingabe = new Eingabe(); // oder = Eingabe.next() oder so - da wäre z.B. ein Iterator Pattern denkbar - also mit hasNext und next Eingaben abarbeiten ...
switch(eingabe.getInputType()) {
    case QUIT_COMMAND: // quit ausführen.
        break;
    case INVALID_INPUT: // was auch immer du da machen möchtest ...
        break;
    case NUMBER:
        int number = eingabe.getNumber(); // Die Zahl wäre dann natürlich so abfragbar ...
        // weitere Nutzung der Zahl ...
        break;
    default:
        // Was auch immer ... z.B. ein throw new RuntimeException("New InputType defined without changing the use in the code!");
}

Die Frage ist immer, was man genau benötigt. Und bei ungültigen Eingaben erneut eine Eingabe abzufragen, wäre dann auch an einer zentralen Stelle.

okay, danke schon mal für die Klarstellungen.
Den Rest muss ich noch testen ;-)

 

[Panda9296]

Eine Usereingabe fällt da nicht drunter

naja was ist mit der Numberformatexception als RuntimeException oder gleich IllegalArgumentexception, die durch aus behandelt werden können und unter einer RuntimeException fallen. Nutzereingabe entspricht zum Beispiel nicht den Vorgaben

 

[kneitzel]

Also in der Regel sollte eine Eingabe überprüft werden. Da kann man gerne streiten, in wie weit bei Zahlen einfach eine Konvertierung versucht wird um dann die Exception zu fangen. Aber bei Applikationen wird oft mehr Komfort gegeben, d.h. in einem Eingabefeld für eine Zahl kann auch nur eine Zahl eingegeben werden ... Das Java Framework bietet bei dem Konvertieren nur den direkten Versuch. Das ist bei anderen Umgebungen teilweise anders - das .Net Framework hat da z.B. auch ein tryParse.

Aber bei der Reihenfolge wäre auf jeden Fall immer etwas wie:
- Erst alle Nicht-Zahlen abprüfen (Also ist es ein Quit Befehl?)
- Dann, wenn es nur noch eine Zahl sein kann, das parsen probieren.
Das kann durchaus akzeptabel sein. Aber ich würde immer erst eine Validierung z.B. mit Regulären Ausdrücken machen oder so ...

 

[Panda9296]

okay naja ich bin vom möglichen ausgegangen und wollte das an dieser Stelle, auch für mich zum Verständnis korrigieren. Ich bin bei dem Beispiel oben von einer Konsolenapplikation ausgegangen. Trotzdem möglich ist es.