Probleme d'affichage JDialog et JProgressBar

Version imprimable

Bonjour à tous,

j'ai un petit soucis avec l'affichage de ma boite de dialogue (plus particulièrement du contenu de celle-ci).

j'ai créé une JFrame avec un bouton, une boite de dialogue contenant une barre de progression doit s'afficher lorsque l'app est lancé.

jusque là tout va bien , ça marche impeccable.

cette boite dialogue doit aussi s'ouvrir lorsque l'on appuie sur le bouton de la JFrame, et là c'est le drame, la boite de dialogue s'affiche mais pas son contenu, pourtant la tache qui doit s’exécuter pendant l'affichage se déroule bien.

Pourriez-vous regarder mon code et me dire ce que j'ai mal fais ?

Tous les commentaires, critiques, félicitations ou autres applaudissement seront bien évidement acceptés. :mrgreen:

Je précise que je me suis basé sur le code trouvé à cette adresse : http://java2s.com

En remerciant par avance tous ceux qui voudront bien m'aider.

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
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
 
import javax.swing.JButton;
import javax.swing.JFrame;
 
public class Main {
	static JFrame parentFrame;
	public static void main(String[] args) {
 
		parentFrame = new JFrame();
		parentFrame.setLayout(null);
		parentFrame.setSize(500, 150);
		parentFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		parentFrame.setVisible(true);
		parentFrame.setLocationRelativeTo(null);
		JButton btnNewButton = new JButton("Bouton");
		parentFrame.add(btnNewButton);
		btnNewButton.addMouseListener(new MouseAdapter() {
			public void mouseClicked(MouseEvent e) {
				maj();
			}
		});
		btnNewButton.setBounds(10, 11, 89, 23);
 
		maj();
	}
	private static void maj(){
 
		BarProg dlg = new BarProg(parentFrame);
		BarProg.lcmt(dlg);
 
	}
}

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
import java.awt.BorderLayout;
 
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JProgressBar;
 
public class BarProg extends JDialog {
	/**
         * 
         */
	private static final long serialVersionUID = 1L;
	static JProgressBar dpb;
	public BarProg(JFrame parentFrame){
		super(parentFrame," ",true);
		dpb = new JProgressBar(0, 500);
		dpb.setIndeterminate(true);
		this.add(BorderLayout.CENTER, dpb);
		this.add(BorderLayout.NORTH, new JLabel("Progress..."));
		this.setDefaultCloseOperation(JDialog.DO_NOTHING_ON_CLOSE);
		this.setSize(300, 75);
		this.setLocationRelativeTo(parentFrame);
 
	}
 
	static void lcmt(final BarProg barProg){
		barProg.repaint();
		Thread t = new Thread(new Runnable() {
			public void run() {
				barProg.setVisible(true);
			}
		});
		t.start();
		for (int i = 0; i <= 500; i++) {
			dpb.setValue(i);
			if(dpb.getValue() == 500){
				barProg.setVisible(false);
			}
			try {
				Thread.sleep(10);
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
		}
	}
}

Salut,

Les évènements SWING s'exécutent dans l'EDT (Event Dispatcher Thread), qui se trouve être également le thread utilisé par SWING pour mettre à jour l'affichage.

Quand la méthode maj() est appelée dans le thread principal (le thread par défaut utilisé par ton programme, qui n'est pas l'EDT), elle appelle Lcmt dans ce même thread, et la boucle for est exécutée donc dans ce thread, ainsi que la fermeture du dialog à la fin. Et tout fonctionne comme attendu.

Quand tu lances la méthode maj() depuis l'événément mouseClicked() (au passage, on utilise plutôt un actionListener pour agir sur l'action d'un bouton), la méthode maj() est appelée donc dans l'EDT, et Lcmt() également, et, donc, la boucle s'exécute dans l'EDT : aucun affichage ne pouvant être rafraichi tant que le code n'est pas terminée (tant que l'EDT est occupé), donc tant que la boucle n'est pas terminée, mais en plus après qu'on ait fermé le dialogue. Donc tu ne vois jamais ta barre de progression.

C'est typiquement le genre d'exemple un peu bidon, sans explication, qui, sorti de son contexte (inexistant), n'est pas utilisable (ou du moins n'est pas efficace pour apprendre). La façon dont tu l'as modifiée le montre : la boucle ne sert à rien dans ce cas (dans l'exemple d'origine elle sert à mettre à jour le label qui affiche le compteur, et la progressbar qui n'est pas "inderminée").

