[Thread] JavaFX et Thread

**dada-06** · 16/07/2014, 16h45

Bonjour à tous,
J’ai un souci. Je veux que mon Thread change la visibilité de ma ProgressBar.
Mais ça me renvoie :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
Exception in thread "Thread-4" java.lang.IllegalStateException: Not on FX application thread; currentThread = Thread-4
	at com.sun.javafx.tk.Toolkit.checkFxUserThread(Unknown Source)
	at com.sun.javafx.tk.quantum.QuantumToolkit.checkFxUserThread(Unknown Source)
	at javafx.scene.Scene.addToDirtyList(Unknown Source)
	at javafx.scene.Node.addToSceneDirtyList(Unknown Source)
	at javafx.scene.Node.impl_markDirty(Unknown Source)
	at javafx.scene.Node$5.invalidated(Unknown Source)
	at javafx.beans.property.BooleanPropertyBase.markInvalid(Unknown Source)
	at javafx.beans.property.BooleanPropertyBase.set(Unknown Source)
	at javafx.css.StyleableBooleanProperty.set(Unknown Source)
	at javafx.scene.Node.setVisible(Unknown Source)
	at application.ThreadImag.run(ThreadImag.java:27)

Voici mon code que j'ai raccourci

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
package application;
 
import javafx.application.Application;
import javafx.scene.Group;
import javafx.scene.Parent;
import javafx.scene.Scene;
import javafx.scene.control.ProgressBar;
import javafx.stage.Stage;
 
public class ProgressBarApp extends Application {
 
    public static ProgressBar p1;
	public Parent createContent() {
 
        p1 = new ProgressBar();
   		ThreadImag t= new ThreadImag();
   		t.start();
        Group group = new Group();
        group.getChildren().addAll(p1);
        return group;
    }
 
    @Override
    public void start(Stage primaryStage) throws Exception {
        primaryStage.setScene(new Scene(createContent()));
        primaryStage.show();
    }
    public static void main(String[] args) {
        launch(args);
    }
}

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
package application;
 
public class ThreadImag extends Thread{
	private int nb_ligne=1200;
	int ii=0;
	public ThreadImag() {
	}
 
	public void run() {
			for (int i = 0; i < nb_ligne; i++) {
				if (i > ii + 99) {
					ii = i;
					System.out.print("\r" + i + "/" + nb_ligne);
					ProgressBarApp.p1.setProgress((double)(i)/nb_ligne);
						try {
							Thread.sleep(100);
						} catch (InterruptedException e) {
							e.printStackTrace();
						}
				}
			}
			ProgressBarApp.p1.setVisible(false);
			System.out.print("\n");
		System.out.println("Thread off");
		Thread.currentThread().interrupt();
	}
}

Sinon j'avais essayé de modifier de texte (pour afficher le pourcentage de ma ProgressBar) et ça me renvoyait la même erreur
J'ai essayé de trouver cette erreur sur Google, mais vue que je ne la comprends pas très bien et que je fais habituellement des Thread en C, j'avoue avoir du mal à trouver.
Merci
David B.

**bouye** · 16/07/2014, 23h38

Dans son API publique JavaFX utilise 2 threads :

Le JavaFX Application Thread - dans lequel sont effectué les manipulation des contrôles attachés a une scène et les réactions aux événements.
Le thread media - qui est utilisé entre autre par les animations et transitions.

Tu modifies une ProgressBar qui est rattachée a une scène, comme l'indique l'erreur une telle manipulation ne peut être faite que sur le JavaFX Application Thread or toi tu le fais depuis un autre thread d’où l'erreur.

Une manière de corriger le problème est de remplacer :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ProgressBarApp.p1.setProgress((double)(i)/nb_ligne);

Par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Platform.runLater(() -> ProgressBarApp.p1.setProgress((double)(i)/nb_ligne));

idem si tu veux modifier le texte.

Sinon, JavaFX offre un framework déjà tout fait pour exécuter des taches de fond, il s'agit de Service<T> et Task<T> (T est ici le type de l'objet produit par la tache et retourné par le service. Utiliser le type Void et faire un return null a la fin de la méthode call() de la tache pour une tache qui ne retourne rien).

Service<T> - est la classe a manipuler depuis l'UI (ex: dans le callback qui effectue l'action d'un bouton) et sert a créer une tache, c'est une classe réutilisable.
Task<T> - est la tache a proprement dite, elle est créée par le service. Elle n'est utilisée qu'une seule et unique fois et le corps de la fonction call() sera appelé dans un autre thread.

La tache dispose de plusieurs methodes qui sont appelable directement depuis le thread d’exécution et qui mettront a jour des propriétés sur le service de manière thread safe.

updateProgress() - mise a jour de la progression.
updateMessage() - mise a jour du message.
updateTitle() - mise a jour du titre.

Ainsi, il est possible de binder la propriété progress de la ProgressBar sur la propriété progress du Service pour voir la barre se remplir automatiquement d’elle-même durant l’exécution du service et sans jamais avoir besoin de la manipuler manuellement depuis le code de la tache (on ne fait que appeler updateProgress() durant la tache).

Coté service on a 3 callbacks qu'on peut utiliser pour réagir lorsque ce dernier se fini:

onCancelled - il a été annulé. La tache peut a tout moment vérifier si le service a été annulé en testant isCancelled().
onFailed - il a échoué, on peut récupérer l'exception en appelant getException().
onSucceeded - tout s'est bien passé, on peut récupérer le résultat en appelant getValue().

PS : J'ai aussi un article sur ça qui prend la poussière depuis des lustres (bien 10 mois maintenant) en attente de relecture donc si l'un d'entre vous se sent l’âme d'un correcteur technique (la correction orthographique sera faite plus tard par des correcteurs dédiés), n’hésitez pas à postuler auprès du responsable de section