progressBar et propertyChange ne réagissent pas!

**hazyscl** · 04/08/2008, 15h49

Bonjour!

Je me met (enfin!) aux GUI... et si j'ai pas mal galéré et pas mal appris, je suis tombée sur un os: lorsque je fais "avancer l'avancement" de ma simulation, mes messages de sortie ne sont pas tous écrits, et ma progressBar .... ne fait rien du tout!!

Voila en bref les lignes intéressantes de mon code :

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
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
public class Type277jGUI extends javax.swing.JFrame {
 
    { TOUS LES TRUCS CLASSIQUES AUXQUELS JAI PAS TOUCHE ET QUI MARCHENT }
 
    // L'action qui doit lancer la simulation
    private void SimulMouseReleased(java.awt.event.MouseEvent evt) {                                    
try {
     Object[] options = {"Lancer la simulation","Annuler"};
     int intHeureDebut = Integer.parseInt(seizeHeureDebut.getText());
     int intHeureFin = Integer.parseInt(seizeHeureFin.getText());
     int choix = ConfirmDialogBox.showOptionDialog(getContentPane(), 
             "La simulation peut durer jusqu'à "+(intHeureFin-intHeureDebut+2)+" minutes. Etes-vous sûr de vouloir continuer ?", 
             "Type 277 j - Confirmation", 
             ConfirmDialogBox.YES_NO_OPTION, 
             ConfirmDialogBox.WARNING_MESSAGE, 
             null, 
             options , 
             options[1]);
     if (choix == ConfirmDialogBox.YES_OPTION) {
         ProgressBar = new JProgressBar(0);
         ProgressBar.setValue(0);
         (new SimulationTask()).execute();
     }
} catch (Exception ex) {
  DeroulementTextField.append("\n\n"+ex.toString());
}
}             
 
 
    // Mon SwingWorker qui execute la simulation :
    private class SimulationTask extends javax.swing.SwingWorker<Void,String> implements PropertyChangeListener, ActionListener {
 