Modifie ta classe BarProg comme ça, au moins :

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
public class BarProg extends JDialog {
	/**
         * 
         */
	private static final long serialVersionUID = 1L;
	static JProgressBar dpb;
	public BarProg(JFrame parentFrame){
		super(parentFrame," ",true);
		dpb = new JProgressBar(0, 500);
		dpb.setIndeterminate(true);
		this.add(BorderLayout.CENTER, dpb);
		this.add(BorderLayout.NORTH, new JLabel("Progress..."));
		this.setDefaultCloseOperation(JDialog.DO_NOTHING_ON_CLOSE);
		this.setSize(300, 75);
		this.setLocationRelativeTo(parentFrame);
 
	}
 
	static void lcmt(final BarProg barProg){
		Thread t = new Thread(new Runnable() {
			public void run() {
				try {
					Thread.sleep(5000);
				} catch (InterruptedException e) { 
				}
				barProg.setVisible(false); // on ferme (on cache) le dialogue
				barProg.dispose(); // si on ne sert plus du dialogue, autant libérer les ressources associées
			}
		});
		t.start(); // lance le thread qui attend 5 s avant de refermer le dialogue
		barProg.setVisible(true); // on affiche le dialogue (comme il est modale, ça bloque le thread courant)
	}
}

En fait, pour être plus clean, il faudrait faire, au moins (parce qu'on pourrait encore utiliser un SwingWorker, et peut être même tester si on lance la méthode depuis l'EDT ou pas) :

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
package project;
import java.awt.BorderLayout;
 
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JProgressBar;
import javax.swing.SwingUtilities;
 
public class BarProg extends JDialog {
	/**
         * 
         */
	private static final long serialVersionUID = 1L;
	static JProgressBar dpb;
	public BarProg(JFrame parentFrame){
		super(parentFrame," ",true);
		dpb = new JProgressBar(0, 500);
		dpb.setIndeterminate(true);
		this.add(BorderLayout.CENTER, dpb);
		this.add(BorderLayout.NORTH, new JLabel("Progress..."));
		this.setDefaultCloseOperation(JDialog.DO_NOTHING_ON_CLOSE);
		this.setSize(300, 75);
		this.setLocationRelativeTo(parentFrame);
 
	}
 
	static void lcmt(final BarProg barProg){
		Thread t = new Thread(new Runnable() {
			public void run() {
                                // on force l'appel sur l'EDT, sans que le thread courant ne soit bloqué (le setVisible(true) va bloquer l'appel, puisque le dialogue est modal)
				SwingUtilities.invokeLater(new Runnable() {
 
					@Override
					public void run() {
						barProg.setVisible(true); // on affiche le dialogue (comme il est modale, ça bloque le thread courant)
					}
 
				});
                                // la boucle d'attente est faite en dehors de l'EDT pour ne pas bloquer l'affichage (et éventuellement d'autres composants SWING dont on voudrait pouvoir se servir pendant la progression)
				try {
					Thread.sleep(5000);
				} catch (InterruptedException e) { 
				}
                                // on ferme le dialogue sur l'EDT, ce qui garanti qu'il soit bien fermé après avoir été ouvert (enfin normalement ça devrait être le cas, même si on l'exécutait dans le thread courant, puisqu'il y a peu de risque que SWING rame pendant 5 secondes avant d'afficher le dialogue, mais on ne sait jamais)
				SwingUtilities.invokeLater(new Runnable() {
 
					@Override
					public void run() {
						barProg.setVisible(false); // on ferme (on cache) le dialogue
						barProg.dispose(); // si on ne sert plus du dialogue, autant libérer les ressources associées
					}
 
				});
			}
		});
		t.start(); // lance le thread, qui affiche le dialogue, qui attend 5 s avant de refermer le dialogue
	}
}

07/03/2014, 15h48
titigafr

j'étais justement sur la piste du swing.utilities,
j’étais un peu perdu entre les invokeLater et les invokeAndWait.

Je vais désormais me pencher sur le swingworker qui semblerait etre une manière plus "propre" de travailler.

Un gros merci pour le temps que tu m'as consacré et pour ton aide.