Discussion :

[Hesiode]

Bonjour,

Voici monde code reduit à sa plus simple expression :

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
import java.awt.Dimension;
import javax.swing.JDialog;
import javax.swing.JPanel;
 
public class Indexation2 extends javax.swing.JDialog {
	public Indexation2(JDialog owner) {
		super(owner,"Indexation en cours...",true);
 
		setPreferredSize(new Dimension(600,400));
		jLabel3 = new javax.swing.JLabel();
        jLabel3.setText("...");
        JPanel messagePane = new JPanel();
        messagePane.add(jLabel3);
 
        pack();  
 
        setVisible(true);
		goIndexer();
	    setVisible(false);
	    dispose();
    }
 
    public void setLabel(String texte){
    	jLabel3.setText(texte);    	
    }
 
    public void goIndexer(){
		for(int i=1;i<100;i++){
			setLabel("Indexation de : "+String.valueOf(i));
			System.out.println("ici");
			try {
				Thread.sleep(100);
			} catch (InterruptedException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
		}
    }
 
    private javax.swing.JLabel jLabel3;
 
}

Je ne comprends pas pourquoi le rafraichissement ne se fait pas...

Merci de vos avis

[plopinou]

Première interrogation, pourquoi un Thread.sleep(100) dans une bouble ? C'est pour simuler un traitement long ?

En tout cas, ton traitement goIndexer prend du temps, il ne faut pas qu'il s'exécute dans le thread de Swing. Je te renvoie à la FAQ sur les traitements long avec Swing.

[Hesiode]

Oui tout à fait, c'est pour simuler.

Merci pour ton lien, je comprends mieux.

[visiwi]

En tout cas, ton traitement goIndexer prend du temps, il ne faut pas qu'il s'exécute dans le thread de Swing.

Cette méthode n'est pas exécutée dans le Thread de Swing. Du moins avec le code qui est donné.

[visiwi]

Salut,

Tu n'as pas ajouté ton panel à la fenêtre :

Code :

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

setContentPane(messagePane);

Ensuite, ta fenêtre est modal, ce qui fait que le code est bloqué suite à :

Code :

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

setVisible(true);

et ton traitement se lance une fois que l'on à fermé la fenêtre manuellement.

Soit tu rend ta fenêtre non modal et sa fonctionnera (mais bloquera ton appli), soit tu devras créer un thread (et le lancer) afin de ne pas bloquer ton appli.
De plus la mise à jour du texte du label devrait être faite dans l'EDT.

Tu peux regarder la FAQ GUI, il y a un article sur "pourquoi ma progressbar se rempli d'un seul coup" ou quelque chose du genre, ça t'aidera.
La classe SwingWorker permet aussi se genre de chose.