Discussion :

[CobolProgrammator]

Bonjour, je commence à avoir lu beaucoup d articles sur l'EDT, les threads, mais je reste dans le doûte. Faut il que j'utilise un swingworker ou bien swingutilities.invokeLater (comme ci dessous, code simplifié) ? Sur quels critères doit on décider ?

Code :

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
public class A110StdListener implements KeyListener {
   @Override
   public void keyReleased(KeyEvent e) {
      if (e.getKeyCode() == KeyEvent.VK_ENTER
          P300traitement();
   }
 
public void P300traitement(){                          
   vue.getGlassPane().setVisible(true);
   SwingUtilities.invokeLater(new Runnable() {
      public void run() {
        boolean isOk = lastContexte().P120cleIsOK();
        vue.getGlassPane().setVisible(false);
      }
   });
   pack(); 
   //--> attente d'un nouvel événement de l'utilisateur

En fait j'ai surtout besoin d'un thread depuis que je veux rajouter un glasspane d'attente pour ma méthode P120cleIsOK qui dure environs 2 secondes.
Pour mémoire, cette méthode

• agit (modif texte, couleur etc...) sur les composants de mon panel (JComboBox, JTextField, JLabel ...)
• fait des accès bdd (lecture, mise à jour)

J'ai trouvé pas mal cette page Oracle SDN, clair et concise.

[sinok]

L'un ou l'autre, peu importe, tant que tu fais tes opérations longues dans un thread séparé et les modifications des composants dans le thread Swing.

Le SwingWorker est un utilitaire qui vient faciliter cette tâche, mais en background il utilise plus ou moins un Thread pour la méthode doInBackground et du SwingUtilities.invokeLater qui entoure l'exécution de la méthode done.


Dans ton exemple tu utilises de façon incorrecte le invokeLater.

La façon de faire sans SwingWorker serait la suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public void P300traitement(){                          
    vue.getGlassPane().setVisible(true);
    new Thread(new Runnable() {
        public void run() {
            boolean isOk = lastContexte().P120cleIsOK();
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    vue.getGlassPane().setVisible(false);
                    pack(); 
                }
            });
        }
    }).start();
}

Avec un SwingWorker ça donnerait:

Code :

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
public void P300traitement(){                          
    vue.getGlassPane().setVisible(true);
    SwingWorker worker = new SwingWorker<Boolean, Void>() {
        @Override
        public Boolean doInBackground() {
            return lastContexte().P120cleIsOK();
        }
 
        @Override
        public void done() {
            vue.getGlassPane().setVisible(false);
            pack(); 
        }
    };
    worker.execute();
}

En général l'approche utilisant un SwingWorker est plus propre niveau coding, ça évite d'empiler des Runnable dans tous les sens, par exemple

[CobolProgrammator]

c'est magnifique merci beaucoup;

effectivement je constatais un problème (après avoir pris la méthode invokelater) d'exécution suite à mon mauvais coding.

Je prends donc ta solution swingworker et je vais me pencher sur cette classe pour mieux comprendre.

Est ce indispensable de mettre le pack dans la méthode done() ou bien puis je l'exécuter après le worker.execute() ? (le code que j'ai envoyé est un peu simplifié).

[sinok]

Tout dépend du moment auquel tu veux que le pack s'exécute.
Si c'est avant le déroulement des opération, tu peux le mettre après le execute.
Si c'est après le déroulement des opérations, il devra impérativement être dans le done. La méthode execute n'attendant pas la fin de l'exécution du swingworker pour rendre la main.

[CobolProgrammator]

bon merci BEAUCOUP sinok, j'ai ttes mes réponses, je clos le sujet.