Wie den Inhalt vom Spinner auf Integer überprüfen?

[_user_q]

Tag,

ich habe das Problem, dass wenn ich spinner.getValue() ausgeben lassen möchte und ich vorher absichtlich keine Zahl eingetragen habe, dass dann die NumberFormatException-Fehlermeldung kommt, obwohl ich folgende Dinge ausprobiert habe:

1:

int test = Integer.parseInt(String.valueOf(controller.options_Spinner_Speed.getValue()));
try {
            Integer.parseInt(String.valueOf(test));
            System.out.println(test);

        } catch (NumberFormatException nfe) {
            System.out.println("Inhalt ist keine Zahl!");
        }

^Diese Methode funktioniert jedoch nicht, da SpinnerFactorys wohl falsche Eingaben nicht so gut abfangen können und weil diese Überprüfung anscheinend zu spät kommt.

2:

SpinnerValueFactory<Integer> speedFactory = controller.options_Spinner_Speed.getValueFactory();
        speedFactory.setConverter(new IntegerStringConverter());

        System.out.println(speedFactory.getValue());

^Ich weiß noch nicht so ganz damit umzugehen, denn die Fehlermeldung kommt hier immer noch und hilfreiche Lösungen finde ich einfach nicht im Internet.
Ich müsste über 12 Spinner auf Integer überprüfen...

Ich danke im Voraus!

 

[Robertop]

Die Meldung kommt beim ersten Beispiel bestimmt direkt aus der ersten Zeile, weil du da ja auch schon versuchst, den Inhalt des Spinners zu einer Int zu parsen. Wenn du die Zeile in den try Block reinschiebst, dann sollte die Exception nicht mehr kommen und stattdessen der String "Inhalt ist keine Zahl!" ausgegeben werden.

Oder du änderst die erste Zeile so, dass test nur den Wert des Spinners bekommt. Ich bin nur nicht Sicher, was für ein Ausgabetyp getValue hat:

Object test = controller.options_Spinner_Speed.getValue();

 

[httpdigest]

Ja, wie @Robertop schon schrieb, machst du effektiv folgendes:

// hier erstmal den String in ein int parsen (könnte fehlschlagen!!!):
int test = Integer.parsInt(einStringDerMoeglicherweiseEinIntegerIst);
try {
  // jetzt erstmal den int wieder in einen String wandeln und das ganze wieder zurück in ein int parsen
  Integer.parseInt(String.valueOf(test));
  ...
}

Das ist natürlich Käse.

 

[_user_q]

Die Meldung kommt beim ersten Beispiel bestimmt direkt aus der ersten Zeile, weil du da ja auch schon versuchst, den Inhalt des Spinners zu einer Int zu parsen. Wenn du die Zeile in den try Block reinschiebst, dann sollte die Exception nicht mehr kommen und stattdessen der String "Inhalt ist keine Zahl!" ausgegeben werden.

Oder du änderst die erste Zeile so, dass test nur den Wert des Spinners bekommt. Ich bin nur nicht Sicher, was für ein Ausgabetyp getValue hat:

Object test = controller.options_Spinner_Speed.getValue();

Beides ausprobiert, bei beidem wird Muss eine Zahl enthalten. nicht ausgegeben, sondern es kommt wieder die Fehlermeldung Exception in thread "JavaFX Application Thread" java.lang.NumberFormatException :/

 

[Robertop]

Hmm. Ist es möglich, dass der Fehler von woanders kommt? In der Konsole beim Stackstrace kann man sehen, in welcher Klasse und in welcher Zeile der Fehler aufgetreten ist. Schau mal nach, in welcher Zeile genau der Fehler auftritt.

 

[Neumi5694]

