Discussion :

[isoman]

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 :

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

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

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

[drhouse]

Je pense que ton erreur vient de :

Code :

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

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 :

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

setText(pourcentage)

et ça devrait soulager ton processeur !

[tchize_]

+1 le renderer n'a pas a tripatouiller le modèle de la table, il a déjà recu (value) la valeur à afficher

[isoman]

j'ai remplacé

Code :

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

data.setValueAt(pourcentage, row, 6);

par

Code :

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

setText(pourcentage)

, la charge du pros est normale mais le texte ne s'affiche plus !

[tchize_]

tu as bien mis le setText après l'appel super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column); ?

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

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

  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 :

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

   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++