JavaFX Toggle_Button_Disable

[Adriano10]

   private void toggleButtonDisable(Button button, boolean disable) {
        button.disableProperty().bind(new SimpleBooleanProperty(disable));
    }

    @FXML
    private void onZoomIn() {
        incZoom();
        toggleButtonDisable(this.zoomIn, this.isDisableZoomIn());
        toggleButtonDisable(this.zoomOut, this.isDisableZoomOut());
    }

    @FXML
    private void omZoomOut() {
        this.decZoom();

        toggleButtonDisable(this.zoomOut, this.isDisableZoomOut());
        toggleButtonDisable(this.zoomIn, this.isDisableZoomIn());
    }

Einen schönen Tag zusammen,

Also ich habe eine Methode toggleButtonDisable geschrieben, die die Buttons deaktiviert und aktiviert.

Also wenn Max-Size erreicht ist, wird ZoomIn deaktviert. Sobald man dann ZoomOut anklickt, wird ZoomIn aktiviert.. Das gleiche passiert auch bei ZoomOut.

Das funktioniert ganz gut, aber finde ich bisschen unschön, dass diese Methode doppelt aufgerufen werden muss.

Kann jemand mir paar Tipps geben, sodass diese Methode toggleButtonDisable nur einmal aufgerufen wird?

Vielen Dank im Voraus

 

[KonradN]

Also Dein Code würde ich jetzt als relativ dubios bezeichnen.

Das mit den Properties dient dem binding. Und dann hast Du eine Property und die wird gesetzt. Immer eine neue SimpleBooleanProperty zu erzeugen, wenn Du etwas umsetzen willst, ist absolut unnötig!

Wenn Du den Wert nur setzen willst, dann nutz einfach direkt ein set, also etwas wie:
button.disableProperty().set(disable)

Wenn Du zwei Properties hast, die voneinander abhängen, dann kannst Du da entsprechende Bindings aufstellen. Also etwas wie:
zoomOut.disableProperty().bind(zoomIn.disableProperty().not());

Damit würde es ausreichen, nur noch zoomIn.disableProperty().set(...) aufzurufen und zoomOut verändert sich automatisch.

Du kannst dazu auch eine eigene SimpleBooleanProperty nutzen. Dann hättest Du als einmalige Initialisierung etwas wie:

// Instanzvariable:
SimpleBooleanProperty zoomProperty = new SimpleBooleanProperty(true);

// Initialisierung:
zoomIn.disableProperty().bind(zoomProperty);
zoomOut.disableProperty().bind(zoomProperty.not());

// Umsetzen Methode
public void setZoomDisabled(boolean disablerd) {
    zoomProperty.set(disabled);
}

zoomProperty wäre ggf. besser zoomDisabledProperty oder so ...

Das alles so im Forum geschrieben - also evtl. habe ich mich vertippt oder irgendwo vertan ....

 

[Adriano10]

Also Dein Code würde ich jetzt als relativ dubios bezeichnen.

Das mit den Properties dient dem binding. Und dann hast Du eine Property und die wird gesetzt. Immer eine neue SimpleBooleanProperty zu erzeugen, wenn Du etwas umsetzen willst, ist absolut unnötig!

Wenn Du den Wert nur setzen willst, dann nutz einfach direkt ein set, also etwas wie:
button.disableProperty().set(disable)

Wenn Du zwei Properties hast, die voneinander abhängen, dann kannst Du da entsprechende Bindings aufstellen. Also etwas wie:
zoomOut.disableProperty().bind(zoomIn.disableProperty().not());

Damit würde es ausreichen, nur noch zoomIn.disableProperty().set(...) aufzurufen und zoomOut verändert sich automatisch.

Du kannst dazu auch eine eigene SimpleBooleanProperty nutzen. Dann hättest Du als einmalige Initialisierung etwas wie:

// Instanzvariable:
SimpleBooleanProperty zoomProperty = new SimpleBooleanProperty(true);

// Initialisierung:
zoomIn.disableProperty().bind(zoomProperty);
zoomOut.disableProperty().bind(zoomProperty.not());

// Umsetzen Methode
public void setZoomDisabled(boolean disablerd) {
    zoomProperty.set(disabled);
}

zoomProperty wäre ggf. besser zoomDisabledProperty oder so ...

Das alles so im Forum geschrieben - also evtl. habe ich mich vertippt oder irgendwo vertan ....

Vielen Dank,die werde ich gern ausprobieren und dann einbauen.. Das ist sehr hilfreich,vielen Dank