Renderer n'est jamais appelé + question

**Nafnlaus** · 20/08/2013, 13h55

Bonjour,

J'ai un renderer de JTable qui n'est pas appelé... J'ai déjà effectué ce genre de traitement dans le passé, sans problème, mais pas cette fois ça ne fonctionne pas.

Voici le code :

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
public MonPnl() {
 
        ArrayList<Affaire> alTest = new ArrayList<Affaire>();
        // Initialisation de alTest, les valeurs sont les bonnes
 
        tab = new JTable(new TabModel(alTest));
 
        tab.setDefaultRenderer(String.class, new TabRenderer());
        System.out.println("on a rajouté le renderer : " + tab.getDefaultRenderer(String.class).getClass().getName());// aucun souci ici, le sysout qui indique ce qu'il faut...
 
        this.add(new JScrollPane(tab));//juste un petit test, sans layout
    }
 
    private class TabModel extends AbstractTableModel {// aucun souci ici, les valeurs sont bonnes
    }
 
    private class TabRenderer extends DefaultTableCellRenderer {
 
        public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
        {
            System.out.println("on lance le renderer"); // n'apparaît jamais ...
            super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
            //traitement et analyse des données, pour la coloration, etc...
            return this;
        }
    }

Quelqu'un saurait-il m'indiquer comment résoudre ce problème ?

De plus, j'aimerais que certaines cases soient éditables (dans l'AbstractTableModel, pas de problème là-dessus) mais seulement en utilisant un menu déroulant. J'ai donc trouvé une méthode, qui fonctionne :

Code :

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

1
2
//initialisation de mon JComboBox
this.tab.getColumnModel().getColumn(noColumn).setCellEditor(new DefaultCellEditor(monComboBox));

Ma question est la suivante :
Si je fais ce traitement après :

Code :

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

tab.setDefaultRenderer(String.class, new TabRenderer());

Est-ce que cela ne va pas "annuler" l'effet du renderer ?

Merci d'avance pour votre aide.

**joel.drigo** · 20/08/2013, 14h19

Salut,

à priori je pense que tu n'a pas implémenté la méthode getColumnClass de ton modèle. Par défaut cette méthode renvoit Object.class pour toutes les colonnes.

Comme ton renderer est installé pour la classe String, il n'est jamais sélectionné.

Donc il faut que tu ajoutes à TabModel un truc dans le genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
@Override
public Class<?> getColumnClass(int columnIndex) {
     // la liste des index de colonnes dont le type est String (ici par exemple, 0 à 3
    switch (columnIndex) {
    case 0: 
    case 1:
    case 2:
    case 3:
       return String.class;
    default:
       return super.getColumnClass(columnIndex);
}
}

Les "CellEditor" ne remplacent le renderer que pendant l'édition de la cellule (quand elle a le focus il me semble, mais je n'en suis plus très sur).

**Nafnlaus** · 20/08/2013, 14h41

Oui, ça marche !

Code :

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

1
2
3
public Class<?> getColumnClass(int columnIndex) {
      return String.class;
}

Je vais essayé concernant les cellEditor, je vous tiens au courant.

Mais d'ailleurs, si jamais dans ce getColumnClass, je définis la colonne X en tant que JComboBox, ça ne fonctionnera pas ? Ou bien elle apparaîtra constamment sous forme d'un JComboBox, c'est ça ? je vais essayer...

Merci beaucoup déjà ! Je mettrai

dès que j'aurai réussi avec les combobox, et je posterai la solution.

**joel.drigo** · 20/08/2013, 14h43

Envoyé par Nafnlaus

Mais d'ailleurs, si jamais dans ce getColumnClass, je défini la colonne X en tant que JComboBox, ça ne fonctionnera pas ? Ou bien elle apparaîtra constamment sous forme d'un JComboBox, c'est ça ? je vais essayer...

Cette méthode sert à indiquer à la JTable les types des données pour chaque colonne, pas la façon de les afficher ou de les saisir.

La notion de jcombobox, à priori pour saisir une valeur sera gérée par le CellEditor.

**Nafnlaus** · 20/08/2013, 15h01

D'accord, merci beaucoup pour toutes ces informations.

Tous fonctionne à présent.

Cependant, d'un point de vue graphique, si la colonne des status indique R, alors le fond doit être vert, écrit en blanc, et si le statut est E, alors c'est en blanc sur fond rouge. Autrement noir sur fond blanc, normal.

J'ai donc ça :

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
public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
        {
            super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
 
            if(column == 10)//Statut
            {
                if(value == "R")
                {
                    this.setBackground(Color.GREEN);
                    this.setForeground(Color.WHITE);
                }
                else if(value == "E")
                {
                    this.setBackground(Color.RED);
                    this.setForeground(Color.WHITE);
                }
                else
                {
                    this.setBackground(Color.WHITE);
                    this.setForeground(Color.BLACK);
                }
            }
            else if(column == 20)//Démat possible
            {
                if(value == "OUI")
                {
                    this.setForeground(Color.GREEN);
                }
                else if(value == "NON")
                {
                    this.setForeground(Color.RED);
                }
            }
 
            if(table.getModel().getValueAt(row, 10).equals("E"))//si on a le statut excuse, on barre tout en rouge + on écrit en rouge
            {
                this.setForeground(Color.RED);
                //TODO barrer le texte
            }
 
            return this;
        }

Et le comportement est plus que bizarre... La ligne est en fond vert à partir de la colonne 10, et si on clique sur les en-têtes, tout devient vert.

