ChangeListener verhält sich komisch

[Joob]

Komischer Effekt, manchmal läuft es, dann wiederum nicht.
Der Fehler tritt auf auch wenn im Changelistener kein Code ausgeführt wird.

Dann habe ich die Exception mal angeschaut, da scheint es sich um ein Mausevent zu drehen, allerdings kenne ich mich nicht gut genug aus um die genaue Ursache zu verstehen. Vielleicht kann mir jemand von euch auf die Sprünge helfen. Wie Ihr seht hat die Exception nur was mit Java Code zu tun und eventuell nur bedingt mit dem was ich programmiert habe.

Also mir geht es darum zu verstehen was die Exception mir sagen will .


folgender Change Listener

        tableview.getSelectionModel().selectedItemProperty().addListener((ObservableValue<? extends WordsWordsTVC> obs, WordsWordsTVC oldSelection, WordsWordsTVC newSelection) -> {
}

Exception in thread "JavaFX Application Thread" java.lang.IndexOutOfBoundsException
    at com.sun.javafx.scene.control.ReadOnlyUnbackedObservableList.subList(ReadOnlyUnbackedObservableList.java:136)
    at javafx.collections.ListChangeListener$Change.getAddedSubList(ListChangeListener.java:242)
    at javafx.scene.control.TableView$TableViewArrayListSelectionModel.handleSelectedCellsListChangeEvent(TableView.java:3004)
    at javafx.scene.control.TableView$TableViewArrayListSelectionModel.clearAndSelect(TableView.java:2423)
    at javafx.scene.control.TableView$TableViewSelectionModel.clearAndSelect(TableView.java:1914)
    at com.sun.javafx.scene.control.behavior.TableCellBehaviorBase.simpleSelect(TableCellBehaviorBase.java:215)
    at com.sun.javafx.scene.control.behavior.TableCellBehaviorBase.doSelect(TableCellBehaviorBase.java:148)
    at com.sun.javafx.scene.control.behavior.CellBehaviorBase.mousePressed(CellBehaviorBase.java:150)
    at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:95)
    at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:89)
    at com.sun.javafx.event.CompositeEventHandler$NormalEventHandlerRecord.handleBubblingEvent(CompositeEventHandler.java:218)
    at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(CompositeEventHandler.java:80)
    at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:238)
    at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:191)
    at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(CompositeEventDispatcher.java:59)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:58)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74)
    at com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:54)
    at javafx.event.Event.fireEvent(Event.java:198)
    at javafx.scene.Scene$MouseHandler.process(Scene.java:3757)
    at javafx.scene.Scene$MouseHandler.access$1500(Scene.java:3485)
    at javafx.scene.Scene.impl_processMouseEvent(Scene.java:1762)
    at javafx.scene.Scene$ScenePeerListener.mouseEvent(Scene.java:2494)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:352)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:275)
    at java.security.AccessController.doPrivileged(Native Method)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler.lambda$handleMouseEvent$355(GlassViewEventHandler.java:388)
    at com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(QuantumToolkit.java:389)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler.handleMouseEvent(GlassViewEventHandler.java:387)
    at com.sun.glass.ui.View.handleMouseEvent(View.java:555)
    at com.sun.glass.ui.View.notifyMouse(View.java:937)
    at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
    at com.sun.glass.ui.win.WinApplication.lambda$null$149(WinApplication.java:191)
    at java.lang.Thread.run(Thread.java:745)

 

[Robat]

Wie Ihr seht hat die Exception nur was mit Java Code zu tun und eventuell nur bedingt mit dem was ich programmiert habe

Ist dein Code etwa kein Java Code? 😉 SCNR 😀

Erzähl mal mehr zu deiner TableView. Der Fehler wird immer mal geworfen, wenn man versucht eine ObservableList zu verändern, während sie gerade verändert/benutzt wird.

 

[Joob]

Ich poste den Code gerne, ist aber ein bisschen unübersichtlich.

Also ich schreibe erst mal.

Das Tableview enthält Vokabeln, dabei gibt es einige Vokabeln die nicht verändert werden dürfen. Da man diese aber als ganzes Löschen können muss wird diese Möglichkeit durch ein Dialogfeld abgefragt.