Du kannst auch noch prüfen, ob der Spinner überhaupt das richtige Datenmodell hat.

    //Abfrage ob es sich um das richtige Datenmodell handelt.
    if (spinner.getModel() instanceof SpinnerNumberModel numbermodel) {
     //vielleicht noch Eigenschaften des Models überprüfen?
     var obj = numbermodel.getValue();
    }

  //Eine Methode ähnlich dieser hab ich mir mal für UIs geschrieben
  public static Integer toInteger(Object o, Integer defaultValue) {
    if (o == null) {
      return defaultValue;
    } else if (o instanceof Integer integer) {
      return integer;
    } else if (o instanceof Number number) {
      return number.intValue();
    } else {
       //eventuell noch in String umwandeln und zu parsen versuchen
       return defaultValue;
    }
  }

 

[_user_q]

Du kannst auch noch prüfen, ob der Spinner überhaupt das richtige Datenmodell hat.

    //Abfrage ob es sich um das richtige Datenmodell handelt.
    if (spinner.getModel() instanceof SpinnerNumberModel numbermodel) {
     //vielleicht noch Eigenschaften des Models überprüfen?
     var obj = numbermodel.getValue();
    }

  //Eine Methode ähnlich dieser hab ich mir mal für UIs geschrieben
  public static Integer toInteger(Object o, Integer defaultValue) {
    if (o == null) {
      return defaultValue;
    } else if (o instanceof Integer integer) {
      return integer;
    } else if (o instanceof Number number) {
      return number.intValue();
    } else {
       //eventuell noch in String umwandeln und zu parsen versuchen
       return defaultValue;
    }
  }

spinner.getModel()

gibt es hier schon nicht.

 

[_user_q]

Hmm. Ist es möglich, dass der Fehler von woanders kommt? In der Konsole beim Stackstrace kann man sehen, in welcher Klasse und in welcher Zeile der Fehler aufgetreten ist. Schau mal nach, in welcher Zeile genau der Fehler auftritt.

Exception in thread "JavaFX Application Thread" java.lang.NumberFormatException: For input string: "0d"
    at java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:68)
    at java.base/java.lang.Integer.parseInt(Integer.java:652)
    at java.base/java.lang.Integer.valueOf(Integer.java:983)
    at javafx.base/javafx.util.converter.IntegerStringConverter.fromString(IntegerStringConverter.java:49)
    at javafx.base/javafx.util.converter.IntegerStringConverter.fromString(IntegerStringConverter.java:35)
    at javafx.controls/javafx.scene.control.Spinner.commitValue(Spinner.java:460)
    at javafx.controls/javafx.scene.control.Spinner.lambda$new$3(Spinner.java:168)
    at javafx.base/com.sun.javafx.binding.ExpressionHelper$Generic.fireValueChangedEvent(ExpressionHelper.java:348)
    at javafx.base/com.sun.javafx.binding.ExpressionHelper.fireValueChangedEvent(ExpressionHelper.java:80)
    at javafx.base/javafx.beans.property.ReadOnlyBooleanPropertyBase.fireValueChangedEvent(ReadOnlyBooleanPropertyBase.java:72)
    at javafx.graphics/javafx.scene.Node$FocusedProperty.notifyListeners(Node.java:8155)
    at javafx.graphics/javafx.scene.Node.setFocused(Node.java:8208)
    at javafx.graphics/javafx.scene.Scene$KeyHandler.setWindowFocused(Scene.java:4032)
    at javafx.graphics/javafx.scene.Scene$KeyHandler.lambda$new$0(Scene.java:4054)
    at javafx.base/com.sun.javafx.binding.ExpressionHelper$Generic.fireValueChangedEvent(ExpressionHelper.java:348)
    at javafx.base/com.sun.javafx.binding.ExpressionHelper.fireValueChangedEvent(ExpressionHelper.java:80)
    at javafx.base/javafx.beans.property.ReadOnlyBooleanPropertyBase.fireValueChangedEvent(ReadOnlyBooleanPropertyBase.java:72)
    at javafx.base/javafx.beans.property.ReadOnlyBooleanWrapper.fireValueChangedEvent(ReadOnlyBooleanWrapper.java:103)
    at javafx.base/javafx.beans.property.BooleanPropertyBase.markInvalid(BooleanPropertyBase.java:111)
    at javafx.base/javafx.beans.property.BooleanPropertyBase.set(BooleanPropertyBase.java:145)
    at javafx.graphics/javafx.stage.Window.setFocused(Window.java:674)
    at javafx.graphics/javafx.stage.Window$1.setFocused(Window.java:149)
    at javafx.graphics/com.sun.javafx.stage.WindowHelper.setFocused(WindowHelper.java:112)
    at javafx.graphics/com.sun.javafx.stage.WindowPeerListener.changedFocused(WindowPeerListener.java:64)
    at javafx.graphics/com.sun.javafx.tk.quantum.GlassWindowEventHandler.run(GlassWindowEventHandler.java:126)
    at javafx.graphics/com.sun.javafx.tk.quantum.GlassWindowEventHandler.run(GlassWindowEventHandler.java:40)
    at java.base/java.security.AccessController.doPrivileged(AccessController.java:391)
    at javafx.graphics/com.sun.javafx.tk.quantum.GlassWindowEventHandler.lambda$handleWindowEvent$4(GlassWindowEventHandler.java:176)
    at javafx.graphics/com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(QuantumToolkit.java:412)
    at javafx.graphics/com.sun.javafx.tk.quantum.GlassWindowEventHandler.handleWindowEvent(GlassWindowEventHandler.java:174)
    at javafx.graphics/com.sun.glass.ui.Window.handleWindowEvent(Window.java:1351)
    at javafx.graphics/com.sun.glass.ui.Window.notifyFocus(Window.java:1330)
    at javafx.graphics/com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
    at javafx.graphics/com.sun.glass.ui.win.WinApplication.lambda$runLoop$3(WinApplication.java:174)
    at java.base/java.lang.Thread.run(Thread.java:832)

