surcharge du processeur a cause d'un cellrenderer!

Version imprimable

Salut,
j'ai étendue un DefaultTableCellRenderer pour que je puisse colorer les cellue de la table selon une condition ,ça fonctionne bien mais ça surcharge le processeur !
En enlevant le renderer tout rentre en ordre et en ajoutant un affichage dans le renderer j'ai remarqué qu'il est appelé tout le temps meme si ma table est inactive !
Que faire ?
Merci.

Code:

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 private class MyCellRenderer extends DefaultTableCellRenderer {
 
 
        private int L,  C;
        private DefaultTableModel data = null;
        private Integer stockFinal;
        private Integer stockMax;
 
        private DecimalFormat df;
        private Double pourcentageD;
        private String pourcentage;
 
        public MyCellRenderer(int ligne, int colonne) {
            super();
            L = ligne;
            C = colonne;
        }
 
        @Override
        public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {
            data = (DefaultTableModel) table.getModel();
 
 
            if (column == C) {
 
                stockFinal = (Integer) data.getValueAt(row, 2);
                stockMax = (Integer) data.getValueAt(row, 3);
 
                df = new DecimalFormat("########.00");
 
 
                pourcentageD = (stockFinal.doubleValue() / stockMax.doubleValue()) * 100;
 
                pourcentage = df.format(pourcentageD).toString() + "%";
 
                data.setValueAt(pourcentage, row, 6);
 
                if (stockFinal == 0) {
 
                    setBackground(noir);
                    setForeground(Color.WHITE);
 
                }  else {
                setBackground(Color.WHITE);
 
            }
            super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
            return this;
        }
    }

et voila comment je l'utilise:

Code:

jTableTest.setDefaultRenderer(String.class, new MyCellRenderer(0, 3));

12/05/2009, 22h20
drhouse

Je pense que ton erreur vient de :

Code:

data.setValueAt(pourcentage, row, 6);

tu mets à jour le modèle et cela doit mettre à jour la table. Cela a pour effet de rafraichir la table, qui va rappeler le renderer, etc. Ca boucle !

Si tu veux afficher, il te suffit de mettre à jour le composant graphique, inutile de mettre à jour le modèle...
Tu peux essayer de remplacer la ligne par :

Code:

setText(pourcentage)

et ça devrait soulager ton processeur !
12/05/2009, 22h25
tchize_

+1 le renderer n'a pas a tripatouiller le modèle de la table, il a déjà recu (value) la valeur à afficher
13/05/2009, 01h24
isoman

j'ai remplacé

Code:

data.setValueAt(pourcentage, row, 6);

par

Code:

setText(pourcentage)

, la charge du pros est normale mais le texte ne s'affiche plus !
13/05/2009, 08h39
tchize_

tu as bien mis le setText après l'appel super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column); ?
13/05/2009, 09h40
adiGuba
Salut,

Je rajouterais également quelques remarques concernant ton code :
- Les attributs L & C ne servent à rien : tu n'utilises pas L, et C peut être remplacé en appliquant le cellrenderer sur une colonne :
  
  Code:
  
  table.getColumnModel().getColumn(3).setCellRenderer(new MyCellRenderer());
- Les attributs data, stockFinal, stockMax, pourcentageD et pourcentage sont utilisé comme des variables locales, et change de valeur à chaque appel getTableCellRendererComponent(). Il serait préférable d'utiliser des variables locale !
- L'attribut df est lui un vrai attribut, qui pourrait rester pendant toute la durée de vie de l'instance. Mais tu l'utilises comme une variable locale :(
  Il serait préférable de le créer une fois pour toute :
  
  Code:
  
  private final DecimalFormat df = new DecimalFormat("########.00");
Enfin, quel est l'objectif de ton renderer ???
Car j'ai vraiment du mal à comprendre ce que tu veux faire !

a++
13/05/2009, 12h15
isoman

Salut,
l'objectif de mon rendere est de modifier la couleur d'une cellule selon sa valeur.
Effectivement le code n'est pas optimisé du tout mais bon c'est un c/c donc faut pas s'attendre un miracle.
Merci a tous .
14/05/2009, 00h08
drhouse

Citation:

Envoyé par adiGuba

Je rajouterais également quelques remarques concernant ton code :

D'accord avec tes remarques, mais s'il y a quelque chose à reprendre en priorité, à mon avis, c'est la conception... un renderer qui effectue des calculs, c'est mal ! même si c'est bien codé...