    @Override
    protected Void doInBackground() {
 
            try {
 
                        {... des trucs pas interessants}
 
                        publish("Recuperation des donnees...\n\n");
 
			Donnees donnees;
                        if (!isCancelled()) {
                            donnees = new Donnees(inputFile);
                            setProgress(1);
                            ProgressBar.setValue(getProgress());
                        }
                        else {
                            throw new InterruptedException();
                        }
 
 
 
{ ma simulation qui marche bien et ou les publish sont plutot bien répercutés, mais pas le progressbar... }
 
 
 
                        setProgress(71);
                        ProgressBar.setValue(getProgress());
                        publish("\nCopie du fichier de sortie...\n\n");
                        publish("    creation des parametres du tableur...\n");
 
			WorkbookSettings wkbkSet = new WorkbookSettings();
			wkbkSet.setSuppressWarnings(true);
 
                        setProgress(72);
                        ProgressBar.setValue(getProgress());
			publish("    ouverture du tableur type...\n");
 
                        Workbook outputtype = Workbook.getWorkbook(new File(outputTypeFile),wkbkSet);
 
                        setProgress(73);
                        ProgressBar.setValue(getProgress());
                        publish("    copie dans le tableur de résultats...\n");
 
                        WritableWorkbook output = Workbook.createWorkbook(new File(outputFile), outputtype, wkbkSet);
 
                        setProgress(74);
                        ProgressBar.setValue(getProgress());
                        publish("    fermeture du tableur type...\n");
 
                        outputtype.close();
 
                        setProgress(75);
                        ProgressBar.setValue(getProgress());
 
 
 
		//---------
		// Ecriture du fichier de sortie:
		//---------
 
                        publish("\nEcriture dans le fichier de sortie...\n\n");
 
                        WritableSheet outputSans = output.getSheet("Energie transmise sans solaire");
			WritableSheet outputAvec = output.getSheet("Energie transmise totale");
 
			HeureActuelle = intHeureDebut;
                        while (HeureActuelle<=intHeureFin) {
 
				// Affectation des la temperatures:
					for (int i=1; i<=2*donnees.NbEltsSolides-1; i++) {
						outputTemp.addCell( new jxl.write.Number(6+2*i, 3+HeureActuelle, temperature[i-1][HeureActuelle-intHeureDebut], outputTemp.getCell(6+2*i, 3+HeureActuelle).getCellFormat()));
					}
 
				// Affectation des energies transmises:
					// Solaire direct:
						outputAvec.addCell( new jxl.write.Number(6, 3+HeureActuelle, solaireDirect[HeureActuelle-intHeureDebut], outputAvec.getCell(6, 3+HeureActuelle).getCellFormat()));
					// Solaire diffus:
						outputAvec.addCell( new jxl.write.Number(9, 3+HeureActuelle, solaireDiffus[HeureActuelle-intHeureDebut], outputAvec.getCell(9, 3+HeureActuelle).getCellFormat()));
					// Radiatif:
						outputAvec.addCell( new jxl.write.Number(12, 3+HeureActuelle, radiatif[HeureActuelle-intHeureDebut], outputAvec.getCell(12, 3+HeureActuelle).getCellFormat()));
					// Convectif:
						outputAvec.addCell( new jxl.write.Number(14, 3+HeureActuelle, convectif[HeureActuelle-intHeureDebut], outputAvec.getCell(14, 3+HeureActuelle).getCellFormat()));
					// Transfert de masse:
						outputAvec.addCell( new jxl.write.Number(16, 3+HeureActuelle, transfert[HeureActuelle-intHeureDebut], outputAvec.getCell(16, 3+HeureActuelle).getCellFormat()));
 
				// Affectation des energies sans solaire incident:
					// Radiatif:
						outputSans.addCell( new jxl.write.Number(6, 3+HeureActuelle, radiatifSansSolaire[HeureActuelle-intHeureDebut], outputSans.getCell(6, 3+HeureActuelle).getCellFormat()));
					// Convectif:
						outputSans.addCell( new jxl.write.Number(8, 3+HeureActuelle, convectifSansSolaire[HeureActuelle-intHeureDebut], outputSans.getCell(8, 3+HeureActuelle).getCellFormat()));
					// Transfert de masse:
						outputSans.addCell( new jxl.write.Number(10, 3+HeureActuelle, transfertSansSolaire[HeureActuelle-intHeureDebut], outputSans.getCell(10, 3+HeureActuelle).getCellFormat()));
 
                                HeureActuelle++;
                                setProgress(getProgress() + (int) (10*HeureActuelle/nbHeures));
                                ProgressBar.setValue(getProgress());
 
			}
 
 
		//---------
		// Termine:
		//---------
 
                        publish("\nCompilation du fichier de sortie...\n\n");
 
                        setProgress(97);
                        ProgressBar.setValue(getProgress());
 
			output.write();
			output.close();
 
            }
            catch (Exception ignore) {
                publish(ignore.getCause().toString());
            }
 
            return null;
 
    }
 
 
    @Override 
    protected void process(java.util.List<String> etapes) {
        String etape = etapes.get(etapes.size() - 1);
        Message.setText(etape);
        DeroulementTextField.append(etape);
        return;
    }
 
 
    @Override
    public void done() {
        String etape = "\n\nSimulation terminee !";
        Message.setText(etape);
        DeroulementTextField.append(etape);
 
        Object[] options = {"Ok"};
        int choix = DoneBox.showOptionDialog(getContentPane(), 
             "Simulation terminee !", 
             "Type 277 j - Fin de la simulation", 
             ConfirmDialogBox.YES_NO_OPTION, 
             ConfirmDialogBox.INFORMATION_MESSAGE, 
             null, 
             options , 
             options[0]);
        return;
    }
 
 
    @Override
    public void propertyChange(PropertyChangeEvent evt) {
            int progress = this.getProgress();
            ProgressBar.setValue(progress);
        return;
    }
 
 
    @Override
    public void actionPerformed(java.awt.event.ActionEvent evt) {
        if ("CancelButton" == evt.getSource()) {
            this.cancel(false);
        }
        return;
    }
 
}

En fait, jai bien l'impression que ma copie, écriture et compilation de tableur Excel n'a pas lieu. En tous les cas, je n'arrive pas à ouvrir mon fichier de résultats alors qu'avant de mettre tout ce code dans un SwingWorker tout marchait tres bien..... donc premier pepin

le deuxieme, c'est comment forcer la récupération de toutes les modifications par publish pour que tout soit bien affiché? et comment faire afficher à la progressBar l'etat d'avancement que j'ai pourtant mis à jour manuellement?

j'espere que vous pourrez m'aider!

bien à vous,
Lydia

**visiwi** · 04/08/2008, 16h16

Salut,

Tu n'utilise pas correctement la classe SwingWorker avec ton JProgressBar.

C'est la méthode setProgress() qui se charge de la mis à jour du JProgressBar, pas besoin de :

Code :

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

ProgressBar.setValue(getProgress());

Encore faut-il que tu lui indique une référence vers ce JProgressBar, ce qui n'est pas le cas.

Voici un pattern :

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
public class SearchTask extends SwingWorker<...> {
 
