Input/Output zahl aus showInputDialog()

[Reggie()]

also ich weiss, dass ich mit int eingabe = Integer.parseInt(JOptionPane.showInputDialog("Zahl eingeben: "); eine Zahl bekomme, aber natürlich kann es auch zum Fehler kommen, wenn keine Zahl eingegeben wird. Da dies offensichtlich ein ganz häufiges Problem ist, wundert es mich, dass die Methode nur einen String zurückgibt. Es wäre doch perfekt, wenn nur eine Zahl angenommen werden würde und bei falscher Eingabe bei klick auf OK einfach nichts passieren würde.
ich würde das wie folgt lösen, bin mir aber fast sicher, dass es noch elegantere Lösungen gibt. da ich Anfänger bin und das Problem - wie bereits gesagt - ein häufiges sein dürfte.

int zahl = 0;
while (true) {
    try {
      zahl = Integer.parseInt(JOptionPane.showInputDialog("Zahl eingeben: "));
      break;
    } catch (Exception e) {
      JOptionPane.showMessageDialog("Eingabe war keine Zahl!");
    }  
}

 

[krgewb]

JOptionPane.showMessageDialog("Eingabe war keine Zahl!");

Die Zeile ist fehlerhaft. Es kann so gemacht werden:

JOptionPane.showMessageDialog(null, "Eingabe war keine Zahl!");

 

[Reggie()]

ja, war "aus dem Kopf" geschrieben und habs später dann auch gemerkt. Den Rest würdest du auch so machen oder kennst du eine elegantere Lösung?

 

[mrBrown]

Da dies offensichtlich ein ganz häufiges Problem ist, wundert es mich, dass die Methode nur einen String zurückgibt. Es wäre doch perfekt, wenn nur eine Zahl angenommen werden würde und bei falscher Eingabe bei klick auf OK einfach nichts passieren würde.

Naja, das ist halt keine Zahlen-eingeben-Dialog, sondern nur Texteingabe 😉

ich würde das wie folgt lösen, bin mir aber fast sicher, dass es noch elegantere Lösungen gibt. da ich Anfänger bin und das Problem - wie bereits gesagt - ein häufiges sein dürfte.

Im Allgemeinen ist das die Lösung 😉

Man könnte die while durch eine do-while mit "besserer" Bedingung ersetzten und statt dem try-catch die Eingabe erst auf Validität prüfen.

 

[Reggie()]

Man könnte die while durch eine do-while mit "besserer" Bedingung ersetzten und statt dem try-catch die Eingabe erst auf Validität prüfen.

genau darum geht es ja. wenn man eine reine Zahleneingabe will, wird es IMMER eine exception geben, wenn etwas anderes als eine Zahl eingegeben wird. kann man das ohne try-catch lösen?

 

[mrBrown]

genau darum geht es ja. wenn man eine reine Zahleneingabe will, wird es IMMER eine exception geben, wenn etwas anderes als eine Zahl eingegeben wird. kann man das ohne try-catch lösen?

Ja, für positive Integer zB prüfen, ob der String nur Zahlen enthält.

 

[Reggie()]

meinst du mit .isNumeric(String) ?

 

[krgewb]

Ja, für positive Integer zB prüfen, ob der String nur Zahlen enthält.

Du meinst Ziffern.
Aber wenn die Zahl größer als 2147483647 ist fliegt uns wieder alles um die Ohren.

 

[mrBrown]

Du meinst Ziffern.

Stimmt 😉

Aber wenn die Zahl größer als 2147483647 ist fliegt uns wieder alles um die Ohren.

Ja - die Exception muss man trotzdem fangen 😉 ich würde trotzdem den String vorher prüfen, das was ich anfangen kann, fang ich ab

meinst du mit .isNumeric(String) ?

Gibt es die in der Standard-Lib?

 

[thecain]

Gibt es die in der Standard-Lib?

Nicht das ich wüsste... mMn Apache Commons... z.b.

 

[JStein52]

Mach dir eine regular expression die den von dir gewünschten Wertebereich abdeckt und prüfe das mit myString.matches(...) und ob du dann die Exception immer noch abfangen willst ist deine Gewissensentscheidung

 

[Xyz1]

Alles gut, so wie es ist. Würde durchs Refactoring kommen.
Nur die mögliche potenzielle Exception könntest du spezifizieren/konkretisieren/einengen.

 

[Reggie()]

Mach dir eine regular expression die den von dir gewünschten Wertebereich abdeckt und prüfe das mit myString.matches(...) und ob du dann die Exception immer noch abfangen willst ist deine Gewissensentscheidung

kannst du mir dafür ein code-Beispiel geben? ich weiss nicht wirklich was du meinst.
Könnte ich nicht auch die Funktion showInputDialog überschreiben mit anderem Rückgabewert?
public static int showInputDialog(String txt)
und müsste ich dafür von der Klasse JOptionPane erben?

 

[JStein52]

Könnte ich nicht auch die Funktion showInputDialog überschreiben mit anderem Rückgabewert?
public static int showInputDialog(String txt)

Sowas könntest du natürlich andenken aber was gibst du dann zurück um eine fehlerhafte Eingabe zu signalisieren ?

 

[mrBrown]

Sowas könntest du natürlich andenken aber was gibst du dann zurück um eine fehlerhafte Eingabe zu signalisieren ?

Logischerweise eine Exception 😛

 

[JStein52]

Und ein Beispiel wäre :

 if(myString.matches("^[1-9]?[0-9]{1}$|^100$")) {
// es war eine Zahl von 1 .. 100
}

Die Epression ist aber ungetestet

 

[Xyz1]

Mal ein ganz einfacher Ansatz:

    public static void main(String[] args) {
        Number n = null;

        while (true) {
            String inp = JOptionPane.showInputDialog(null, "Gib was ein, was als Zahl interpretiert werden könnte:");
            if (inp == null) {
                continue;
            }
            Matcher m1 = Pattern.compile("\\-").matcher(inp);
            int c = 0;
            while (m1.find()) {
                c++;
            }
            Matcher m2 = Pattern.compile("\\d+").matcher(inp);
            String str = "";
            while (m2.find()) {
                str += m2.group();
            }
            if (str.isEmpty()) {
                continue;
            }
            try {
                int i = Integer.parseInt(str);
                if (c % 2 == 1) {
                    i = -i;
                }
                n = i;
            } catch (NumberFormatException e1) {
                try {
                    long l = Long.parseLong(str);
                    if (c % 2 == 1) {
                        l = -l;
                    }
                    n = l;
                } catch (NumberFormatException e2) {
                    BigInteger b = new BigInteger(str);
                    if (c % 2 == 1) {
                        b = b.negate();
                    }
                    n = b;
                }
            }
            break;
        }

        System.out.println("Du hast eingegeben: " + n);
    }

 

[krgewb]

bin mir aber fast sicher, dass es noch elegantere Lösungen gibt. da ich Anfänger bin und das Problem - wie bereits gesagt - ein häufiges sein dürfte.

Ich glaube du willst auf so etwas hinaus: Bei einer Webseite, die Bootstrap verwendet, kann man den Typ auf "number" setzen. Wenn der Benutzer auf seiner Tastatur auf einen Buchstabentaste drückt dann passiert einfach nichts. Nur wenn er Ziffern eingibt erscheinen Sie in dem Textfeld. In Chrome funktioniert dies sauber. Auf Firefox scheint es jedoch nicht zu funktionieren.

<input type="number" class="form-control" id="InputPersonalnummer" name="InputPersonalnummer" maxlength="9" min="1" max="999999999" step="1" placeholder="Geben Sie eine Personalnummer ein" required>

Ich weiß nicht, wie man das in Java macht. Fakt ist: Wenn du ein JTextField verwendest hast du viel mehr Möglichkeiten. Es lässt sich z.B. einem JTextField ein KeyListener hinzufügen. Du kannst dann sogar jedesmal darauf reagieren wenn der Benutzer ein Zeichen eingegeben hat und noch gar nicht auf OK geklickt hat.

 

[JStein52]

Ich weiß nicht, wie man das in Java macht.

Man macht das in Java z.B. so:

    public static void setNumericOnly(JTextField jTextField) {
        jTextField.addKeyListener(new KeyAdapter() {
            public void keyTyped(KeyEvent e) {
                char c = e.getKeyChar();
                if ((!Character.isDigit(c)
                        || (c == KeyEvent.VK_BACK_SPACE)
                        || (c == KeyEvent.VK_DELETE))) {
                    e.consume();
                }
            }
        });
    }

Aber ich könnte schwören dass er was von JOptionPane und showInputDialog geschrieben hat.

 

[Reggie()]

ich glaube mittlerweile, dass dieses "Problem" doch nicht so alltäglich ist, weil wohl kaum noch einer JOptionPane.showInputDialog() benutzt, sondern alles im Rahmen einer GUI mit actionListenern händelt, richtig?
Ich hatte das "Problem" ja auch nur, weil ich erstmal ein lauffähiges Proogramm schreiben wollte und dafür die einfachste Textform für Input/Output gewählt habe.

 

[JStein52]

Das Problem ist deshalb alltäglich weil jeder die Exception beim parseInt() abfängt und es deshalb keines ist.

 

[Reggie()]

also exakt so, wie ich es im Eingangspost (ohne den Schreibfehler bei showMessageDialog) geschrieben habe?

 

[mrBrown]

Ich glaube du willst auf so etwas hinaus: Bei einer Webseite, die Bootstrap verwendet, kann man den Typ auf "number" setzen. Wenn der Benutzer auf seiner Tastatur auf einen Buchstabentaste drückt dann passiert einfach nichts. Nur wenn er Ziffern eingibt erscheinen Sie in dem Textfeld. In Chrome funktioniert dies sauber. Auf Firefox scheint es jedoch nicht zu funktionieren.

Das ist HTML (5), Bootstrap ist nur ein CSS-Framework und hat damit nichts zu tun.
So langsam fangen zum Glück alle Browser an, das zu unterstützen, Firefox ist da nur etwas langsam...

 

[JStein52]

also exakt so, wie ich es im Eingangspost (ohne den Schreibfehler bei showMessageDialog) geschrieben habe?

Genau.

 

[Reggie()]

dann freue ich mich, dass ich Fortschritte gemacht habe und danke allen Helfern hier für ihre Hilfe und Geduld. #Themacanbeclosed

 

[Blender3D]

Vielleicht ist das FormattedTextfield eine Lösung für Dich?
https://docs.oracle.com/javase/tuto...ct/src/components/FormattedTextFieldDemo.java
😉