Discussion :

[thibaut]

Bonjour à tous,

Je voudrais avoir des conseils pour le problème suivant.

J'ai crée une interface graphique, qui quand on appui sur un bouton valider lance un traitement assez long (ganération d'un fichier PDF). Or pour que l'interface ne se gêle pas, j'ai mis la génération de ce fichier dans un Thread lancé depuis l'interface graphique.

Or il peut arriver que la génération de se fichier se passe mal (le fichier xml ou xsl-fo n'est pas trouvé par exemple). Donc je voudrais afficher le status de la génération dans un JTextField de mon interface. Or le problème est que je dois savoir si le Thread lancé par mon interface s'est terminé correctement.

Alors comme dois-je procéder ?

Merci pour vos conseils.

[otsgd]

Le plus propre je crois est d'utiliser un listener.

En gros, ton interface écoute le traitement du Thread qui l'informe de son execution.

[thibaut]

Mais existe t-il déjà des listeners de Thread ?

[otsgd]

Mais existe t-il déjà des listeners de Thread ?

Tu fais ton propre Listener :

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
 
public class MonPanneau implements MyConversionListener {
 
   // ....
 
  private void lancerConversion() {
     ConversionHandler h = new ConversionHandler(this);
     new Thread ... {
        h.lanceConversion();
     }
  }
 
  public void maConversionEstFinie() {
     monTextField.setText("Conversion terminée");
  }
 
}

Et ton ConversionHandler prends en paramètre un MyConversionListener qu'il informe lorsque la conversion est terminée.

Code :

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

1
2
3
4
 
public interface MyConversionListener {
   public void maConversionEstFinie(); 
}

C'est plus clair ?

[thibaut]

C'est bon j'ai trouvé, la solution, il suffit de passer la référénce à mon JTextField dans mon Thread...

Merci à tous

[otsgd]

C'est bon j'ai trouvé, la solution, il suffit de passer la référénce à mon JTextField dans mon Thread...

Merci à tous

C'est pas très propre comme solution, avec un listener tu peux utiliser ta conversion à plusieurs endroits, là ce n'est pas le cas.

N'oublies pas de marquer résolu ce thread.

[thibaut]

Oui mais un listener m'oblige de créer un autre Thread pour écouter et mettre à jour le JTextField.

Oui chef je met résolu dès que quelqu'un peut éventuellement me donner un meilleure solution ...

PS: J'ai un peu été modérateur avant...

[otsgd]

Oui mais un listener m'oblige de créer un autre Thread pour écouter et mettre à jour le JTextField.

Oui chef je met résolu dès que quelqu'un peut éventuellement me donner un meilleure solution ...

PS: J'ai un peu été modérateur avant...

Et avec mes 33 messages, je pese pas lourd

C'est marrant je suis aussi à Chatillon.

Pour le listener, je ne vois pas pourquoi ca t'oblige à créer un Thread, que tu passes ton JTextField ou une Interface, ca revient au meme, l'interface est plus générique et peut servir dans d'autres cas.

[adiGuba]

Salut,


Tu pourrais utiliser la classe SwingWorker pour cela :

• pour Java 5.0 : https://swingworker.dev.java.net/
• java 1.4 et inférieur : http://java.sun.com/docs/books/tutor...ml#SwingWorker

Tu trouveras plein d'info sur cette classe dans le forum GUI et/ou AWT/Swing

a++

[thibaut]

[..]C'est marrant je suis aussi à Chatillon.

Cool

[®om]

Oui mais un listener m'oblige de créer un autre Thread pour écouter et mettre à jour le JTextField.

Euh, non pas vraiment...

Pourquoi faire cela?

Il suffit de mettre à jour par SwingUtilities.invokeLater(...) (tu es obligé de faire cela à partir de ton Thread pour respecter la règle du Thread unique d'interface graphique, ou alors tu utilises SwingWorker comme l'a dit adiGuba...)

[had35]

Cool

Euh, non pas vraiment...

[thibaut]

Merci pour vos réponses en tout cas.

J'ai utilisé le SwingWorker de adiGuba (le super-boss). Par contre le SwingWorker bloque mon bouton, est-ce normal ?

Merci

[adiGuba]

adiGuba (le super-boss).

N'exagéront rien !

Par contre le SwingWorker bloque mon bouton, est-ce normal ?

Comment tu l'utilises ? on pourrait voir ton code ?

a++

[thibaut]

Le voici:

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
        bGenerer.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {            
                final SwingWorker worker = new SwingWorker() {
 
                    protected Object doInBackground() throws Exception {
                        PdfGenerator generator = new PdfGenerator(); 
                        String result = "Ok";
                        try {
                            generator.run(xslFile.getText(), xmlFile.getText(), output.getText());
                        } catch (Exception e) {
                            result = e.getMessage();
                        }
 
                        return result;
                    }
                };
 
                worker.run();
 
                try {
                    message.setText((String) worker.get());
                }
                catch (Exception e1) {
                    message.setText(e1.getMessage());
                }
            }
        });

Et j'explicite un peu:

xslFile, xmlFile, output sont des JTextField qui réprésentent un fichier xsl, xml et un fichier de sortie (pdf).


PdfGenerator s'occupe de transformer le XSL en PDF avec FOP.

message est un JTextfield qui affiche ok ou l'erreur rencontrée.

Voilà

[adiGuba]

Justement tu l'utilises mal

Tout comme les threads tu ne dois pas appeller la méthode run() directement, sinon le traitement long s'exécutera dans le thread courant !!!

A la place il faut soit appeller la méthode execute() qui place la tâche dans une queue d'exécution (10 tâches à la fois max il me semble), soit passer directement le SwingWorker à un ExecutorService qui se chargera de l'exécuter selon d'autres critères...

De plus tu peux utiliser les Generics pour modifier le type de retour de get() et doInBackground(). Enfin le code a effecter dans l'EDT à la fin du traitement doit être dans la méthode done() du SwingWorker, ce qui donnerait dans ton cas (en bleu les principales différences) :

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
        bGenerer.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                new SwingWorker<String,Object>() {

                    protected String doInBackground() throws Exception {
                        PdfGenerator generator = new PdfGenerator();
                        String result = "Ok";
                        try {
                            generator.run(xslFile.getText(), xmlFile.getText(),
                                    output.getText());
                        } catch (Exception e) {
                            result = e.getMessage();
                        }

                        return result;
                    }
                    
                    @Override
                    protected void done() {
                        try {
                            message.setText(get());
                        } catch (Exception e1) {
                            message.setText(e1.getMessage());
                        }
                    }
                }.execute();
            }
        });

a++

[thibaut]

Merci pour te lumières monsieur le boss.

Ca marche miquel.

A+