Discussion :

[Bighobbit]

Bonjour,

Arrêtez moi si je me trompe
J'ai l'impression que dans Swing (et peut etre ailleurs), les boutons ont un effet mémoire.

Exemple,
Mon appli à un bouton qui lance une action qui dure 2 secondes.
Je click 12 fois sur ce bouton, et me voila parti pour 24 secondes d'attente...

Je souhaiterai que tant qu'une action n'est pas terminée, les clicks successif soient "oubliés".
J'ai essayé en mettant un boulean dans l'action, mais ca ne semble pas fonctionner.


Avez vous déja été confronté à ca, ou avez vous des pistes pour résoudre mon probleme ?
Merci d'avance
Richard

[minus]

Un truc du genre tonBouton.setEnable(false) en debut de ta methode et tonBouton.setEnable(true) en final de ta methode.

J'ai jamais essayé mais ca doit pas être super dur a faire

[mavina]

Salut,

Comme l'a dit minus, un setEnable() suffit :

Code :

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

1
2
3
4
5
6
7
 
public void actionPerformed(ActionEvent e)
{
  monBouton.setEnable(false);
  /* tes opérations */
  monBouton.setEnable(true);
}

+

[Bighobbit]

ben euh non, ca ne fonctionne pas. Les clicks souris sont toujours mis dans une espèce de file d'attente, et sont exécutés un par un...
Peut etre que c'est tout simplement la methode qui ne va pas
Je vous joins un bout de code tout con.

Le code du bouton :

Code :

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

1
2
3
4
 
    JButton btn_next = new JButton (">");
    GestionProchaineImage gestionProchaineImage = new GestionProchaineImage ();
    btn_next.addActionListener(gestionProchaineImage);

et le code de l'actionListener

Code :

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

1
2
3
4
5
6
7
8
 
class GestionProchaineImage implements ActionListener {
 public void actionPerformed(ActionEvent contexte) {
   Fenetre_principale.panneau_navigation.btn_next.setEnabled(false);
   ProjetBD.chargement_image_suivante_dans_app();
   Fenetre_principale.panneau_navigation.btn_next.setEnabled(true);
 }
}

Merci pour ces premières réponses en tout cas.

[xavlours]

Bonjour,

la fonction : ProjetBD.chargement_image_suivante_dans_app(); n'utilise-t-elle pas un thread ? Dans ce cas, il faut attendre la fin du thread pour réactiver le bouton (méthode join()).

[®om]

Salut,

Comme l'a dit minus, un setEnable() suffit :

Code :

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

1
2
3
4
5
6
7
 
public void actionPerformed(ActionEvent e)
{
  monBouton.setEnable(false);
  /* tes opérations */
  monBouton.setEnable(true);
}

+

Ou plutôt:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public void actionPerformed(ActionEvent e) {
    new Runnable() {
        public void run() {
            /// tes actions qui durent longtemps
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    /// les mises à jour graphiques
                }
            });
        }
    }.start();
}

avec à la limite la désactivation du bouton si tu veux pas que ça se fasse plusieurs fois si tu cliques plusieurs fois... Mais de toute façon tu as toujours la main de cette manière

[Bighobbit]

En fait mon probleme n'est pas la durée du traitement, mais le fait que 12 clicks successif = 12 actions.
Désactiver le bouton lors du premier traitement ne semble pas avoir d'effet.

Ha oui, le Runnable démarre avec Run() et pas start(), je le dis pour d'autres qui se creuserait la tete en copiant le bout de code

plize hellpp

[yann2]

En fait mon probleme n'est pas la durée du traitement, mais le fait que 12 clicks successif = 12 actions.
Désactiver le bouton lors du premier traitement ne semble pas avoir d'effet.

Ha oui, le Runnable démarre avec Run() et pas start(), je le dis pour d'autres qui se creuserait la tete en copiant le bout de code

plize hellpp

*


Non, un Runnable ne se lance pas avec run...

C'est juste que Rom s'est un petit peu trompé...

Soit tu fais :

Code :

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

1
2
3
4
5
6
7
8
9
10
  new Thread() {
        public void run() {
            /// tes actions qui durent longtemps
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    /// les mises à jour graphiques
                }
            });
        }
    }.start();

soit :

Code :

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

1
2
3
4
5
6
7
8
9
10
  new Thread(new Runnable() {
        public void run() {
            /// tes actions qui durent longtemps
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    /// les mises à jour graphiques
                }
            });
        }
    }).start();

Toutefois je suis bien perplexe devant ce code et j'imagine que ceci devrait suffire :

Code :

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

1
2
3
4
5
6
 
SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    /// les mises à jour graphiques
                }
            });

Bon courage

[xavlours]

En fait mon probleme n'est pas la durée du traitement, mais le fait que 12 clicks successif = 12 actions.
Désactiver le bouton lors du premier traitement ne semble pas avoir d'effet.

