Discussion :

[Heavy Metal Hero]

Mon programme boucle sur une liste de JButton. Il repeint chaque bouton toutes les secondes

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
29
30
31
32
33
34
35
    @Override
public class a {
    static JFrame frame = new JFrame();
    static Set<JButton> components = new HashSet<JButton>();
 
    public static void main(String[] args) {
        JPanel lPanel = new JPanel();
 
        for (int i = 0; i < 15; i++) {
            JButton lButton = new JButton("Hello" + i);
            lPanel.add(lButton);
            components.add(lButton);
        }
        frame.add(lPanel);
        frame.pack();
        frame.setVisible(true);
        blinking();
    }
 
    public static void blinking() {
        Timer blinkTimer = new Timer(1000, new ActionListener() {
            boolean on = false;
 
            public void actionPerformed(ActionEvent e) {
                for (JButton lButton : components) {
                    // blink the buttons background on and off
                    lButton.setBackground(on ? Color.DARK_GRAY : Color.LIGHT_GRAY);
                    lButton.setForeground(on ? Color.WHITE : Color.BLACK);
                }
                on = !on;
            }
        });
        blinkTimer.start();
    }
}

Le problème n'apparaitra peut-être pas en exécutant cette exemple, car les repaint() fonctionnent bien sur les petits programmes. Mon problème se vérifie quand la machine rame

Je voudrais que la couleur background et forground de chaque bouton soit mise à jour en même temps.

JButton hérite de JComponent, et JComponent.setBackground() appelle repaint(), même chose pour JComponent.setForeground.

Par conséquent, chaque seconde, mon programme appelle repaint() deux fois pour chaque bouton, et les couleurs ne sont pas toujours mises à jour en même temps.

Comment puis-je faire en sorte que toutes les couleurs soient mises à jour en même temps ?

Merci de votre aide

[joel.drigo]

Salut,

C'est étonnant ce que tu dis. C'est difficile de juger vu que tu nous dis toi-même que le code que tu montres ne correspond pas ton implémentation.
Les invocations de repaint génèrent des évenements traités par la boucle évenementielle de l'EDT qui fait d'ailleurs un traitement pour limiter les repaints successifs trop fréquents. A l'inverse, si des évenements de repaint sont générés par d'autres composants concurament, on peut avoir des déclenchements de rafraîchissement entre 2 changements sur le bouton, un le background, un sur le forground, Pendant l'exécution de ta boucle sur les boutons, l'EDT ne fait rien, donc aucun rafraîchissement. Si tu as des rafraîchissements intermédiaires, ou des délais entre deux repaints, c'est à priori que l'EDT est libre pour les exécuter et changer l'affichage. Ils peuvent venir néanmoins du système, pas de tes ordres de repaint() (enfin ceux implicitement fait par les setBackground()/setForeground()).
Enfin, tout ça c'est de la théorie mais sans le programme réel, difficile de savoir d'où vient le problème, et si c'est bien le problème que tu décris. Surtout de la définition de rame : normalement, dans une application avec interface graphique les process longs ne devraient pas interagir avec l'UI avant la fin de leur exécution, donc ne pas être bloquant pour l'UI.

Cependant, restons dans la théorie.

Essaye de forcer un repaint après les deux changements de couleurs:

Code :

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

1
2
3
4
5
6
7
8
 
for (JButton lButton : components) {
                    // blink the buttons background on and off
    lButton.setBackground(on ? Color.DARK_GRAY : Color.LIGHT_GRAY);
    lButton.setForeground(on ? Color.WHITE : Color.BLACK);
    lbutton.repaint();
} 
on = !on;

Moi ça fonctionne avec 10000 boutons. Mais ce n'est peut-être pas à cause du nombre de boutons que ton programme rame.

puis essaye sur la frame (ou le panel)

puis tente des setIgnoreRepaint()...

Code :

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

1
2
3
4
5
6
7
8
9
frame.setIgnoreRepaint(true);
for (JButton lButton : components) {
                    // blink the buttons background on and off
    lButton.setBackground(on ? Color.DARK_GRAY : Color.LIGHT_GRAY);
    lButton.setForeground(on ? Color.WHITE : Color.BLACK);
}
frame.setIgnoreRepaint(false);
frame.repaint();
on = !on;

la doc de setIgnoreRepaint est un peu trompeuse puisqu'elle dit suspendre le repaint pour des évenements système. Donc on pense a des évenements que l'OS pourrait envoyer. Mais en fait il s'agit des repaints (implicites ou explicites), par opposition aux appels de méthode directement sur le composant (paint/paintImmediatly...tout ça).
Essayer de passer à du double ou triple buffering ensuite (voir JFrame.createBufferStrategy()), pour éviter le flickering.

Si tout ça ne fonctionne pas, il faudra envisager d'utiliser des boutons customs plutôt que des boutons Swing qui sont relativement lourds. Voir de procéder plus bas niveau, par dessin direct dans un buffer (en double buffering par exemple). S'il s'agit d'un clignotement en continu, peut-être même un simple gif animé en fond de bouton fera l'affaire (qu'on produira dynamiquement, un peu en simulant ce que fait une JTable pour faire son rendu).

[Heavy Metal Hero]

Si on change mon code pour mettre 500 boutons et un gridLayout pour les afficher tous, le problème est reproductible:


La solution proposée avec le setIgnoreRepaint est la bonne, merci !

Code :

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

1
2
3
4
5
6
7
8
9
frame.setIgnoreRepaint(true);
for (JButton lButton : components) {
                    // blink the buttons background on and off
    lButton.setBackground(on ? Color.DARK_GRAY : Color.LIGHT_GRAY);
    lButton.setForeground(on ? Color.WHITE : Color.BLACK);
}
frame.setIgnoreRepaint(false);
frame.repaint();
on = !on;