Discussion :

[zorglubpok]

Bonjour à tous,

Comment bloquer un swingworker tant qu’un utilisateur n’a pas saisi et validé dans un Jdialog une information ? Je m’explique.

Je dois lancer un traitement qui prend du temps à partir d’une IHM Swing. Ce traitement réalise divers calculs et à la fin doit stocker le résultat dans un fichier qui sera écrit sur le disque.

Le nom du fichier qui contiendra le résultat est choisi par l’utilisateur dans un Jdialog.
Comme mon traitement est long, je veux profiter du temps que prendra l’utilisateur à rentrer dans le Jdialog le nom du fichier, et lancer en parrallèle le SwingWorker.

Mes questions :

• Comment bloquer le SwingWorker si l’utilisateur n’a pas encore saisi et validé le nom du fichier à utiliser au moment d'écrire le résultat ?
• Comment récupérer le nom que l’utilisateur aura saisi ?

J’ai bien pensé à utiliser un CountDownLatch dans MySwingWorker, mais dans ce cas je ne peux pas récupérer la valeur de filename.

Autre possibilité : utiliser une BlockingQueue, mais j’ai l’impression d’utiliser un marteau-pilon pour écraser une mouche, car il faudrait que je gère également l’envoi d’un fileName bidon pour terminer le traitement et fermer la queue.

Avez-vous une solution simple et efficace (j’utilise Java SE7) ?

Voici ci-dessous le squelette de code :

Le code de lancement à partir de mon IHM Swing :

Code :

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

1
2
3
4
5
6
7
 
    	protected void someButtonActionPerformed(ActionEvent e) {
            MySwingWoker woker = new MySwingWorker();
            Worker.execute;
            String fileName = JOptionPane.showInputDialog(this,
				"Enter the name of the file");
        }

et le code de MySwingWorker:

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
 
public class MyWorker extends SwingWorker<Void, Void> {
 
…
 
    @Override
    protected void doInBackground() throws Exception {
        doSomeBigComputation();
        WriteResultInFile();
    }
 
    protected void WriteResultInFile() {
 
        // Comment attendre ici ?	
        Path p = Paths.get(fileName);
        // Comment récupérer la valeur de fileName de manière propre ?
        try {
            Files.createFile(p);
        } catch (FileAlreadyExistsException e) {
            Logger.getLogger(HMyWorker.class.getName()).log(Level.WARNING, null, e);
        } catch (IOException e) {
            Logger.getLogger(MyWorker.class.getName()).log(Level.WARNING, null, e);
        }
    }
}

[sinok]

Autant demander à l'utilisateur de saisir le nom du fichier avant de lancer le SwingWorker. Le gain en temps sera vraiment minimal pour complexifier à ce point ton code.

C'est le genre de truc que l'on met dans la catégorie optimisation non nécessaire, tu ferais mieux de voir si tu n'as pas moyen d'optimiser le traitement en lui même.

[zorglubpok]

Ok promis, j'optimiserai le code de ma tâche longue
Mais elle sera vraiment longue et donc je me dis que j'ai tout intérêt à profiter de l'inertie/lenteur de l'utilisateur.
De plus la question m'intéresse d'un point de vue théorique.
Donc, j'ai hate de lire vos propositions

[joel.drigo]

Salut,

moi je mettrai juste une variable file dans MySwingWorker, avec une méthode pour le passer sa valeur appelée après la saisie, et le tester à la fin de ton process long, dans une boucle while, pour attendre qu'il passe à !=null.

de la même façon que tu vas appeler cancel sur ton worker si l'utilisateur clique annuler lors de la sélection du fichier

Code :

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

1
2
3
4
5
6
if ( file==null ) {
    // annulation
    myswingworker.cancel(true);
} else {
    myswingworker.setTargetFile(file);
}

et à la fin de ton traitement long, juste avant d'exploiter ton fichier :

Code :

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

while(file!=null && !isCancelled()) yield();

[zorglubpok]

Salut,

moi je mettrai juste une variable file dans MySwingWorker, avec une méthode pour le passer sa valeur appelée après la saisie, et le tester à la fin de ton process long, dans une boucle while, pour attendre qu'il passe à !=null.

de la même façon que tu vas appeler cancel sur ton worker si l'utilisateur clique annuler lors de la sélection du fichier

Code :

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

1
2
3
4
5
6
if ( file==null ) {
    // annulation
    myswingworker.cancel(true);
} else {
    myswingworker.setTargetFile(file);
}

et à la fin de ton traitement long, juste avant d'exploiter ton fichier :

Code :

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

while(file!=null && !isCancelled()) yield();

Ca revient à utiliser un CountDownLatch non ?

[joel.drigo]

oui, un CountDownLatch avec une valeur de compteur à 1, mais il faut quand même avoir une variable file en plus. La boucle d'attente sera probablement mieux gérée (plus proprement du moins) qu'avec un while (on pourrait utiliser des wait() et notify() aussi pour éviter de consommer du cpu à faire un while).

[zorglubpok]

oui, un CountDownLatch avec une valeur de compteur à 1, mais il faut quand même avoir une variable file en plus. La boucle d'attente sera probablement mieux gérée (plus proprement du moins) qu'avec un while (on pourrait utiliser des wait() et notify() aussi pour éviter de consommer du cpu à faire un while).

Pour la variable filename, on peut faire comme tu as dit et ensuite on décrémente le compteur

Code :

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

1
2
3
4
5
6
7
8
9
10
 
    protected void someButtonActionPerformed(ActionEvent e) {
        CountDownLatch latch = new CountDownLatch(1);
        MySwingWorker worker = new MySwingWorker<>(latch);
        worker.execute;
        String fileName = JOptionPane.showInputDialog(this,
                            "Enter the name of the file");
        worker.setTargetFile(fileName);
        latch.countDown();
    }

ça doit marcher, non ?

[joel.drigo]

oui ça marche comme ça, et je n'ai pas dit le contraire,

donc pour une solution claire pour tous :

Code :

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

1
2
3
4
5
6
7
protected void someButtonActionPerformed(ActionEvent e) {
           MySwingWoker worker = new MySwingWorker();
            worker.execute();
            String fileName = JOptionPane.showInputDialog(this,
				"Enter the name of the file");
            worker.setTargetFile(fileName);
}

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
 
public class MySwingWorker extends SwingWorker<Void, Void> {
 
     private String filename;
     private CountDownLatch latch;


     public MySwingWorker() {
         latch=new CountDownLatch(1);
     }

     public void setTargetFile(String filename) {

          this.filename=filename;
          latch.countDown();
          if ( filename==null ) {
             cancel(true);
          }
     }

…
 
    @Override
    protected void doInBackground() throws Exception {
        doSomeBigComputation();
        WriteResultInFile();
    }
 
    protected void WriteResultInFile() {
 
        latch.await();

        if ( filename==null ) {
            // annulation
            return;
        }

        // Comment attendre ici ?	
        Path p = Paths.get(fileName);
        // Comment récupérer la valeur de fileName de manière propre ?
        try {
            Files.createFile(p);
        } catch (FileAlreadyExistsException e) {
            Logger.getLogger(HMyWorker.class.getName()).log(Level.WARNING, null, e);
        } catch (IOException e) {
            Logger.getLogger(MyWorker.class.getName()).log(Level.WARNING, null, e);
        }
    }
}