Discussion :

[mavina]

Salut,

Je suis ammené à faire une copie d'un fichier local vers un serveur distant.

J'ai donc trouvé une fonction qui marche parfaitement, mais quand on copie des gros fichiers, celà prend du temps. Du coup, j'aimerai faire un JProgressBar et l'intégrer à cette fonction.

Voici ce que j'ai fait :

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
 
		private void copyFile(File in, File out) 
	    {
	    	FileChannel inChannel=null;
	    	FileChannel outChannel=null;
	    	try
	    	{
 
	    		JProgressBar progressBar = new JProgressBar();
 
				/*progressBar.setIndeterminate(true);
 
				progressBar.setVisible(true);*/
 
				//progressBar.setIndeterminate(false);
 
 
 
 
	            inChannel = new FileInputStream(in).getChannel();
	       	 	outChannel = new FileOutputStream(out).getChannel();
	            /**specially for windows cases**/
 
	            // magic number for Windows, 64Mb - 32Kb)
	           int maxCount = (64 * 1024 * 1024) - (32 * 1024);
	           long size = inChannel.size();
	           long position = 0;
 
	           progressBar.setStringPainted(true);
	           progressBar.setMaximum((int)(size/maxCount));
	           progressBar.setMinimum(0);
	           progressBar.setValue(0);
 
	           final JDialog jd=new JDialog((JFrame)null, true);
	           jd.setSize(400,150);
	           jd.add(progressBar);
	           jd.setLocationRelativeTo(null);
 
 
				new Thread(new Runnable()
				{
					public void run()
					{
						System.out.println("setting progressbar visible");
						jd.setVisible(true);
					}
				}
				).start();
 
 
 
	           //progressBar.setVisible(true);
 
	           while (position < size) 
	           {
 
					position += inChannel.transferTo(position, maxCount, outChannel);
					progressBar.setValue(new Long(position).intValue());
					progressBar.setString(position/maxCount+"/"+size/maxCount);
	            }
 
	            jd.dispose();
	            //progressBar.close();
	            System.out.println("end of copy");
	        }catch(Exception e)
	    	{
	    		e.printStackTrace();
	    	}
	        finally 
	        {
 
	        }
 
 
	    }

Je précise que cette fonction est appelée lors d'un clic de bouton, dans un ActionListener. Pour afficher ma jdialog sans que celà ne gèle le traitement, je le met visible dans un thread approprié, puisque l'appel à invokeLater de SwingUtilities donnerai un affichage apres le traitement (qui est appelé dans un event, je le rappelle).

Or, le problème est qu'on se retrouve avec une jdialog qui s'affiche mais dans laquelle rien n'est affiché pendant le traitement (bug d'affichage "gel" classique à java) et lorsque le traitement est fini, elle s'affiche bien si on ne dispose pas, sinon elle dispose tout simplement.

J'aimerai savoir pourquoi, et comment je peux y remédier pour bien afficher mon traitement.

Merci

Fred

[Pill_S]

Hello,

ce n'est pas le setvisible qu'il faut mettre dans un thread, mais bien le traitement de copie

et ensuite utilise invokeandwait pour repainter les composants swing

[mavina]

Salut,

Merci de ta réponse!

