Thread sleep dans une boucle

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:

---

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

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

Code:

---

1 2 3 4 5 6 7 8

//nécessaire private volatile boolean continuer = true ; ... //facultatif catch(InterruptedException e) { break ; }

---

Code:

---

1 2 3

catch(InterruptedException e) { break ; }

---

Oui, c'est mieux

Code:

---

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 ?

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.

Citation:

---

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.

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

Code:

---

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:

---

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]);

---

J'aime bien. Je prends.

Merci

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:

---

1 2 3 4

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

---