We behandele ich diese Exception?

[nossek]

Guten Tag,

derzeit beschäftige ich mich erstmals mit Exceptions. Habe mir ein sehr einfaches Beisiel überlegt, nur leider funktioniert es noch nicht:

import java.util.Scanner;
import java.io.IOException;

public class Main {

    public static void main(String[] args) {

        int i = 1;
        Scanner scanner = new Scanner(System.in);
 
        while(i != 0 ){
            try{System.out.println("Bitte geben sie eine Zahl ein. 0 beendet das Programm.");
                i = scanner.nextInt();
                System.out.println("Sie haben die Zahl "  +i +" eingegeben");
            }
            catch(Exception e){
                System.out.println("Ihre Eingabe war fehlerhaft. Bitte versuchen Sie es erneut.");             
            }   
        }
        
    }
}

Hier fehlt etwas, das ist klar. Die Ausnahme wird ja garnicht behandelt. Derzeit wird bei einer fehlerhaften Eingabe die while schleife endlos weiter ausgeührt, ohne daß eine erneute Eingabe möglich ist (Wieso eigentlich?). Ich dachte zunächst, die Lösung sei, im catch-block der Variable i irgendeinen zulässigen Wert zuzuweisen, das führt aber zum selben Ergebnis.

Wie behandle ich diese Ausnahme korrekt?

Grüße nossek

 

[sowieso2]

public class Main {

public static void main(String[] args) {

int i = 1;
boolean bedingung = true;
Scanner scanner = new Scanner(System.in);

while(bedingung){
try{System.out.println("Bitte geben sie eine Zahl ein. 0 beendet das Programm.");
i = scanner.nextInt();
if( i == 0)
break; // oder bedingung = false;
System.out.println("Sie haben die Zahl " +i +" eingegeben");
}
catch(Exception e){
System.out.println("Ihre Eingabe war fehlerhaft. Bitte versuchen Sie es erneut.");
}
}

}
}

 

[sowieso2]

Oops

 	 public class Main {

public static void main(String[] args) {

int i = 1;
boolean bedingung = true;
Scanner scanner = new Scanner(System.in);

while(bedingung){
try{System.out.println("Bitte geben sie eine Zahl ein. 0 beendet das Programm.");
i = scanner.nextInt();
if( i == 0)
break; // oder bedingung = false;
System.out.println("Sie haben die Zahl " +i +" eingegeben");
}
catch(Exception e){
System.out.println("Ihre Eingabe war fehlerhaft. Bitte versuchen Sie es erneut.");
}
}

}
}

 

[Final_Striker]

Wieso? Du sagts doch, dass eine ungültige Zahl eingegeben wurde.
Oder was willst du stattdessen machen?

 

[Flown]

Ich glaub du meintest so:

import java.util.Scanner;


public class Test {

	public static void main(String[] args) {
		int i = 1;
        Scanner scanner = new Scanner(System.in);
 
        while(i != 0 ){
            try{System.out.println("Bitte geben sie eine Zahl ein. 0 beendet das Programm.");
                i = scanner.nextInt();
                System.out.println("Sie haben die Zahl "  +i +" eingegeben");
            }
            catch(Exception e){
                System.out.println("Ihre Eingabe war fehlerhaft. Bitte versuchen Sie es erneut.");
                scanner = new Scanner(System.in);
            }   
        }
	}
}

 

[SlaterB]

@sowieso2
das ändert doch nix, die Bedingung sitzt schon an der richtigen Stelle,
Problem ist hierbei, dass die fehlerhafte andere Eingabe im Scanner verbleibt,
ein scanner.nextLine(); räumt in Fehlerfall auf

komplizierte Klasse..

edit:
ständig
> scanner = new Scanner(System.in);
ist nicht schön..

 

[Ark]

Das Problem ist, dass die Eingabe beim Scanner nach wie vor verfügbar ist (bei einem Fehler schreitet der Scanner nicht in der Eingabe voran) und er deshalb immer wieder dieselbe eingegebene Zeichenkette umzusetzen versucht.

Ark

 

[nossek]

Mensch seid ihr schnell!

@final-striker:
klar, die Fehlermeldung kommt, es ging darum nach der fehlerhaften Eingabe eine neue Eingabe zu ermöglichen.

@sowieso2:
Dein Beispiel zeigt das gleiche Verhalten (nach fehlerhafter Eingabe wird die Schleife endlos durchlaufen ohne weitere Eingabemöglichkeit). Aber Du hast wohl Recht damit daß die Schleifenbedingung besser ein boolean sein sollte.

