[debutant] effet mémoire des boutons swing.

Version imprimable

19/05/2006, 12h44
Bighobbit

[debutant] effet mémoire des boutons swing.

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
19/05/2006, 12h47
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

Salut,

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

Code:

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

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:

1 2 3 4 JButton btn_next = new JButton (">"); GestionProchaineImage gestionProchaineImage = new GestionProchaineImage (); btn_next.addActionListener(gestionProchaineImage);
et le code de l'actionListener
Code:

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.

19/05/2006, 13h29
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()).

Citation:

Envoyé par mavina

Salut,

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

Code:

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:

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 :)

19/05/2006, 15h29
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

Citation:

Envoyé par 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

*

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

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

Soit tu fais :

Code:

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:

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:

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

Bon courage

19/05/2006, 16h11
xavlours

Citation:

Envoyé par 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.

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.
19/05/2006, 16h15
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
19/05/2006, 16h23
xavlours

NON ! :nono: 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é.

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:

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.

19/05/2006, 16h54
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:

((JButton)contexte.getSource()).setEnabled(false);
19/05/2006, 17h22
®om

Citation:

Envoyé par 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.

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...
19/05/2006, 17h50
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
19/05/2006, 18h20
xavlours

Citation:

Envoyé par Bighobbit

Formidable, ca fonctionne bien maintenant...

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

et voilou

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

Code:

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();
 }
}

19/05/2006, 19h02
®om

Citation:

Envoyé par 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

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)...