Voici mon code, modifié en conséquence (c'est ce que j'avais fait à la base) :

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
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
 
		private void copyFile(File in, File out) 
	    {
 
	    	try
	    	{
 
	    		final JProgressBar progressBar = new JProgressBar();
 
				/*progressBar.setIndeterminate(true);
 
				progressBar.setVisible(true);*/
 
				//progressBar.setIndeterminate(false);
 
 
 
	    		final FileChannel inChannel=new FileInputStream(in).getChannel();
	    		final FileChannel outChannel=new FileOutputStream(out).getChannel();
	            /**specially for windows cases**/
 
	            // magic number for Windows, 64Mb - 32Kb)
 
 
	           progressBar.setStringPainted(true);
	           //
	           progressBar.setMinimum(0);
	           progressBar.setValue(0);
 
	           final JDialog jd=new JDialog((JFrame)null, true);
	           jd.setSize(400,150);
	           jd.add(progressBar);
	           jd.setLocationRelativeTo(null);
 
 
				SwingUtilities.invokeAndWait(new Runnable()
				{
					public void run()
					{
						try
						{
							int maxCount = (64 * 1024 * 1024) - (32 * 1024);
				            long size = inChannel.size();
				            long position = 0;
 
				           progressBar.setMaximum((int)(size/maxCount));
							while (position < size) 
				           {
 
								position += inChannel.transferTo(position, maxCount, outChannel);
								progressBar.setValue(new Long(position).intValue());
								progressBar.setString(position/maxCount+"/"+size/maxCount);
				            }
 
				            jd.dispose();
				            //progressBar.close();
				            System.out.println("end of copy");
						}catch(Exception e)
						{
							e.printStackTrace();
						}
 
					}
				}
				);
 
 
 
	           //progressBar.setVisible(true);
 
 
	        }catch(Exception e)
	    	{
	    		e.printStackTrace();
	    	}
	        finally 
	        {
 
	        }
 
 
	    }

Et l'exception levée :

Exception in thread "AWT-EventQueue-0" java.lang.Error: Cannot call invokeAndWai
t from the event dispatcher thread
at java.awt.EventQueue.invokeAndWait(EventQueue.java:980)
at javax.swing.SwingUtilities.invokeAndWait(SwingUtilities.java:1323)
at XMLTree$AttachFile.copyFile(XMLTree.java:3645)
at XMLTree$AttachFile.actionPerformed(XMLTree.java:3604)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:19
95)

Je suppose que c'est parceque j'appelle la fonction copy dans un event non ?

F.

[Pill_S]

En pseudo-code, ça donnerait un truc comme ceci selon moi:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
dans le event:
 
- créer un progress dialog
- le rendre visible
- lancer le thread de traitement (copie)
 
dans le thread
- a chaque boucle, incrémenter la progression de la barre
- invoquer invokeandwait avec un runnable qui ne fait que le repaint de la barre

[sinok]

Cadeau de type FAQ: http://java.developpez.com/faq/gui/?...raitement_long

Et re cadeau de type tuto: http://gfx.developpez.com/tutoriel/j...ing-threading/

Et re re cadeau de type tuto pour une solution bien foutue: http://rom.developpez.com/java-swingworker/

[Pill_S]

Cadeau de type FAQ...
Et re cadeau de type tuto...
Et re re cadeau de type tuto ...

c'est noël!

[adiGuba]

Salut,



Tu mélanges EDT et threads.

Pour rappel l'EDT est le thread graphique dédié à gérer l'interface graphique (affichage, évènements, etc.), et il ne doit être exécuté QUE pour cela !

invokeLater() et invokeAndWait() permettent d'exécuter du code dans l'EDT, mais cela ne doit contenir que du code modifiant l'interface graphique et en aucun cas des traitements divers, et surtout pas des traitement potentiellement lourds...

Il faut voir l'EDT comme un thread qui tourne en boucle sur une pile d'action à exécuté, contenant les mises à jours de l'affichage, les évènements et les appels à invokeLater()/invokeAndWait(). Si tu effectues un traitement lourd dedans tu décaleras tout le reste et l'interface sera figé...


En clair tu dois utiliser un thread pour le copie des fichiers, et invokeLater()/invokeAndWait() pour la mise à jour de l'interface. Or tu fais exactement l'inverse...



SwingWorker permet de faciliter tout cela, en séparant bien le code qui doit être exécuté dans un thread séparé de celui qui devra être dans l'EDT...


Sinon il y a aussi ProgressMonitor qui permet d'afficher une fenêtre de progression assez simplement...


a++

[mavina]

Résolu et fonctionnel, merci !