Discussion :

[Sharcoux]

Bonjour,

Je suis débutant sur les threads, et j'aurais besoin d'une petite aide :

J'ai un thread 1 qui crée une interface graphique, et un Thread 2 qui charge des données utilisateur. Quand les deux tâches sont finies, il faut charger le profil dans l'interface. Comment je suis sensé faire pour faire ça de façon propre ?

La solution que j'utilise actuellement c'est ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
//thread 1
//traitements...
if(!interfaceReady) {chargementEnAttente = true; }//
else {chargement();}
 
 
//thread 2
//traitements...
interfaceReady = true;
if(chargementEnAttente) {chargement();}

Mais ça me parait pas fou...

Merci de votre aide

[JoeChip]

Pourquoi faire ça avec 2 threads ?

[adiGuba]

Salut,


De quel type d'interface graphique s'agit-il ?
Par exemple Swing tu peux utiliser un SwingWorker pour cela...


a++

[Sharcoux]

Pourquoi faire ça avec 2 threads ?

Parce que comme ça, pendant que l'utilisateur choisit quel fichier il veut ouvrir, crée un nouveau fichier, etc, je peux charger toute l'interface, les icônes, etc. Ca fait gagner du temps.

Par exemple Swing tu peux utiliser un SwingWorker pour cela...

Jamais entendu parler. Mais cool, je vais regarder ça. Merci bien !

[tchize_]

Sauf que:

choisir un fichier ne nécessite pas de thread particulier: tu va charger le fichier en réaction au clic sur l'icone du fichier, donc dans le thread de l'EDT (quitte à utilise un swingworker après pour la partie lente de parsing du fichier)

Charger l'interface (icone, création de fenetre, boutons, etc), ça doit aussi se faire dans l'EDT....

[Sharcoux]

Charger l'interface (icone, création de fenetre, boutons, etc), ça doit aussi se faire dans l'EDT....

Ah bon ? Pourquoi ? Tant que je n'ai pas fait setVisible, pourquoi je ne pourrais pas préparer tranquillement mon interface dans un Thread secondaire pendant que j'utilise l'EDT pour faire patienter l'utilisateur ?

choisir un fichier ne nécessite pas de thread particulier: tu va charger le fichier en réaction au clic sur l'icone du fichier, donc dans le thread de l'EDT (quitte à utilise un swingworker après pour la partie lente de parsing du fichier)

Là encore, je mentionnais

un Thread 2 qui charge des données utilisateur

Donc, justement, il va, entre autre faire du parsing de fichier, des initialisations dans le modèle, etc. Rien qui, à mon sens, ne nécessite à priori de figurer dans l'EDT.

Par contre, une fois ces deux Threads terminés, je veux rejoindre l'EDT, charger le modèle dans l'interface et faire setVisible(true) sur mon interface

C'est pas bon ?

[tchize_]

Ah bon ? Pourquoi ? Tant que je n'ai pas fait setVisible, pourquoi je ne pourrais pas préparer tranquillement mon interface dans un Thread secondaire pendant que j'utilise l'EDT pour faire patienter l'utilisateur ?

Dans les doc mentionnant qu'on ne dois pas manipuler un composant swing en dehors de l'EDT, je n'ai jamais vu mentionné 'sauf si il n'est pas visible', mais admettons. Tu peux très bien utiliser le swingworker pour tout ce que tu mentionne.

Il va falloir être un peu plus explicite sur ce qui est "long" dans la création de ton interface, les solutions varient suivant ce qui pose problème. La lecture du fichier, elle, peux se faire dans le swingworker.


Enfin, d'un manière générale, pour attendre qu'un thread aie fini, pas besoin de variable, un simple thread.join() fait l'affaire.

[Sharcoux]

En fait, mon problème, c'est que je sais pas quel Thread aura fini le premier, et que pour passer à la suite, les deux doivent avoir terminé. Au final, je n'ai pas trouvé mieux que d'utiliser 1 ou 2 boolean volatile et à la fin de chaque thread, vérifier que l'autre a fini, comme j'ai montré dans mon premier post. Je pense que si j'avais plus de Thread, la meilleure solution serait d'avoir un objet type manager qui soit appelé à la fin de chaque thread. Quand il a été appelé par tous les Threads qu'il attendait, il lance la suite. Un truc du style

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public class Manager {
    private PredefinedInterface callback;
    private volatile Map<Integer, Boolean> finished;
    public Manager(PredefinedInterface callback, int[] threadIDs) {
        this.callback = callback;
        for(int id : threadIDs) {
             finished.put(id,false);
        }
    }
    public void finished(int id, String msg) {
         if(msg.equals("done")) {finished.put(id,true);}
         if(!finished.containsValue(false)) {callback.callMethod();}
    }
}

Qu'est-ce que vous en pensez ?

[JoeChip]

Alors tu dois mettre le ready à true à la fin du traitement dans le thread, non avant le "if'

[adiGuba]

Qu'est-ce que vous en pensez ?

Je ne sais pas !
Tu nous expliques comment tu fais... mais on ne sait rien de l'objectif de tout cela !


Bref : que veux tu faire ?


a++