Gar keine Zeile von meinem Code wird angegeben. :/

 

[Neumi5694]

spinner.getModel()

gibt es hier schon nicht.

Ah, JavaFX. Meine Lösung ist für Swing.

Zu deinem letzten Post : Kannst du mal zeigen, wo du den Spinner initialisierst/konfigurierst und ihm einen Wert zuweist?
"0d" klingt doch sehr verdächtig.

 

[_user_q]

Ah, JavaFX. Meine Lösung ist für Swing.

Zu deinem letzten Post : Kannst du mal zeigen, wo du den Spinner initialisierst/konfigurierst und ihm einen Wert zuweist?
"0d" klingt doch sehr verdächtig.

"0d" habe ich absichtlich eingegeben, da der ja eigentlich nur Integer annehmen soll. Ich möchte für eine Falscheingabe quasi eine andere Errornachricht, wenn man aus Versehen oder absichtlich einen nicht-Integer eingibt.

Hier ist der Spinner sonst untergebracht:
[CODE lang="java" title="Controller.java"]
@Override
public void initialize(URL url, ResourceBundle rb) {
SpinnerValueFactory<Integer> click = new SpinnerValueFactory.IntegerSpinnerValueFactory(0, 60, 0);
this.test_Spinner.setValueFactory(click);
}
[/CODE]

 

[Jw456]

"0d" dann ist es ein double was du zu einen int machen willst also wenn es ein primitiver DatenTyp ist einfach zu (int) Casten.

controller.options_Spinner_Speed.getValue()  // welcher datentyp wir hier zurückgegen?
String.valueOf(controller.options_Spinner_Speed.getValue()) // jetzt ist es ein String gib ihn dir mal aus

 

[Neumi5694]

"0d" habe ich absichtlich eingegeben, da der ja eigentlich nur Integer annehmen soll. Ich möchte für eine Falscheingabe quasi eine andere Errornachricht, wenn man aus Versehen oder absichtlich einen nicht-Integer eingibt.

