Input/Output JOptionPane Fehlermeldung Catchen

[PStigerID]

Guten Abend liebes Forum,

Erstmal, vorweg schöne Pfingsten!

Momentan ist dieser code in meinem Constructor:

        String password = "";
        try { password = (String)JOptionPane.showInputDialog(this, "Please enter your passowrd:", "Identification  Required", JOptionPane.WARNING_MESSAGE); }
        catch(NullPointerException ex) {
            JOptionPane.showMessageDialog(this, "ERROR 401","401 Authorization Required",JOptionPane.ERROR_MESSAGE);
            System.err.println(ex+"Window closed!");
            password = (String)JOptionPane.showInputDialog(this, "Please enter your passowrd:", "Identification  Required", JOptionPane.WARNING_MESSAGE);
        }

Dies soll, wenn das Input Fenster geschlossen wird (x oder escape), die 401 message displayen, und den User zur erneuten Eingabe zwingen.

Geht leider nicht ???:L

Könnt ihr mir erklären wieso es nicht geht und eine Lösung vorschlagen?

Danke

PStiger

 

[Clayn]

Wo soll denn eine NullPointerException auftreten?

mach statt dem try catch block einfach ne If abfrage ob password==null ist.
Die Exception tritt ja nur auf wenn JOptionPane null wäre was ja nicht sein kann bzw ist.

Außerdem wozu der cast zu String? Laut meiner API gibt .showInputDialog nen String zurück

MFG
Clayn

 

[KingOfExceptions]

Wie wärs mit

String password=JOptionPane.showInputDialog(null, "Password:");

while(password==null)
{
     password=JOptionPane.showInputDialog(null,"Password:");
}

Habs jetzt zwar nicht geteste müsste aber gehen.

 

[PStigerID]

Wie wärs mit

String password=JOptionPane.showInputDialog(null, "Password:");

while(password==null)
{
     password=JOptionPane.showInputDialog(null,"Password:");
}

Habs jetzt zwar nicht geteste müsste aber gehen.

Vielen Dank - hat super geklappt!!!!!

Außerdem wozu der cast zu String? Laut meiner API gibt .showInputDialog nen String zurück

Ich bin dieser Erklärung gefolgt :rtfm:Getting the User's Input from a Dialog (Englisch)

 

[Clayn]

In dem Fall müsstest du auch nach String casten aber dein InputDialog ist ein anderer.
Sie haben dort den benutzt, der dir quasi jede Einstellung lässt und du hast einen einfacheren genommen der einen String zurück gibt

 

[U2nt]

String password=JOptionPane.showInputDialog(null, "Password:");
 
while(password==null)
{
     password=JOptionPane.showInputDialog(null,"Password:");
}

da fände ich

String password = null;

do {
  password = JOptionPane.showInputDialog(null, "Password: ");
} while(password == null)

schöner. Aber ist ja nicht gerade extrem entscheidend.