Java Fehlermeldung ausgeben bei unerwarteter Scanner Eingabe

Java Fehlermeldung ausgeben bei unerwarteter Scanner Eingabe (18 Beiträge, 597 Mal gelesen)

Du bist nicht angemeldet. [ Login/Registrieren ]

Ich schreibe ein kleines Programm das die Gültigkeit des eingegebenen Datums überprüfen soll. An sich bin ich fertig, aber:

1) Wenn man etwas anderes als eine Zahl eingibt kommt eine Absturzmeldung von Integer.parseInt(). Wie sorge ich dafür, dass das Programm schreibt "Bitte nur Zahlen eingeben!" und einem dann noch x Chancen gibt?
2) Wie sorge ich dafür, dass es eine individuelle Fehlermeldung gibt wenn man eine Zahl eingibt die außerhalb des möglichen Bereichs liegt, Tage über 31 und unter 1 z.B.
3) Wie überspringe ich mit if Anweisungen Teile des Programms?
Also wenn x eintrifft bzw nicht, soll es auf Zeile y springen und alles dazwischen ignorieren.

So hab ich das realisiert:

Scanner scanner1 = new Scanner(System.in);

System.out.println("Bitte Tag eingeben.");
String TagS = scanner1.nextLine();
Tag = Integer.parseInt(TagS);

System.out.println("Bitte Monat eingeben.");
String MonatS = scanner1.nextLine();
Monat = Integer.parseInt(MonatS);

1.) try, catch
2.) if (tag > 31 || tag < 1) { fehler...}
3.) if (xyz) { anweisungen wenn wahr } danach gehts weiter

Danke, jetzt funktioniert alles wie ich will.
http://pastebin.com/Q6Y03EKJ

Kompilierfehler oder wo liegt das Problem?

Mir wäre jetzt kein Fehler aufgefallen...

Ich wuerde mal Folgendes ansehen: http://pastebin.com/MRsdfB95
Spart unnoetige Codeduplizierung...

Und ein kleiner Tipp:
Variablen beginnen mit einem kleinbuchstaben - und sind im Idealfall englisch.

edit:

Alternativ in deiner Loesung im catch-Block noch ein scanner.next(); einfuegen - das verhindert die Endlosschleife

lg

25.03.2012, 00:18 Uhr - Editiert von bdb, alte Version: hier

http://pastebin.com/MRsdfB95

Das ist toll und ich habs zum laufen bekommen, ich weiß aber gar nicht wo ich anfangen soll Fragen zu stellen was da passiert.

beim ersten steht String TagS im Block 2 steht nur Tag ... ?

Tipps:

- es hat nicht jeder Monat 31 Tage, eine richtige Überprüfung anhand des vollständig eingegebenen Datums kann man z.B. mit parse() in der Klasse java.text.SimpleDateFormat durchführen.

- statt "Tag > 0 && Tag < 32" sollte man "Tag >= 1 && Tag <= 31" schreiben wenn das die tatsächlichen Grenzen sind.

ACTA wird Menschen töten | wikileaks.geizhals.org

"Wenn Behörden solche Trojaner in Auftrag geben und sie benutzen, handelt es sich um eine neue Form der Staatskriminalität" (Heribert Prantl, sueddeutsche.de)

"I haven't had a lifelong campaign against authority. Legitimate authority is important. All human systems require authority, but authority must be granted as a result of the informed consent of the governed. Presently, the consent, if there is any, is not informed, and therefore it's not legitimate." (J. Assange)

- statt "Tag > 0 && Tag < 32" sollte man "Tag >= 1 && Tag <= 31" schreiben wenn das die tatsächlichen Grenzen sind.

Warum? Braucht das < CPU cycles? Liest sich gleich gut...
(Real interest, not trolling.)

Weil:
a)
dein Programm wächst, und wächst, und wächst.
Irgendwann kommst du drauf, ha, ich hab da bei der Begrenzung was falsch, oder ich möchte da noch was dazutun, oder es wurde der 32. Kalendertag eingeführt.

Jetzt suchst du im Programm wo du überall so Begrenzungen drin hast. Du wirst mit 31 suchen, nicht mit 32

b)
Wenn es jetzt nicht grad um Kalendertage geht sondern um was anderes, ist es oft gleich einleuchtender wenn du später die richtige Begrenzung liest, und nicht die werte davor/danach

c)
wenn du Tag aus irgendeinem Grund mal durch einen nicht integer Wert ersetzt, würdest du 0,000000000000000000001 und 31,999999999999999999 auch zulassen, was spätestens nach deiner ersten Rundung (oder nach dem ersten impliziten cast auf int, wobei ich nicht weiß obs sowas in Java gibt? komm aus einer anderen Sparte...) wieder 0 und 32 ergibt, was ja nicht gültig sein sollte...

Edit:
c) ist wohl am wichtigsten, a) und b) begegnen mir aber seeeeehr oft

26.03.2012, 16:09 Uhr - Editiert von zeddicus, alte Version: hier

Jetzt suchst du im Programm wo du überall so Begrenzungen drin hast. Du wirst mit 31 suchen, nicht mit 32

Wenn es da echt mehrere Stellen gibt, klingt das wie ein Verstoß gegen das SRP.

Wenn es jetzt nicht grad um Kalendertage geht sondern um was anderes, ist es oft gleich einleuchtender wenn du später die richtige Begrenzung liest, und nicht die werte davor/danach

Aber ein Vergleich mit 0 ist oft schneller als mit einer Zahl...

wenn du Tag aus irgendeinem Grund mal durch einen nicht integer Wert ersetzt, würdest du 0,000000000000000000001 und 31,999999999999999999 auch zulassen,

Nein. Dann sollte man wirklich auf >= wechseln [ evtl. mit passendem Epsilon für Ungenauigkeiten, aber das ist eine andere Geschichte

]

Nein. Dann sollte man wirklich auf >= wechseln

Das hab ich doch gesagt!?

Wenn es da echt mehrere Stellen gibt, klingt das wie ein Verstoß gegen das
SRP.

Nö, denn nicht jeder Eingabewert wird durch die selbe Routine geprüft

Aber ein Vergleich mit 0 ist oft schneller als mit einer Zahl...

Ein Vergleich mit NULL ev., aber bei 0 sollts eigentlich nichts bringen. Kann aber sein daß ich mich täusche. Ansonsten müßt ich ja vorher immer schaun obs überhaupt 0 ist damit ich den beschleunigten Vergleich verwenden kann, und würd für alle anderen Zahlen Geschwindigkeit einbüßen. Wenn dann sollte der Unterschied auch marginal sein. Außerdem war das mit 0 jetzt nur das genannte Beispiel, kann ja auch 4859279 sein.

Dieses Forum ist eine frei zugängliche Diskussionsplattform.
Der Betreiber übernimmt keine Verantwortung für den Inhalt der Beiträge und behält sich das Recht vor, Beiträge mit rechtswidrigem oder anstößigem Inhalt zu löschen.
Datenschutzerklärung