Beim ersten mal läuft auch alles, und zwar unabhängig von dem Umstand ob gelöscht wird oder nicht. Sobald ich aber die Zeile noch einmal aufrufe, wird der Changelistener ein zweites Mal aufgerufen und zwar ohne für mich erkennbaren Grund.

Ich poste mal den Code im Listener, hoffentlich nicht zu unübersichtlich. Es gibt verschiedene Prüfungen, eimal Sperrungen, dann ob die Vokabel momentan in einem anderen Zusammenhang verwendet wird und zusätzlich ob die Vokabel den Status readonly hat. Zudem wird geprüft ob die Zeile verändert wurde und ggf. nach dem Wechsel gespeichert.

        tableview.getSelectionModel().selectedItemProperty().addListener((ObservableValue<? extends WordsWordsTVC> obs, WordsWordsTVC oldSelection, WordsWordsTVC newSelection) -> {
           
            // es wurde eine neue Zeile ausgewählt
            if ((newSelection != null)) { 
               
                String[]    lockInfo        = null;
                String      messuccess      = "";
                String      newID           = newSelection.getId();
                String      useState        = newSelection.getUsestate();
                String      baselanguage    = newSelection.getBaseLanguage();    
                   
                   
                try {

                    // geht in die Zeile und sperrt, wenn nicht gesperrt ist
                    DataLockClass dlc = new DataLockClass();

                    // wenn newID gleich 0 dann wird neuer Datensatz geschrieben
                    if (!newID.equals("0")) {
                        changePossible = dlc.lockDS(newID, "vokabeln", "0");
                    }

                    // Datensatz ist gesperrt
                    if (!changePossible.substring(0, 1).equals("0")) {

                        tableview.setEditable(false);
                        lockInfo = changePossible.substring(2).split(",");

                        Alert alert = new Alert(Alert.AlertType.INFORMATION);
                        alert.setTitle("Information Dialog");
                        alert.setHeaderText("Change not possible");
                        alert.setContentText("Data locked from user " + lockInfo[1].trim() + " till " + lockInfo[0]);
                        DialogPane dialogPane = alert.getDialogPane();
                        dialogPane.getStylesheets().add(getClass().getResource("/CSS/DialogBase.css").toExternalForm());
                        dialogPane.getStyleClass().add("Dialog");
                        alert.show();
                        // ACHTUNG : bei [alert.showandwait] kommt es zu eine Exception
                        // wenn das Dialogfeld innerhalb des Selectionmodells verwendet wird warum weiß ich nicht

                       
                    // READONLY - handelt es sich um eine readonly Vokabel darf diese nicht verändert werden
                    } else if (useState.equals("1")) {

                        tableview.setEditable(false);

                        Alert alert = new Alert(Alert.AlertType.CONFIRMATION);
                        alert.setTitle("Information Dialog");
                        alert.setHeaderText("Change not possible");
                        alert.setContentText("The vocabulary: " + newID + " - " + baselanguage + " comes from an imported read only Block, it is not allowed to change it " + "\n\n"
                                + "But you have the possibility to delete the vocabulary " + "\n\n"
                                + "Do you want to delete the vocabulary?");

                        ButtonType yes = new ButtonType("Yes");
                        ButtonType no = new ButtonType("No");

                        // Remove default ButtonTypes
                        alert.getButtonTypes().clear();

                        alert.getButtonTypes().addAll(yes, no);
                        DialogPane dialogPane = alert.getDialogPane();
                        dialogPane.getStylesheets().add(getClass().getResource("/CSS/DialogBase.css").toExternalForm());
                        dialogPane.getStyleClass().add("Dialog");

                        // option != null.
                        Optional<ButtonType> option = alert.showAndWait();

                        if (option.get() == null) {
                        } else if (option.get() == yes) {

                            messuccess = newSelection.delObj();
                            if (!(messuccess.substring(2, 3).equals("-"))) {
                                filltv();
                            }

                        } else if (option.get() == no) {
                        } else {
                        }

                        // ACHTUNG : bei [alert.showandwait] kommt es zu eine Exception
                        // wenn das Dialogfeld innerhalb des Selectionmodells verwendet wird warum weiß ich nicht

                    } else {
                        tableview.setEditable(true);
                        goinTVO = newSelection;
                    }

                    // oldSelection ist das Objekt welches in der Zeile gespeichert war die verlassen wurde
                    // bevor diese Zeile ausgewählt wurde
                    // wenn oldSelection = null dann hat man gerade in die Tabelle geklickt
                    if (oldSelection != null) {                            
                        // Daten waren leer, also wurde neu gedrückt aber nichts eingegeben
                        // d.h. es wird das neue Objekt in der ObservableList wieder gelöscht
                        // size() - 1, weil Index 0 basiert ist
                        if (oldSelection.getBaseLanguage().equals("")
                                || oldSelection.getWord().equals("")) {
                            ctvMainOLst.remove(ctvMainOLst.size()-1);

                        } else {
                             // mustSafeLanguage wird beim editieren der Zelle gesetzt
                            if (mustSafe == true) {
                                messuccess = oldSelection.saveObj(languageid, username, baseLanguageByGoin, wordByGoin);

                                // Doppelter Eintrag
                                if (messuccess.split(";")[1].equals("-1")) {

                                    Alert alert = new Alert(Alert.AlertType.INFORMATION);
                                    alert.setTitle("Information Dialog");
                                    alert.setHeaderText("Store or Update impossible !");
                                    alert.setContentText("Word couldn't safe, because the word is allready in list.");
                                    DialogPane dialogPane = alert.getDialogPane();
                                    dialogPane.getStylesheets().add(getClass().getResource("/CSS/DialogBase.css").toExternalForm());
                                    dialogPane.getStyleClass().add("Dialog");
                                    alert.show();  
                                    // ACHTUNG : bei [alert.showandwait] kommt es zu eine Exception
                                    // wenn das Dialogfeld innerhalb des Selectionmodells verwendet wird warum weiß ich nicht

                                    // es darf nur bei Insert removed werden
                                    if( messuccess.substring(0, 1).equalsIgnoreCase("I")) {
                                        ctvMainOLst.remove(ctvMainOLst.size()-1);

                                        // wenn bei Update ein Wert eingtragen wird der schon vorhanden ist   
                                    } else if ( messuccess.substring(0, 1).equalsIgnoreCase("U")) {
                                        // alter Wert muss eingesetzt werden
                                        if (!wordByGoin.equals("")) {
                                            oldSelection.setWord(wordByGoin);
                                        }

                                        if (!baseLanguageByGoin.equals("")) {
                                            oldSelection.setBaseLanguage(baseLanguageByGoin);
                                        }

                                        tableview.refresh();

                                        wordByGoin = "";
                                        baseLanguageByGoin = "";
                                    }

                                } else {
                                    // Neuer Datensatz dazugekommen also + 1
                                    if ( messuccess.substring(0, 1).equalsIgnoreCase("I")) {
                                        anzds = anzds + 1;
                                        insertcorrection = 1;
                                    }
                                }
                                mustSafe = false;                           
                            }
                        }
                        dlc.unlockDS(oldSelection.getId(), "vokabeln", "0","TAB_TABID_USERID");
                    }

                } catch (InstantiationException | IllegalAccessException | ClassNotFoundException | SQLException | IOException | JSONException ex) {
                    Logger.getLogger(WordsController.class.getName()).log(Level.SEVERE, null, ex);
                }
            }
        });        
    }

 

