Изменяя ObservableList через свой changeListener, генерируется исключение UnsupportedOperationException - PullRequest
Новая
       43

Изменяя ObservableList через свой changeListener, генерируется исключение UnsupportedOperationException

0 голосов
/ 05 января 2019

Я делал простую игру в крестики-нолики. Для этого я использовал 2D ObservableList, заполненный перечислителем фигур. Каждый ход я изменяю значение выбранной ячейки на X или O, а затем проверяю, выиграл ли кто-то или игра закончилась ничьей. Если это так, я пополняю список пробелами. По крайней мере, так должно быть. Я столкнулся с проблемой, описанной в названии. Эта проблема не возникает ни в коем случае, я не понимаю, что здесь происходит и не смог найти это в Google. Вот упрощенная версия моего кода: (помощь оценена) 

public class Example{
    enum Figures { NONE, X, O }
    static ObservableList<ObservableList<Figures>> list = FXCollections.observableArrayList();

    public static void main(String... args) {
        for (int i = 0; i < 3; i++) {
            ObservableList<Figures> subList = FXCollections.observableArrayList();
            list.add(subList);

            subList.addListener((ListChangeListener<Figures>) change -> {
                while (change.next()) {
                    Figures figure = subList.get(change.getFrom());
                    if (figure == Figures.NONE) {
                        //show Blank space
                    } else {
                        if (figure == Figures.X) {
                            //show X
                        }
                        if (figure == Figures.O) {
                            //show O
                        }

                        checkState();
                    }
                }
            });
            for (int j = 0; j < 3; j++) {
                subList.add(Figures.NONE);
            }
        }

        //it is a graphical game but here is what I do by clicking buttons to get the exception
        list.get(2).set(0, Figures.X);
        list.get(1).set(0, Figures.O);
        list.get(1).set(1, Figures.X);
        list.get(0).set(1, Figures.O);
        list.get(0).set(2, Figures.X);
    }

    private static void checkState() {
        //checking state and if win or draw
            restartGame();
    }

    private static void restartGame() {
        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                list.get(i).set(j, Figures.NONE);
            }
        }
    }
}

А на выходе 

Exception in thread "JavaFX Application Thread" java.lang.UnsupportedOperationException
at java.util.Collections$UnmodifiableList.add(Collections.java:1314)
at javafx.collections.ListChangeBuilder.nextRemove(ListChangeBuilder.java:208)
at javafx.collections.ListChangeBuilder.nextSet(ListChangeBuilder.java:453)
at javafx.collections.ObservableListBase.nextSet(ObservableListBase.java:115)
at javafx.collections.ModifiableObservableListBase.set(ModifiableObservableListBase.java:162)
at Main.restartGame(Main.java:239)
at Main.lambda$gameScene$2(Main.java:79)
at com.sun.javafx.collections.ListListenerHelper$SingleChange.fireValueChangedEvent(ListListenerHelper.java:164)
at com.sun.javafx.collections.ListListenerHelper.fireValueChangedEvent(ListListenerHelper.java:73)
at javafx.collections.ObservableListBase.fireChange(ObservableListBase.java:233)
at javafx.collections.ListChangeBuilder.commit(ListChangeBuilder.java:482)
at javafx.collections.ListChangeBuilder.endChange(ListChangeBuilder.java:541)
at javafx.collections.ObservableListBase.endChange(ObservableListBase.java:205)
at javafx.collections.ModifiableObservableListBase.set(ModifiableObservableListBase.java:163)
at Main.gameScene(Main.java:170)
at Main.start(Main.java:27)
at com.sun.javafx.application.LauncherImpl.lambda$launchApplication1$161(LauncherImpl.java:863)
at com.sun.javafx.application.PlatformImpl.lambda$runAndWait$174(PlatformImpl.java:326)
at com.sun.javafx.application.PlatformImpl.lambda$null$172(PlatformImpl.java:295)
at java.security.AccessController.doPrivileged(Native Method)
at com.sun.javafx.application.PlatformImpl.lambda$runLater$173(PlatformImpl.java:294)
at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95)
at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
at com.sun.glass.ui.win.WinApplication.lambda$null$147(WinApplication.java:177)
at java.lang.Thread.run(Thread.java:748)
Exception in thread "JavaFX Application Thread" java.lang.UnsupportedOperationException
at java.util.Collections$UnmodifiableList.add(Collections.java:1314)
at javafx.collections.ListChangeBuilder.nextRemove(ListChangeBuilder.java:208)
at javafx.collections.ListChangeBuilder.nextSet(ListChangeBuilder.java:453)
at javafx.collections.ObservableListBase.nextSet(ObservableListBase.java:115)
at javafx.collections.ModifiableObservableListBase.set(ModifiableObservableListBase.java:162)
at Main.restartGame(Main.java:239)
at Main.lambda$gameScene$2(Main.java:79)
at com.sun.javafx.collections.ListListenerHelper$SingleChange.fireValueChangedEvent(ListListenerHelper.java:164)
at com.sun.javafx.collections.ListListenerHelper.fireValueChangedEvent(ListListenerHelper.java:73)
at javafx.collections.ObservableListBase.fireChange(ObservableListBase.java:233)
at javafx.collections.ListChangeBuilder.commit(ListChangeBuilder.java:482)
at javafx.collections.ListChangeBuilder.endChange(ListChangeBuilder.java:541)
at javafx.collections.ObservableListBase.endChange(ObservableListBase.java:205)
at javafx.collections.ModifiableObservableListBase.set(ModifiableObservableListBase.java:163)
at Main.gameScene(Main.java:171)
at Main.start(Main.java:27)
at com.sun.javafx.application.LauncherImpl.lambda$launchApplication1$161(LauncherImpl.java:863)
at com.sun.javafx.application.PlatformImpl.lambda$runAndWait$174(PlatformImpl.java:326)
at com.sun.javafx.application.PlatformImpl.lambda$null$172(PlatformImpl.java:295)
at java.security.AccessController.doPrivileged(Native Method)
at com.sun.javafx.application.PlatformImpl.lambda$runLater$173(PlatformImpl.java:294)
at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95)
at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
at com.sun.glass.ui.win.WinApplication.lambda$null$147(WinApplication.java:177)
at java.lang.Thread.run(Thread.java:748)

1 Ответ

0 голосов
/ 05 января 2019

Проблема в функции checkState() в вашем ListChangeListener. Согласно javadoc в ListChangeListener

Предупреждение. Этот класс напрямую обращается к списку источников, чтобы получить информацию об изменениях. Это фактически делает объект Change недействительным, когда в списке происходит другое изменение. По этой причине небезопасно использовать этот класс в другом потоке. Это также означает, что список источников не может быть изменен внутри слушателя, так как это сделает недействительным этот объект Change для всех последующих слушателей.

...