GUI grisée quand je fais un expand sur un JTree

Version imprimable

Bonjour,

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

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
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

19/01/2007, 14h14
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.
19/01/2007, 14h45
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. :roll:
19/01/2007, 15h07
schniouf

Citation:

Envoyé par 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. :roll:

Hum hum, bonne question... Qu'en pensent les pros du Swing ?
24/01/2007, 17h17
infaac

hi,

pourquoi est ce que personne ne répond

please help me:help: :help:

Salut,

Citation:

Envoyé par infaac

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

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:

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/

25/01/2007, 14h50
infaac

still exist!!

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?:?