Discussion :

[Tiberizz]

Salut,

Je développe une petite appli swing avec des traitements assez long. J'utilise une jprogressbar pour indiquer l'avancement du traitement.
Tout fonctionne bien, je passe l'objet jprogressbar au thread qui fait les traitement et je lui demande de la rafraichir ... Le problème c'est que cette solution ne me parait pas propre du tout :-\ Des classes metier se retrouve à manipuler des objects swing ...

Je fait pê une crise de MVCite aïgue mais ça me gêne une peu maintenant que je voudrais utiliser autrement ces classes metier sans barre de progression.
Comment doit on faire ? Quelqu'un peut-il me donner un exemple ?

[fr1man]

C'est ton thread qui manipule tes objets swings, pas tes objets metiers, non ?

[Tiberizz]

Comment ça ?
Je créer un thread pour les traitements, et dans ce run je fait des appels jprogressbar.setValue().
Comment faire autrement ?

[bouye]

Tu peux utiliser le pattern Observer, plutot que MVC et oui si possible mieux vaut utiliser une Thread separee pour ta requete qui enverra des notifications a un observateur/listener qui lui mettra a jour la progress barre durant l'EDT a l'aide de SwingUtilities.invokeLater() et SwingUtilities.invokeAndWait()

De maniere simplifiee et un peu bidon :

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
 
public class Task {
  /** Gets the length (number of notifications, not time length) of the task depending on the task parameter.
  * @return A positive number, -1 is the task length is unknown/undefined.
  */
  public int getTaskLength() {
    ...
  }
 
  /** Do the task.
   * @param obs An optional observer that is interrested in receiving this task's notifications, may be <code>null</code>.
   */
  public void doTask(TaskObserver obs) { 
    ... 
    if ((obs != null) && (obs.hasBeenCanceled())) {
       return;
    }
    if (obs != null) {
      obs.stepDone();
      obs.setNote("Switching to next sub task.");
    }
    ...
  }
}
 
/** An implementation that only holds a <code>JProgressBar</code>.
 */
public class ProgressBarTaskObserver implements TaskObserver {
 private JProgressBar bar = ...
 
  ...
 
  /** @inheritDoc
   * <br>This implementation always returns <code>false</code> as we do not have a cancel button.
   */
  public boolean hasBeenCanceled() {
    return false;
  }
 
  /** @inheritDoc
   * <br>Increments the bar.
   */
  public void stepDone() {
    try {
      // Update at EDT and wait until update is done.
      SwingUtilities.invokeAndWait(new Runnable() {
         public void run() {
           bar.setValue(bar.getValue()+1);
         }
      });
    }
    // Silently consume exception.
    catch (Exception e) {
    }
  }
}

A noter que Swing dispose deja de ProgressMonitor que tu peux enrober dans ta propre classe d'observateur.

[fr1man]

Je disais juste qu'un Thread qui manipule des objets métiers et qui fait des mises à jour d'objets Swing (avec le invokeLater comme l'a dit bouye), ça ne me choque pas.
Je ne vois pas comment faire autrement d'ailleurs.

[iohack]

Question conne : il n'existe pas de nom pour une "sous-classe" entre métier et vue ?