@SlaterB:
mit scanner.nextLine(); im catch Block funktioniert.

Danke erstmal an alle.

Offenbar wirft scanner.nextInt() nur die InputMismatchException. Ich möchte als nächstes versuchen, eine Prog zu schreiben, bei der verschiedene Exepitons geworfen werden können - InputMismatchException oder ArrayIndexOutOfBoundsException - und die dann unterschiedlich behandeln...

grüße

nossek

 

[nossek]

Sooo, dieses Prog macht folgendes:

Zuerst soll eine ganze Zahl eingegeben werden.
(Fehlerbehandlung wie gehabt, wenn es keine Zahl ist oder die Zahl den Wertebereich von int überschreitet: eine neue Eingabe ist möglich)

Wenn ersteres geklappt hat: es soll eine Position angeben werden, unter der diese Zahl in einem Array gespeichert werden soll.
Hier werden zwei mögliche Fehler abgefangen:
- Die Eingabe ist keine Zahl bzw. überschreitet den Wertebereich.
- Die Position überschreitet die Araygrenzen.

Wenn ein Fehler bei der Eingabe der Position passiert, muß der Benutzer allerdings ganz von vorne anfangen (also auch die zu speichernde Zahl neu eingeben) - Schicker wäre es, wenn bei einem solchen Fehler nur die Positionseingabe wiederholt werden müsste. Gibt es da einen Kniff oder ist in diesem Design ohnehin der Wurm drin?

Freue mich wie immer über Hinweise und Verbesserungsvorschläge jeder Art...

import java.util.Scanner;
import java.io.IOException;

public class Main {

    public static void main(String[] args) {

        int[] intArray = new int[10];
        int i;
        int pos;

        Scanner scanner = new Scanner(System.in);

        while(true){
            System.out.print("\nZustand des Arrays: ");
            for(int x : intArray){
                System.out.print(x +" ");
            }
            try{System.out.println("\nBitte geben sie eine Zahl ein. 0 beendet das Programm.");
                i = scanner.nextInt();
                if(i == 0){
                    break;
                }
                
            }
            catch(java.util.InputMismatchException e){
                System.out.println("Ihre Eingabe war keine Zahl. Bitte versuchen Sie es erneut.");
                scanner.nextLine();
                continue;
            }

            try{
                System.out.println("Bitte geben sie die Position ein, unter der die Zahl " +i +" gespeichert werden soll.");
                pos = scanner.nextInt();
                intArray[pos] = i;
            }
            catch(java.util.InputMismatchException e){
                System.out.println("Ihre Eingabe war keine Position. Bitte versuchen Sie es erneut.");
                scanner.nextLine();
            }
            catch(java.lang.ArrayIndexOutOfBoundsException e){
                System.out.println("Die Position die Sie eingegeben haben überschreitet die Grenzen des Arrays. Bitte versuchen Sie es erneut.");
            }
        }

    }
}

 

[SlaterB]

die doppelte Eingabe verhinderst du offensichtlich, indem du für jede Frage eine eigene while-Schleife verwendest,
damit verhindest du auch das komplizierte continue; im ersten catch-Block

das bringt einem logischerweise dazu, eine eigenständige Methode
> public int readInt(String question, String errormessage)
zu definieren, Aufruf
> int i = readInt(..,..);
> int pos = readInt(..,..);

so fällt der Großteil doppelten Codes weg,
wegen Array musst du aber eher doch noch eine (kleinere) while-Schleife basteln,
ArrayIndexOutOfBoundsException fangen bietet sich da weniger an, wieso vergleichst du nicht auf >0, < Länge des Arrays?

falls solche Bereichsabfragen öfter kommen, kann man wieder an eine Methode
> public int readInt(String question, String errorMessage, int min, int max, String rangeErrorMessage)
denken

 

[nossek]

die doppelte Eingabe verhinderst du offensichtlich, indem du für jede Frage eine eigene while-Schleife verwendest,

Dann brauche ich also drei while schleifen, so in der Art?!:

while(solage das programm läuft){
      while(bis die zahl erfolgreich eingegeben wurde){...}
      while(bis die position erfolgreich eingegeben wurde){...}
      {zahl wird ins array geschrieben}
}

wieso vergleichst du nicht auf >0, < Länge des Arrays?

Ja, weil ich Ausnahmebehandlung üben wollte. Ist es grundsätzlich besser, wenn möglich eine eigene Parameterprüfung einzubauen, damit die Exception garnicht erst auftritt?

Werde jedenfalls versuchen, was Du vorgeschlagen hast, danke!

 