[Robat]

Beim ersten mal läuft auch alles, und zwar unabhängig von dem Umstand ob gelöscht wird oder nicht. Sobald ich aber die Zeile noch einmal aufrufe, wird der Changelistener ein zweites Mal aufgerufen und zwar ohne für mich erkennbaren Grund.

Kannst du das noch mal versuchen genauer zu beschreiben? "Beim ersten mal" -> meinst du damit wenn du eine Zeile das erste mal anklickst? "Sobald ich die Zeile noch einmal aufrufe" -> wechselst du zwischendrin die Zeile?

 

[Joob]

Ich versuche es:
Wenn ich das erste mal auf eine Zeile klicke, die ein Readonly enthält werden die Meldungen korrekt angezeigt und es kommt zu keiner Exception. Danach ist die erste Zeile des Tableview gewählt.

Klicke ich dann noch einmal auf eine Zeile mit readonly, erscheinen die Meldungen einmal, und dann nocheinmal und dann kommt es zu der geposteten Exception.

Ich hatte schon in Erwägung gezogen den Focus am Ende des Listeners auf das AnchorPane zu setzten um immer wieder einen definierten Startpunkt zu haben, das ist mir aber nicht gelungen. Der Controller beinhaltet auch nur das AnchorPane und das Tableview und wenn das Tableview ausgewählt wird, wird immer auch eine Zeile gewählt, was zur Auslösung des Listeners führt.

 

