Discussion :

[infaac]

Bonjour,

J'ai un JTree assez grand :2000 entités.
J'utilise ce code pour l'ouvrir:

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
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
if (ae.getActionCommand().equals(SecBabel.getString("sec.expand")))
    {
      trace_.debug("Clicks on expand button");
      SwingWorker sw = new SwingWorker() 
      {
    	  //amell
        public Object construct()
        {
          AciModalityManager.startModality();
 
          return null;
        }
 
        public void expandAll(JTree tree) 
        {
          TreeNode root = (TreeNode)tree.getModel().getRoot();
 
          // Traverse tree from root
          expandAll(tree, new TreePath(root));
        }
 
        private void expandAll(JTree tree, TreePath parent) 
        {
          // Traverse children
          TreeNode node = (TreeNode)parent.getLastPathComponent();
          if (node.getChildCount() >= 0) 
          {
            for (Enumeration e=node.children(); e.hasMoreElements(); ) 
            {
              TreeNode n = (TreeNode)e.nextElement();
              TreePath path = parent.pathByAddingChild(n);
              expandAll(tree, path);
            }
          }    
 
          tree.expandPath(parent);
        }
        public void finished()
        {
        	expandAll(fadViewTree_);
        	AciModalityManager.stopModality();
 
        }
 
	  };
      sw.start("");

Lors de l'éxécution de ce code, ma GUI devient toute grisée dés que je l'iconnifie.


Pourrais-je améliorer ce code pour éviter que la GUI soit grisée?


--Merci

[schniouf]

La méthode finished() de SwingWorker est exécutée dans l'EDT (thread d'affichage graphique), et dedans tu fais appel à une grosse fonction, donc c'est normal que l'affichage se bloque.
Essaie plutot de mettre l'appel expandAll(fadViewTree_); dans la méthode doInBackgound() de SwingWorker que tu surcharges. C'est cette méthode qui est exécutée dans un thread à part.

[infaac]

Hi,

est ce que ceci n'est pas en contradiction avec la régle de base : Un seul thread qui s'occuppe de l'affichage.

[schniouf]

Hi,

est ce que ceci n'est pas en contradiction avec la régle de base : Un seul thread qui s'occuppe de l'affichage.

Hum hum, bonne question... Qu'en pensent les pros du Swing ?

[infaac]

hi,

pourquoi est ce que personne ne répond

please help me

[adiGuba]

Salut,

est ce que ceci n'est pas en contradiction avec la régle de base : Un seul thread qui s'occuppe de l'affichage.

Non : car ce qui c'est le parcours des différents noeuds qui doit prendre beaucoup de temps, et cela n'a rien à voir avec de l'affichage !

De plus ton SwingWorker est un peu inutile car la seule tâche qu'il effectue en tâche de fond est de définir la modalité, ce qui devrait être dans l'EDT s'il s'agit bien de modifier la modalité de la fenêtre...

Pour moi cela donnerait quelque chose comme cela :

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
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
		if (ae.getActionCommand().equals(SecBabel.getString("sec.expand"))) {
			trace_.debug("Clicks on expand button");
 
			// A déplacer ici si on est bien dans l'EDT
			AciModalityManager.startModality();
 
			SwingWorker sw = new SwingWorker() {
				// amell
				public Object construct() {
 
					// On parcours tous les noeuds en tâche de fond :
					TreeNode root = (TreeNode) tree.getModel().getRoot();
 
					// Traverse tree from root
					expandAll(tree, new TreePath(root));
 
					return null;
				}
 
				private void expandAll(final JTree tree, final TreePath parent) {
					// Traverse children
					TreeNode node = (TreeNode) parent.getLastPathComponent();
					if (node.getChildCount() >= 0) {
						for (Enumeration e = node.children(); e
								.hasMoreElements();) {
							TreeNode n = (TreeNode) e.nextElement();
							TreePath path = parent.pathByAddingChild(n);
							expandAll(tree, path);
						}
					}
 
					// Lorsqu'on doit modifier l'interface on utilise l'EDT :
					SwingUtilities.invokeLater(new Runnable() {
						public void run() {
							expandPathOnEDT(tree, parent);
						}
					});
 
				}
 
				private void expandPathOnEDT(JTree tree, TreePath parent) {
					tree.expandPath(parent);
				}
 
				public void finished() {
					AciModalityManager.stopModality();
				}
 
			};
			sw.start("");
		}

a++

PS : Si tu utilises Java 5.0, tu devrais plutôt utiliser la nouvelle version de la classe SwingWorker : https://swingworker.dev.java.net/

[infaac]

merci beaucoup de me répondre mais le problème existe encore cad que lors de l'expand, la GUI devient grisée quand je l'iconnifie ...


Comment pouurais-je diminuer le temps de l'expand ?y-a t'il une optimisation à faire pour améliorer la performance?