Discussion :

[Sharcoux]

Bonjour,

J'ai juste une petite question d'esthétisme :

Je voulais faire un cadre clignotant autour d'un bouton. N'étant pas trop habitué à faire des animations, j'ai bricolé un truc :

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
    private class Blinking extends Thread {
        private boolean continuer=true;
        @Override
        public void run() {
            while(continuer) {
                monBouton.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.RAISED, Color.red, Color.red.brighter()));
                try {
                    Thread.sleep(500);
                } catch (InterruptedException ex) {
                    Logger.getLogger(Loading.class.getName()).log(Level.SEVERE, null, ex);
                }
                monBouton.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.RAISED, ColorManager.transparent(), ColorManager.transparent()));
                try {
                    Thread.sleep(500);
                } catch (InterruptedException ex) {
                    Logger.getLogger(Loading.class.getName()).log(Level.SEVERE, null, ex);
                }
            }
        }
        public void arreter() {continuer=false;}
}

Je n'utilise pas stop() d'une part parce que l'instruction est dépréciée et d'autre part afin de m'assurer de l'état final de la bordure.

Questions :
- Est-ce que vous pensez que c'est un bonne façon de procéder ? Y a-t-il une façon plus conventionnelle ou plus propre ?
- NetBeans me met un Hint sur le "Thread.sleep(..);". Il y a une meilleure façon de faire ?

Merci

[professeur shadoko]

je ne réponds pas à la question posée ....
mais juste quelques détails qui sautent aux yeux

Code :

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

1
2
3
4
5
6
7
8
 
//nécessaire
private volatile boolean continuer = true ;
...
//facultatif
catch(InterruptedException e) {
  break ;
}

[Sharcoux]

Code :

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

1
2
3
catch(InterruptedException e) {
  break ;
}

Oui, c'est mieux

Code :

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

1
2
//nécessaire
private volatile boolean continuer = true ;

Bien vu.

Par contre, toujours aucun avis sur la forme ? Je dois comprendre que vous auriez procédé de cette façon ?

[thelvin]

Hello,

moi j'aurais utilisé un javax.swing.Timer. Ça fait la même chose mais ça évite de programmer soi-même Threads et sleep()s.

Mais bon, c'est des question de UI Java, ça, le forum dédié pourrait proposer des retours plus intéressants.

[Sharcoux]

moi j'aurais utilisé un javax.swing.Timer. Ça fait la même chose mais ça évite de programmer soi-même Threads et sleep()s.

Bon, je considère donc que je suis pas complètement à côté de la plaque.

Merci pour votre aide.

[forthx]

pour éviter d'attendre l'arrêt j'ajouterai :

Code :

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

1
2
 
public void arreter() {continuer=false; this.interrupt();}

et je préfère la mémorisation des bords pour ménager le GC aussi, mais c'est très personnel comme approche, globalement c'est du détail.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public void run() {
Border b[] = new Border[2];
b[0]=BorderFactory.createEtchedBorder(EtchedBorder.RAISED, Color.red, Color.red.brighter());
b[1]=BorderFactory.createEtchedBorder(EtchedBorder.RAISED, ColorManager.transparent(), ColorManager.transparent());
 
while(continuer) {
...
 
monBouton.setBorder(b[x]);

[Sharcoux]

J'aime bien. Je prends.

Merci

[tchize_]

Tu ne devrais pas modifier, surtout pas, ton Border dans ton Thread. Les composants swing ne sont pas Thread Safe. Soit tu utilise un timer, comme suggéré (du coup t'es dans le thread graphique de l'EDT), soit tu doit invoquer l'EDT pour tes modifications:

Code :

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

1
2
3
4
 while(continuer) {
                SwingUtilities.invokeAndWait(new Runnable(){ public void run() {
                    monBouton.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.RAISED, Color.red, Color.red.brighter()));
                    }});