SwingWorker peut il être modal?

Version imprimable

08/10/2010, 11h59
Micke7

SwingWorker peut il être modal?

Bonjour,

Est-il possible de bloquer les actions utilisateurs sur une application lorsqu'un gros traitement (+affichage de barre de progression) est en cours dans un SwingWorker? J'ai bien tenté de rendre le JDialog, contenant la barre de progression, modal mais cela gèle l'avancement de la barre.
De même j'ai essayé en créant un autre JDialog (hors et in SwingWorker) mais cela ne fonctionne pas non plus (hors => le traitement ne se lance pas tant que le jdialog n'est pas fermé, in => géle de la barre de progression).

Merci.
08/10/2010, 13h30
sinok

Comment mets tu à jour ta barre de progression dans le SwingWorker?

Elle se faut dans une boucle for dans le doInBackground() avec un setProgress()

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
private int recalculStock(double progressStart, double progressEnd) {
  double step = (progressEnd - progressStart) / _listArticles.size();
  for (Article article:_listArticles){
    try {
      progressStart += step;
      publish(article.toString());
      article.setQteStock(article.getQteEnStock());
      ArticleIO.updateArticle(article);
      setProgress((int) progressStart);
    } catch (ControleurException e) {
      // TODO Auto-generated catch block
      e.printStackTrace();
    }
  }
  return _listArticles.size();
}

Le setProgress ne met pas automatiquement à jour une JProgressBar.

Il te faut ajouter un PropertyChangeListener à ton SwingWorker qui va écouter les modifications sur la property progress et les répercuter sur ta progressBar

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
final JProgressBar progressBar = new JProgressBar(0, 100);
PrimeNumbersTask task = new PrimeNumbersTask(textArea, N); 
task.addPropertyChangeListener(      
   new PropertyChangeListener() {          
      public  void propertyChange(PropertyChangeEvent evt) {              
         if ("progress".equals(evt.getPropertyName())) {
            progressBar.setValue((Integer)evt.getNewValue());
         }
      }     
    }
);

08/10/2010, 14h53
Micke7

Oui oui, il est est, j'ai oublié de le préciser ;)
Le problème de non mise à jour n'intervient que lorsque mon jdialog est modal.
08/10/2010, 16h04
sinok

Fais voir le code dans lequel tu lances ton SwingWorker et tu attaches ton listener.

Voilà comment je lance mon SwingWorker

Code:

1
2
3
4
5
6
7
8
9
10
11
 
private void onClickRecalculStock(){
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                /* Démarrage de l'interface graphique et du SwingWorker. */
            	barreProgressionSwingWorker BpSW = new barreProgressionSwingWorker(_lesArticlesIntrants);
            	barreProgressionSwingWorker.MonSwingWorker swingWorker = BpSW.new MonSwingWorker();
                swingWorker.execute();
            }
        });
    }

et pour mon listener rien de spécial

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
public MonSwingWorker() {
	            /* On ajoute un écouteur de barre de progression. */
	            addPropertyChangeListener(new PropertyChangeListener() {
	                public void propertyChange(PropertyChangeEvent evt) {
	                    if("progress".equals(evt.getPropertyName())) {
	                        progressBar.setValue((Integer) evt.getNewValue());
	                        progressBar.setString ("Progression en cours " + ((Integer)evt.getNewValue()).toString() + "%");
	                    }
	                }
	            });
	        }

08/10/2010, 16h16
sinok

Je suppose que ta méthode onClickRecalculStock est appelée depuis un listener, donc de fait s'exécute déjà dans l'EDT.
Essaie donc de virer ton SwingUtilities.invokeLater qui englobe son contenu.
08/10/2010, 16h36
Micke7

Je pensais que l'utilisation d'un SwingWorker se faisait toujours avec un SwingUtilities.invokeLater() :?
Donc avec ce code, ce que je faisais c'était créer un nouveau thread pour la création de ma barre de défilement et au autre pour le traitement? J'ai bien compris?

Sinon après la modification le résultat reste le même: aucun problème si ma JDialog n'est pas modal et freeze si elle l'est.
08/10/2010, 17h41
sinok

Fais voire ton code complet. (JDialog+SwingWorker)

De retour après le week end 8-)

L'appel de mon JDialog set fait ainsi via la méthode OnClickRecalculStock qui est appelée dans un listener.

Code:

1
2
3
4
5
6
 
private void onClickRecalculStock(){
    barreProgressionSwingWorker BpSW = new barreProgressionSwingWorker(_lesArticlesIntrants);
    barreProgressionSwingWorker.MonSwingWorker swingWorker = BpSW.new MonSwingWorker();
    swingWorker.execute();
    }

Mon Jdialog est une classe interne à la classe appelant onClickRecalculStock()

Code:

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
 
class barreProgressionSwingWorker extends JDialog {
    private static final long serialVersionUID = -6419612858688347140L;
    private JLabel labelText;
    private JProgressBar progressBar;
    private List<Article> _listArticles;
 
    public class MonSwingWorker extends SwingWorker<Integer, String> {
        public MonSwingWorker() {
            /* On ajoute un écouteur de barre de progression. */
            addPropertyChangeListener(new PropertyChangeListener() {
                public void propertyChange(PropertyChangeEvent evt) {
                    if("progress".equals(evt.getPropertyName())) {
                        progressBar.setValue((Integer) evt.getNewValue());
	                 progressBar.setString ("Progression en cours " + ((Integer)evt.getNewValue()).toString() + "%");
                    }
                }
            });
        }
 
        @Override
        public Integer doInBackground() {
            return recalculStock(0, 100);
        }
 
        private int recalculStock(double progressStart, double progressEnd) {
            double step = (progressEnd - progressStart) / _listArticles.size();
            for (Article article:_listArticles){
            try {
                progressStart += step;
                publish(article.toString());
                article.setQteStock(article.getCalculQteEnStock());
                ArticleIO.updateArticle(article);
                setProgress((int) progressStart);
	     } catch (ControleurException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
	     }
        }
 
        return _listArticles.size();
    }
 
    @Override
    protected void process(List<String> strings) {
        /* Affichage des publications reçues dans le textarea. */
        for(String s : strings)
            labelText.setText("Article en cours de traitement: " + s);
    }
 
    @Override
    protected void done() {
        /* Le traitement est terminé. */
        barreProgressionSwingWorker.this.dispose();
        JOptionPane.showMessageDialog(
        ViewInterface.getMainFrame(), 
        "Mise à jour des stocks effectuée avec succés.",
        "Information",
        JOptionPane.INFORMATION_MESSAGE);
    }
 
 
    public barreProgressionSwingWorker(List<Article> listArticles) {
        /* Construction de l'interface graphique. */
        super(ViewInterface.getMainFrame(), "Mise à jour stocks",false);
        Dimension dimension = new Dimension(350,80);
        this.setPreferredSize(dimension);
        this.setResizable(false);
        _listArticles = listArticles;
        setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
 
        labelText = new JLabel();
        labelText.setHorizontalAlignment(SwingConstants.CENTER);
 
        progressBar = new JProgressBar(0,99);
        progressBar.setStringPainted(true);
        progressBar.setString ("Progression 1%");
 
        JPanel content = new JPanel(new BorderLayout());
        content.add(labelText,BorderLayout.CENTER);
        JPanel south = new JPanel(new BorderLayout());
        south.add(progressBar);
        content.add(south, BorderLayout.SOUTH);
        setContentPane(content);
        pack();
        setVisible(true);
        Utils.centrer(this);
    }
}

Quelque chose de mauvais?