Par contre si on sélectionne la ligne, elle est en bleu (normal, couleur par défaut, grâce au "super.get...", sauf la colonne 10 (normal puisque pour l'instant, je n'ai pas utilisé isSelected) qui est verte...

Je me demande donc pourquoi toutes les autres cases sont vertes passé la colonne 10 ? Elles ne vérifient pourtant pas le test :

Code :

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

if(column == 10)

je ne comprends pas...

**Nafnlaus** · 20/08/2013, 15h04

Correction qui a fonctionné, mais je ne sais pas pourquoi...

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
public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
        {
            super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
 
            if(!isSelected)//prise en compte de la sélection
            {
                if(column == 10)//Statut
                {
                    if(value == "R")
                    {
                        this.setBackground(Color.GREEN);
                        this.setForeground(Color.WHITE);
                    }
                    else if(value == "E")
                    {
                        this.setBackground(Color.RED);
                        this.setForeground(Color.WHITE);
                    }
                }
                else if(column == 20)//Démat possible
                {
                    if(value == "OUI")
                    {
                        this.setForeground(Color.GREEN);
                    }
                    else if(value == "NON")
                    {
                        this.setForeground(Color.RED);
                    }
                }
                else //déplacement du else
                {
                    this.setBackground(Color.WHITE);
                    this.setForeground(Color.BLACK);
                }
 
                if(table.getModel().getValueAt(row, 10).equals("E"))//si on a le statut excuse, on barre tout en rouge + on écrit en rouge
                {
                    System.out.println("la ligne en rouge");
                    this.setForeground(Color.RED);
                    //TODO barrer le texte
                }
            }
 
            return this;
        }

Le renderer garde en mémoire le dernier aspect graphique qu'il a créé ?

**joel.drigo** · 20/08/2013, 15h18

Regarde le code de la méthode getTableCellColumnRendererComponent(...) de DefaultTableCellRenderer :

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
 public Component getTableCellRendererComponent(JTable table, Object value,
                          boolean isSelected, boolean hasFocus, int row, int column) {
 
	if (isSelected) {
	   super.setForeground(table.getSelectionForeground());
	   super.setBackground(table.getSelectionBackground());
	}
	else {
	    super.setForeground((unselectedForeground != null) ? unselectedForeground 
	                                                       : table.getForeground());
	    super.setBackground((unselectedBackground != null) ? unselectedBackground 
	                                                       : table.getBackground());
	}
 
	setFont(table.getFont());
 
	if (hasFocus) {
            Border border = null;
            if (isSelected) {
                border = UIManager.getBorder("Table.focusSelectedCellHighlightBorder");
            }
            if (border == null) {
                border = UIManager.getBorder("Table.focusCellHighlightBorder");
            }
            setBorder(border);
 
	    if (!isSelected && table.isCellEditable(row, column)) {
                Color col;
                col = UIManager.getColor("Table.focusCellForeground");
                if (col != null) {
                    super.setForeground(col);
                }
                col = UIManager.getColor("Table.focusCellBackground");
                if (col != null) {
                    super.setBackground(col);
                }
	    }
	} else {
            setBorder(getNoFocusBorder());
	}
 
        setValue(value); 
 
	return this;
    }

On voit bien comment il faut gérer les attributs graphiques du renderer en fonction des différents arguments de la méthode et comment on doit les modifier le cas échéant.

Au début, il y a le traitement de la sélection : si tu mets une valeur de background et/ou de foreground sans prendre en compte isSelected, tu modifies le comportement standard de l'affichage de lignes sélectionnées. Il faut tenir compte donc de ce booléen en plus des index de colonnes et des valeurs. Idem un peu plus loin, si la cellule à le focus et si la cellule est éditable et si elle n'est pas sélectionnée.

A noter que c'est la même instance de renderer qui est utilisée à chaque fois et que l'appel de setForeground et setBackground "sauve" la couleur passée en argument, donc il faut que tu affectes les background et les foreground pour tous les cas : donc si tu as un truc comme tu as :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
else if(column == 20)//Démat possible
			{
				if(value == "OUI")
				{
					this.setForeground(Color.GREEN);
				}
				else if(value == "NON")
				{
					this.setForeground(Color.RED);
				}
			}

il faut toujours qu'il y ait un else final qui mets la valeur par défaut.

Le plus simple étant d'écrire un truc comme ça :

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
 
if( isSelected ) {
    // faire le cas des lignes sélectionnées
}
else {
Color foreGround = table.getForeground();
Color backGround = table.getBackground();
 
if(column == 10)//Statut
			{
				if("R".equals(value) )
				{
					foreGround = Color.GREEN;
					backGround  = Color.WHITE;
				}
				else if("E".equals(value))
				{
					background = Color.RED;
					foreground = Color.WHITE;
				}
				else
				{
					background = Color.WHITE;
					foreground = Color.BLACK;
				}
			}
			else if(column == 20)//Démat possible
			{
				if("OUI".equals(value))
				{
					foreground = Color.GREEN;
				}
				else if("NON".equals(value) )
				{
					foreground = Color.RED;
				}
			}
 
			if("E".equals(table.getModel().getValueAt(row, 10)))//si on a le statut excuse, on barre tout en rouge + on écrit en rouge
			{
				foreground = Color.RED;
				//TODO barrer le texte
			}
 
setForeground(foreground);
setBackground(background);
}

Renderer n'est jamais appelé + question

Composants Java

Vue hybride

Discussions similaires

Partager

Partager