IllegalArgumentException

[JavaCat++]

Hi Leute,

ich muss ein Bankenverwaltungssystem schreiben und stehe nun vor einem kleinem Problem:
Die Kunden sollen Ihre Kontonummer eingeben. Jedoch sollen Fehler wie 1234567R89 natürlich kontrolliert abgefangen werden. Jetzt ist InputMismatchException ja eine RuntimeExcpetion, die ja eigentlich nicht aufgefangen werden sollen. Ist es trotzdem eine gute Lösung, so etwas zu schreiben:

import java.util.InputMismatchException;
import java.util.Scanner;

public class Test {

public static void main(String[] args) {
try {

Scanner sc = new Scanner(System.in);
int kontonummer = sc.nextInt();

}

catch (InputMismatchException e) {
}

}

LG

 

[JStein52]

Ja klar ist das gut so. Eine Alternative Lösung wäre du liest das mit sc.nextLine(); zunächst mal als String ein und überprüfst erst mal selber ob nur gültige Zeichen, sprich Ziffern, darin enthalten sind. Aber ich kann bei der Exception-Lösung nichts nachteiliges erkennen

 

[mrBrown]

Nein, catch leer zu lassen, ist die allerschlechteste Lösung
Jede gefangene Exception sollte man auch sinnvoll behandeln, in diesem Fall zB mit (sinnvoller!) Fehlermeldung und nochmaliger Eingabe. Anstatt die Exception zu nutzen, kann man auch den String einlesen, prüfen und dann erst umwandeln, so geht zB -42 als Kontonummer durch.


Kontonummer als int ist aber auch schon eine schlechte Lösung...

 

[JavaCat++]

Cool Danke! Ich habe mich nun für folgende Lösung entschieden:

public static void main(String[] args) {

String regex = "(1|2|3|4|5|6|7|8|9)*";

Scanner sc = new Scanner(System.in);

String kontonummer = sc.next();

if (kontonummer.matches(regex)) {
// weiter mit Verarbeitung

}
else {

System.out.println("Fehler!");
// Kunde wurd aus Menü geworfen


}

 

[krgewb]

Können nun Kontonummern Nullen enthalten oder nicht?

 

[JavaCat++]

Stimmt, hast recht. ^^

Ich wollte dann noch mit length() prüfen, ob die Kontonummer die entsprechende Anzahl an Stellen hat.

 

[mrBrown]

Stimmt, hast recht. ^^

Ich wollte dann noch mit length() prüfen, ob die Kontonummer die entsprechende Anzahl an Stellen hat.

Du kannst auch im RegEx die Anzahl der Stellen mit angeben

 

[JStein52]

Ist Kontonummer nicht veraltet ? IBAN ist doch angesagt.

 

[thecain]

Und führende Nullen? Int ist ein schlechter Kontonummer Datentyp

 

[mrBrown]

Und führende Nullen?

Die sind doch sowieso optional, nen Unterschied zwischen 1, 01 und 0001 gibts nicht (zumindest bei Kontonummern, bei Strings natürlich schon ;P)

Und führende Nullen? Int ist ein schlechter Kontonummer Datentyp

Wie schon gesagt, allerdings ist ja auch nicht ersichtlich ob es das immer noch ist

 

[JavaCat++]

Wenn ich richtig nachgelesen habe, kann man den regex auch so schreiben, dass immer 12 Zahlen eingegeben werden müssen, dann bräuchte ich kein lenght() und man kann ihm mitgeben, dass die erste Zahl keine Null sein darf. ^^ Und mit der IBAN: Klar ist die Kontonummer veraltet, aber so ist halt die Aufgabe . Für die IBAN bräuchte man schließlich auch noch den Prüfziffer-Algorithmus...der kommt vielleicht später.

 

[thecain]

Die sind doch sowieso optional, nen Unterschied zwischen 1, 01 und 0001 gibts nicht (zumindest bei Kontonummern, bei Strings natürlich schon ;P)

Theoretisch ja. Wenn aber im Folgesystem die Kontonummer als String hinterlegt ist und dann ein Match mit .equals() gemacht wird, hat man den Salat.. Alles schon passiert.