[Joob]

Ich habe immer mehr das Gefühl das die Exception in dem von mir gesehenen Problem mit dem showandwait liegt.
Leider kann ich das wohl jetzt nicht mehr umgehen, da ich ja auf die Antwort des Benutzers warten muss.

 

[Joob]

meinst du das ich das Problem umgehen kann, wenn ich die ganze Alertbox mit der Abfrage in einem andern Thread ablaufen lasse?

 

[Robat]

Du könntest mal versuchen das showAndWait() in Paltform.runLater(() -> ... ); laufen zu lassen

 

[Joob]

Ich habe eine Lösung, die Exception kommt nicht wenn ich alertbox mit der Abfrage in einen Service packe.

Aber den Grund für dieses Verhalten habe ich immer noch nicht kapiert. Also es hängt definitiv am ShowandWait im Listener, aber was führt hier dann zu dem Problem und was genau ist das Problem auf das die Exception hinweist.

Kannst Du mir das erklären ?

 

[Joob]

Ich habe jetzt mal so wie du vorgeschlagen nur PlatformRunLater verwendet. Das reicht um die Exception auszuschließen.

Aber vielleicht könntest du mir mal erklären was genau da abgeht und warum die Exception ausgelöst wird wenn ich showAndWait verwende.

 

[Robat]

MWn wird die Exception gerne geworfen, wenn man versucht die Daten (zb eine ObservableList) zu verändern, während die gerade bearbeitet wird.
Wenn ich das richtig überflogen habe macht genau das dein Dialog ja, oder?

 

[Joob]

Eigentlich nicht, da die Exception auch geworfen wird wenn ich bei Delete no anklick und auch filltv führe ich ja nur aus wenn ein Delete erfolgreich gemacht wurde.

Auch die anderen Dialoge würden eine Exception erzeugen wenn ich ShowandWait angebe.

 

[Robat]

Vielleicht hat @dzim eine andere Idee. Der arbeitet mit FX schon etwas länger als ich 😉

 

[dzim]

Puh. Ich war ein paar Tage krankheitsbedingt abwesend. Ich schau mal den Thread an. 😉
Aber keine Garantie, denn manchmal sieht man auch als "alter Hase" den Wald vor Bäumen nicht... 😛

 

[dzim]

showAndWait blockiert den Thread - schlecht. Denn in der Zeit wird auch dein Listener blockiert. Daher bringt auch die Lösung Platform.runLater eine marginale Verbesserung.

Ohne den kompletten Code (an dem man selbst herumschrauben kann) zu haben, ist es etwas schwer und dein Code ist recht kompliziert. 😛

Ich vermute aber, das konkrete Problem entsteht durch folgende Code-Zeilen

ctvMainOLst.remove(ctvMainOLst.size()-1);

Du könntest daher ja mal einen Breakpoint an den dazugehörigen if-Block machen und dann Schritt für Schritt durchsteppen und schauen, wo die Exception auftritt.

Auch finde ich dein tableView.setEditable() etwas befremdlich (zumal man auch editable auf Zeilen oder Zellen setzen kann, wenn ich mich recht entsinne) - man könnte alternativ auch den disabled-Zustand für so etwas verwenden.

#edit: Sorry, aber mehr sehe ich "auf die Schnelle" auch nicht... 😕

#edit2: Was ich häufig statt ctvMainOLst.remove(ctvMainOLst.size()-1); mache: Ich sammele die zu löschenden Elemente in einer Liste und und lösche nicht anhand eines Indexes, sondern der Referenz(en) yourList.removeAll(toRemoveList) - bei dir reicht natürlich nur das eine Element, eine separate Liste wäre Quatsch.