[SlaterB]

mal Exception, mal besser if-Test, das ist so und so,
InputMismatchException macht Sinn, da wäre es aufwändig, wenn auch nicht unmöglich, vorher isParsableAsInt() auszuführen,
quasi das ganze zweimal zu parsen

es gibt allerdings ein paar grundsätzliche RuntimeExceptions, zu denen ich bisher glaube ich noch kein sinnvolles catch gesehen habe,
NullPointerException, ArrayIndexOutOfBoundsException usw.,
wenn man vermutet dass irgendwas null sein kann oder Grenzen unterboten werden, dann lieber mit if testen

wobei selbst dann die Exception denkbar ist, z.B. wenn ein catch 10 ifs ersetzt

 

[nossek]

So, habe versucht Deine Vorschläge umzusetzen, funktionieren tut es anscheinend.

Was ich mich frage:

- Sollte man an readPos(...) statt "maxPos" lieber den Bezeichner des Arrays übergeben?

- ist dieses Konstrukt "while(true)" und dann mit break bzw. return rausspringen wenn alles okay ist denn guter Stil? Es erscheint ja erstmal recht praktisch und klar.

- (vlt. eine blöde Frage): sollte man konsequenterweise noch eine statische methode putIntIntoArray(.....) schreiben?

- Hat sonst jemand irgendwelche Verbesserungsvorschläge?!?

import java.util.Scanner;
import java.io.IOException;

public class Main {

    public static int readInt(String question, String errorMessage){
        Scanner scanner = new Scanner(System.in);
        while(true){
            try{
                System.out.println(question);
                return scanner.nextInt();
            }
            catch(java.util.InputMismatchException e){
                System.out.println(errorMessage);
                scanner.nextLine();
            }
        }
    }

    public static int readPos(String question, String errorMessage, int maxPos, String rangeErrorMessage){
        int result;
        while(true){
            result = readInt(question, errorMessage);
            if (result <= maxPos & result >= 0){
                return result;
            }
            else{
                System.out.println(rangeErrorMessage);
            }
        }
    }

    public static void printArray(int[] array){
        System.out.print("\nZustand des Arrays: ");
            for(int x : array){
                System.out.print(x +" ");
        }
    }

    public static void main(String[] args) {
        int[] intArray = new int[10];
        int i;
        int pos;

        while(true){
            printArray(intArray);

            i = readInt("\nBitte geben Sie eine Zahl ein. 0 beendet das Programm", "Das war keine Zahl.");

            if(i==0){
                break;
            }

            pos = readPos("Bitte geben sie eine Position ein.", "Das war keine Position.",
                  intArray.length-1, "Die Position ueberschreitet die Arraygrenze.");
            
            intArray[pos] = i;

        }
    }

}

 

[SlaterB]

ob Array oder nicht musst du schon selbst entscheiden, vielleicht nur auf die -1 verzichten und dann < prüfen statt <=,

while-true-break geht schon, vielleicht return statt break

putIntIntoArray halte ich für nicht sinnvoll, kein doppelte Code zu verhindern

 

[Der Müde Joe]

Persönlich finde ich das Parsen des Inputs schöner:

Scanner scanner = new Scanner(System.in);
while(true) {
	String line = scanner.nextLine();
	try {
		Integer i = Integer.decode(line);
		System.out.println("well done: " + i.intValue());
		break; //return i;
	} catch(NumberFormatException e) {
		System.out.println("Not a number...try again");
	}
}

 

[SlaterB]

ach ja, und gegebenenfalls nur einen Scanner im Programm, z.B. als statische Variable wie System.in auch eine ist,

hat Vor- und Nachteile, nämlich dass etwa bei einer Eingabe "4 5" der zweite Wert noch da ist, was auch Probleme machen kann, oder überlesen wird,
eine Zeile wird immer komplett an einem der Scanner übergeben

 

[nossek]

Müder Joe, ich finde Dein kleines Codebeispiel sehr hübsch und lehrreich.

Habe viel gelernt heute, besten Dank nochmals an alle!

Thema erledigt!

 

[nossek]

ach ja, und gegebenenfalls nur einen Scanner im Programm, z.B. als statische Variable wie System.in auch eine ist,

hat Vor- und Nachteile, nämlich dass etwa bei einer Eingabe "4 5" der zweite Wert noch da ist, was auch Probleme machen kann, oder überlesen wird,
eine Zeile wird immer komplett an einem der Scanner übergeben

ich habe das erfolgreich vrsucht und glaube zu verstehen!