	public SearchTask(final JProgressBar bar) {
		if (bar != null) {
			addPropertyChangeListener(new PropertyChangeListener() {
				public void propertyChange(final PropertyChangeEvent evt) {
					if ("progress".equals(evt.getPropertyName())) {
						bar.setValue((Integer) evt.getNewValue());
					}
				}
			});
		}
	}
 
 
	...
 
 
}

Maintenant ta progress bar se mettra à jour avec un setProgress(x).

**hazyscl** · 04/08/2008, 16h51

yeah !!! ca marche ! ma progressBar repond, merci visiwi !

par contre, je n'ai toujours pas tous les publish(String...) répercutés par process(List<String> ...) donc, ils ne sont pas imprimés dans mon DeroulementTextField....
Je n'arrive toujours pas a lire le tableur resultant, donc je pense que les actions ne sont meme pas effectuées, et je n'arrive pas a savoir pourquoi ! Pourtant, si je lance le meme programme d'un autre fichier java ou je n'utilise pas le GUI, ca marche ?! et aucune action ne me ferait sortir du execute() a ce moment là....

Est-ce que tu aurais une idée pour m'aider?

Merci!!

Lydia

**visiwi** · 04/08/2008, 17h29

Si tu regarde ton code :

@Override
protected void process(List<String> etapes) {
String etape = etapes.get(etapes.size() - 1);
Message.setText(etape);
DeroulementTextField.append(etape);
return;
}

Tu n'affiche dans DeroulementTextField (surement un JTextArea) que le dernier String de la List (

je crois) :

String etape = etapes.get(etapes.size() - 1);

Il te faut prendre en compte toute la List :

Code :

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

1
2
3
4
5
6
@Override 
protected void process(List<String> etapes) {
        for(String s : etapes) {
                DeroulementTextField.append(s + '\n');
        }
}

**hazyscl** · 04/08/2008, 17h54

oui mais alors il va me réimprimer tous les messages que j'ai deja imprimés!

pourquoi le fait de publish un nouveau String ne modifie pas mes property, et n'invoque pas propertychange?

**visiwi** · 04/08/2008, 18h01

oui mais alors il va me réimprimer tous les messages que j'ai deja imprimés!

Non pas. La list contient les String qui n'ont pas encore été "publiés".

pourquoi le fait de publish un nouveau String ne modifie pas mes property, et n'invoque pas propertychange?

Comprends pas bien la question. Je crois que la réponse est : c'est pas fait pour ça. publish est typiquement fait pour mettre à jour un composant graphique avec des objet passé dans la List. Cela se fait dans l'EDT. La règle d'unicité est donc respecter sans s'en occupé. Cela peut être du texte dans un TextArea ou bien encore des noeuds dans un JTree.