Justement, si la méthode que tu appelles se contente de lancer un thread, le bouton sera désactivé puis réactivé aussitôt.
Sinon simplement encadrer le contenu de actionPerformed() par des setEnabled comme tu le fais devrait marcher.

[Bighobbit]

Donc j'oublis le thread (parfait, ca m'arrange dans un premier temps )

Non l'encadrement par des setenable() ne fonctionne pas chez moi.
J'ai l'impression (attention grosse boulette en approche), j'ai l'impression que l'ActionListener qui est attaché au bouton continu de mémoriser les clicks souris.
Ainsi une fois le bouton réactivé (donc à la fin du premier appel), les events sont dépilés et lancés.

arghh

[xavlours]

NON ! Ce n'est pas une pile d'évènements, c'est une file (bon ok, je pinaille).
Sinon c'est bien le principe. Si tu regardes la stackTrace d'une exception levée dans un actionPerformed, tu vois :
- le nom du thread (awt-event-queue-dispatcher thread ou dans le genre)
- des méthodes genre pumpEvent, dispatchEventForListeners, ... qui finissent par appeler ton actionPerformed.

Mais si le bouton est désactivé, normalement aucun ActionEvent n'est reçu dans la file. Essaie de mesurer le temps passé dans la méthode actionPerformed (avec System.currentTimeMillis()) pour voir combien de temps ton bouton reste désactivé.

[Bighobbit]

ca chipotte dur !

Ben disons qu'en 2 secondes je clicke 5 fois.
et chaque passage dans l'ActionPerfomed me prend environ 2850 Millisecondes.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
class GestionProchaineImage implements ActionListener {
 public void actionPerformed(ActionEvent contexte) {
  long date_debut = System.currentTimeMillis(); 
  Fenetre_principale.panneau_navigation.btn_next.setenabled(false);
  ProjetBD.chargement_image_suivante_dans_app();
  Fenetre_principale.panneau_navigation.btn_next.setenabled(true);
  long date_fin = System.currentTimeMillis();
  long temps = date_fin - date_debut;
  System.out.println("temps event : " + temps);
}
}

voilou.

[xavlours]

Bon, au moins, tu n'as pas de thread parasite. C'est au moins ça de réglé.
Mais ça n'en est que plus bizarre.

On sait jamais, essaie plutôt :

Code :

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

((JButton)contexte.getSource()).setEnabled(false);

[®om]

ca chipotte dur !

Ben disons qu'en 2 secondes je clicke 5 fois.
et chaque passage dans l'ActionPerfomed me prend environ 2850 Millisecondes.

Un passage dans ActionPerformed doit tres rapide...
Si tu as des actions longues a effectuer, il faut le faire dans un Thread... comme j'ai ecrit plus haut (en mettant Thread a la place de Runnable)...

Et donc ton passage dans actionPerformed ne durera que qqs millisecondes maxi...

[Bighobbit]

Formidable, ca fonctionne bien maintenant...

Donc effectivement,
- un passage rapide dans ActionPerformed,
- doit absoluement etre combiné avec un mon_bouton.setenable(false), avant le traitement.
- et le setenable(true) doit etre positionné non pas dans le ActionPerformed (car le bouton serait rendu dispo avant la fin du traitement), mais au contraire en fin du traitement.

Merci à tous.
Richard

[xavlours]

Formidable, ca fonctionne bien maintenant...

Qu'as-tu changé ? On peut voir le code de la méthode actionPerformed qui marche ?

[Bighobbit]

et voilou

Le dévérouillage du bouton à lieu dans le traitement (très important, sinon marche pas).

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
class GestionProchaineImage implements ActionListener {
 public void actionPerformed(ActionEvent contexte) {
  long date_debut = System.currentTimeMillis(); 
  Fenetre_principale.panneau_navigation.verouillage_app();
  //((JButton)contexte.getSource()).setEnabled(false);
  new Thread(new Runnable() {
     public void run() {
      chargement_image_suivante_dans_app();
     }
    }).start();
 }
}

[®om]

Formidable, ca fonctionne bien maintenant...

Donc effectivement,
- un passage rapide dans ActionPerformed,
- doit absoluement etre combiné avec un mon_bouton.setenable(false), avant le traitement.
- et le setenable(true) doit etre positionné non pas dans le ActionPerformed (car le bouton serait rendu dispo avant la fin du traitement), mais au contraire en fin du traitement.

Merci à tous.
Richard

Ouais, et ton mon_bouton.setEnabled(false) tu peux le faire avant de lancer le nouveau Thread...
Par contre le mon_bouton.setEnabled(true), il faut le mettre à la fin de ton traitement dans le Thread, mais en passant par Swing.invokeLater... Sinon c pas correct (même si ça marche globalement assez bien)...