JAVAFX Вызов функции в потоке

Question

В настоящее время я пытаюсь вызвать функцию из моего Thread (который является простым счетчиком). Поток работает хорошо без функции, но когда я пытаюсь вызвать его, я получаю эту ошибку:

Exception in thread "Thread-5" java.lang.IllegalStateException: Not on FX application thread; currentThread = Thread-5
at com.sun.javafx.tk.Toolkit.checkFxUserThread(Toolkit.java:236)
at com.sun.javafx.tk.quantum.QuantumToolkit.checkFxUserThread(QuantumToolkit.java:423)
at javafx.scene.Parent$2.onProposedChange(Parent.java:367)
at com.sun.javafx.collections.VetoableListDecorator.remove(VetoableListDecorator.java:329)
at window.GamePane.destroyCardPane(GamePane.java:716)
at window.GamePane.timeIsUp(GamePane.java:810)
at window.GamePane$1.run(GamePane.java:74)
at java.lang.Thread.run(Unknown Source)

Итак, есть моя тема:

public Thread getCounter() {
    if(counter == null) {
        counter = new Thread(new Runnable() {

            @Override
            public void run() {
                for (int i = 5; i >= 0; i--) {
                    try {
                        System.out.println(i);
                        if(i == 0) {
                            System.out.println("Time is up");
                            timeIsUp();
                            System.out.println("test");
                            break;
                        }
                        Thread.sleep(1000);
                    } catch (InterruptedException e) {
                        Thread.currentThread().interrupt();
                        break;
                    }
                }
            }
        });
        counter.setDaemon(true);
    }
    return counter;
}

Появляются числа от 5 до 0 и время истекло, но затем я получаю сообщение об ошибке.

И эту функцию я пытаюсь вызвать, когда счетчик достигнет 0:

public void timeIsUp() {
    getCounter().interrupt(); // stopping the thread? I tried without this and it's still the same error
    if (turn+1 < nbPlayers) {
        turn++;
    }  else {
        turn = 0;
    }
    getBtnDes().setDisable(false); // Disable a button
    destroyCardPane(); // remove an AnchorPane from this.children()
}

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

Спасибо

Answer 1

Все операции пользовательского интерфейса должны выполняться в потоке приложений FX. Вы можете вызвать Platform.runLater(...) из фонового потока и передать ему Runnable, который будет выполняться в потоке приложения FX.

Нет необходимости прерывать существующий поток. Если вы переупорядочите код так, чтобы вы спали сначала, а затем проверьте, то поток все равно собирается завершиться:

public Thread getCounter() {
    if(counter == null) {
        counter = new Thread(new Runnable() {

            @Override
            public void run() {
                for (int i = 5; i >= 0; i--) {
                    try {
                        Thread.sleep(1000);
                        System.out.println(i);
                        if(i == 0) {
                            System.out.println("Time is up");

                            Platform.runLater(this::timeIsUp);

                            System.out.println("test");
                            break;
                        }
                    } catch (InterruptedException e) {
                        Thread.currentThread().interrupt();
                        break;
                    }
                }
            }
        });
        counter.setDaemon(true);
    }
    return counter;
}

Теперь вы можете удалить вызов прерывания из метода timeIsUp().

public void timeIsUp() {
    if (turn+1 < nbPlayers) {
        turn++;
    }  else {
        turn = 0;
    }
    getBtnDes().setDisable(false); // Disable a button
    destroyCardPane(); // remove an AnchorPane from this.children()
}

Обратите внимание, что вы можете делать все это без потоков, используя Timeline:

Timeline timeline = new Timeline();
for (int i = 5 ; i >= 0 ; i=--) {
    final int count = i ;
    KeyFrame frame = new KeyFrame(Duration.seconds(5-i), e -> System.out.println(count));
    timeline.getKeyFrames().add(frame);
}
timeline.setOnFinished(e -> {
    System.out.println("Time is up");
    timeIsUp();
});
timeline.play();

Здесь нет блокировки (поэтому нет необходимости в потоках), а обработчик событий ключевого кадра и обработчик onFinished выполняются в потоке приложений FX, поэтому безопасно обновлять там пользовательский интерфейс.

Answer 2

Только поток JavaFX может выполнять операции над графом сцены JavaFX. Если вам нужно изменить что-то в JavaFX из другого потока, вам нужно использовать Platform.runLater(Runnable).