Hier ist der Spinner sonst untergebracht:
[CODE lang="java" title="Controller.java"]
@Override
public void initialize(URL url, ResourceBundle rb) {
SpinnerValueFactory<Integer> click = new SpinnerValueFactory.IntegerSpinnerValueFactory(0, 60, 0);
this.test_Spinner.setValueFactory(click);
}
[/CODE]

Wo hat du das angegeben? Kannst du die Zeile posten? Oder war das eine Nutzereingabe? Denn im Spinner oder was auch immer diese Methode auswerten soll steht kein Double 0, sondern ein String "0d".
Und das kann man natürlich nicht zu parsen, vorher muss das "d" weg. So was schreibt man nur direkt im Quellcode, damit der Compiler weiß, was man double meint.

 

[_user_q]

Wo hat du das angegeben? Kannst du die Zeile posten? Oder war das eine Nutzereingabe? Denn im Spinner oder was auch immer diese Methode auswerten soll steht kein Double 0, sondern ein String "0d".
Und das kann man natürlich nicht zu parsen, vorher muss das "d" weg. So was schreibt man nur direkt im Quellcode, damit der Compiler weiß, was man double meint.

Ich habe in den Spinner, der auf dem Fenster zu finden ist, absichtlich dieses "d" eingefügt (es war einfach ein random Buchstabe). Im Code selber wird aber immer Integer zu finden sein, wobei das nicht zu "0d" - wegen dem "d" - zutreffen kann. (Zu finden sein im Sinne von, dass der Spinner in initialize() auf ein Integer eingestellt ist.)
Also war "0d" eine Nutzereingabe, wobei der Spinner einen Integer möchte. (Der möchte den essen ... Spaß bei Seite)

 

[Neumi5694]

Ich habe in den Spinner, der auf dem Fenster zu finden ist, absichtlich dieses "d" eingefügt (es war einfach ein random Buchstabe). Im Code selber wird aber immer Integer zu finden sein, wobei das nicht zu "0d" - wegen dem "d" - zutreffen kann. (Zu finden sein im Sinne von, dass der Spinner in initialize() auf ein Integer eingestellt ist.)
Also war "0d" eine Nutzereingabe, wobei der Spinner einen Integer möchte. (Der möchte den essen ... Spaß bei Seite)

Dann ist die Exception ja richtig - wenn auch Überflüssig. In Swing kümmert sich der Validator drum und macht feherhafte Eingaben rückgängig ohne groß rumzumeckern.

 

[_user_q]

Ich habe in den Spinner, der auf dem Fenster zu finden ist, absichtlich dieses "d" eingefügt (es war einfach ein random Buchstabe). Im Code selber wird aber immer Integer zu finden sein, wobei das nicht zu "0d" - wegen dem "d" - zutreffen kann.
Also war "0d" eine Nutzereingabe, wobei der Spinner einen Integer möchte. (Der möchte den essen ... Spaß bei Seite)

Dann ist die Exception ja richtig - wenn auch Überflüssig. In Swing kümmert sich der Validator drum und macht feherhafte Eingaben rückgängig ohne groß rumzumeckern.

Aber ich möchte einen eigenen Fehlertext und keine zugebombte Console 🤔

 

[Neumi5694]

Die Konsole siehst du in einer UI-Anwendung ja nicht mehr. Ist aber schon seltsam, dass da so viel ausgegeben wird, ist wohl was FX eigenes.
Ich hab per Google ein paar Beispiele gefunden, wie man die Eingabe in einem FX-Textfield validiert, beim Spinner dürfte das nicht viel anders laufen. Die Meldung soll doch sofort bei Eingabe ausgegeben werden, oder?
Falls erst später .. dann musst du Strings grundsätzlich zulassen und später selbst prüfen, was drin steht.

Hier geht's um genau dein Problem

Insert only numbers in Spinner Control

I tested Spinner control in Java 8u40 import javafx.application.Application; import javafx.geometry.Insets; import javafx.scene.Scene; import javafx.scene.control.Label; import javafx.scene.control.

stackoverflow.com

So wie's ausschaut, muss man beim Formatter des Editors eingreifen.