Renderer dans JTable

Version imprimable

Bonjour,

Je post rarement mais là je bloque depuis un moment et ça m'exaspère.

Je dois modifier l'affichage des cellules d'une colonne. J'utilise donc un DefaultTableCellRenderer.

Le problème, tout simple, absolument rien ne se passe...
Voilà le code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 setDefaultRenderer( String.class, new DefaultTableCellRenderer(){ @Override public Component getTableCellRendererComponent ( JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column ) { Component compo = getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column); AbstractTable.IColonne col = getColonneAt(column); Data lignePb = getRowAt(row); JLabel label = new JLabel(); label.setOpaque(true); label.setBackground(compo.getBackground()); label.setText( "text" ); return label; } });
Ca ne fonctionne pas non plus avec Object.class au lieu de String.class.
Quelque chose doit m’échapper dans le fonctionnement, mais quoi ?

Merci :)

25/07/2011, 16h54
le y@m's

Tu as oublié un "super." à la ligne :

Code:

Component compo = getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);

Parce que là tu as un appel récursif qui ne s'arrête jamais.

Code:

Component compo = super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
25/07/2011, 17h00
Logan Mauzaize

Il faut également vérifier la classe affectée à la colonne (lien)
25/07/2011, 17h10
Fyvan

Merci à vous deux pour vos réponses.

Cependant, cela ne change rien, aucun traitement n'est appliqué à ma Table...

Ma colonne est bien typé String...
26/07/2011, 08h55
Logan Mauzaize

Je t'invite donc à nous poster un bon ECM (voir ma signature ;))

essayez avec une seule colonne :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
table.getColumnModel().getColumn(0).setCellRenderer( new DefaultTableCellRenderer(){
 
	@Override
	public Component getTableCellRendererComponent ( JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column )
	{
 
		Component compo = getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
 
		AbstractTable.IColonne col = getColonneAt(column);
		Data lignePb = getRowAt(row);
 
		JLabel label = new JLabel();
		label.setOpaque(true);
		label.setBackground(compo.getBackground());
 
 
		label.setText( "text" );
 
		return label;
	}
});

si ça n'a pas marcher peut être que vous n'avez pas utiliser un AbstractTableModel ou indiquer qu'il s'agit de colonnes typés String :

Code:

1
2
3
4
5
 
public Class getColumnClass(int columnIndex){
 
            return String.class;
	}

Salut,
je ne suis pas un expert, mais j'ai une bonne expérience en matière de JTable.
Bon, je vois tout de suite une chose ... c'est qu'il n'y a pas de constructeur.
Peut-être qu'il y a moyen de faire sans, toujours est-il que ce code marche très bien chez moi :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 // imports public class InfosCellRenderer extends DefaultTableCellRenderer { ImageIcon icon; public InfosCellRenderer(){ super(); this.setText(""); icon = new ImageIcon("Organizator_Ressources/medias/images/prog/screenActivity/infos.png"); this.setIcon(icon); } @Override public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) { this.setToolTipText(value.toString()); return this; } }
le JLabel est initialisé dans le constructeur, et je fais un setIcon dans le constructeur aussi, pour pas le faire à chaque demande ;)

Donc essaye avec une constructeur, ça devrait marcher ... perso, je n'ai jamais fait ce genre de CellRenderer "à la volée", c'est pour ça que je ne m'avance pas plus

30/08/2011, 10h46
Logan Mauzaize

L'attribut "icon" est utilisé comme une variable et tu n'appelles pas le code la méthode parente.
30/08/2011, 14h22
Dominique49

je suppose que tu t'adresses à moi puisque que l'icone est dans mon exemple ...

euh ... c'est vrai que j'ai bricolé un peu, mais logiquement c'est plus rapide que si tu fait un setIcon à chaque fois.
Dans mon exemple, l'icône est instanciée au début ... puis on renvoi toujours la même, sans gaspiller de ressource en refaisant un setIcon ...

enfin, n'hésite pas à me dire si cela présente des inconvénients ;)

Merci.

Comme ça c'est mieux :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
// imports
 
public class InfosCellRenderer extends DefaultTableCellRenderer {
	//TODO Supprimes l'attribut non utilisé ImageIcon icon;
	public InfosCellRenderer(){
		super();
		this.setText("");
                //TODO Remplaces l'attribut par une variable
		ImageIcon icon = new ImageIcon("Organizator_Ressources/medias/images/prog/screenActivity/infos.png");
		this.setIcon(icon);
	}
	@Override
	public Component getTableCellRendererComponent(JTable table, Object value,	boolean isSelected, boolean hasFocus, int row, int column) {
                //TODO Appelles le code par défaut pour initialiser le renderer
                super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column)
		this.setToolTipText(value.toString());
		 return this;
	}
}

Le problème c'est pas le constructeur mais le fait d'ajouter un attribut qui n'est pas utilisé en tant que tel.
D'autant plus qu'il existe déjà une propriété pour cela !

Et pour être vraiment propre

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
// imports
 
public class InfosCellRenderer extends DefaultTableCellRenderer {
    private ImageIcon icon;
    public InfosCellRenderer(){
        super();
        icon = new ImageIcon("Organizator_Ressources/medias/images/prog/screenActivity/infos.png");
 
    }
    @Override
    public Component getTableCellRendererComponent(JTable table, Object value,    boolean isSelected, boolean hasFocus, int row, int column) {
                //TODO Appelles le code par défaut pour initialiser le renderer
        JLabel l = (JLabel) super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
        l.setText("");
        l.setIcon(icon);
        return l;        
    }
}

En effet, il se peut que le composant renvoyé par super.getTableCellRendererComponent ait une configuration légèrement différente du composant courant (c'est en particulier le cas quand le L&F utilisé est Nimbus).

De fait il est préférable d'utiliser le composant retourné par cette méthode.

31/08/2011, 08h53
Logan Mauzaize

La classe DefaultTableCellRenderer renvoie toujours "itself". C'est la logique même des renderer de Swing, ce sont que des "stamp" (tampons).

De plus inutile d'appeler setIcon à chaque fois et si tu veux un text vide pourquoi ne pas passer une chaîne vide (ou null) à l'appel de la méthode parente ?
Ou mieux ne redéfinir que la méthode setValue afin qu'elle ne fasse rien.

En tout cas ca me semble étrange de ne rien faire avec la valeur.

Citation:

Envoyé par Nemek

De plus inutile d'appeler setIcon à chaque fois et si tu veux un text vide pourquoi ne pas passer une chaîne vide (ou null) à l'appel de la méthode parente ?
Ou mieux ne redéfinir que la méthode setValue afin qu'elle ne fasse rien.

En tout cas ca me semble étrange de ne rien faire avec la valeur.

d'abord, merci beaucoup pour tous ces éclairages ;)
Sinon, je fais quelque chose avec la valeur ! Je l'affiche en bulle :

Code:

this.setToolTipText(value.toString());

Sinon, d'accord avec toi pour la variable icon ...
par contre, pas tout à fait d'accord avec ceci :

Code:

super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column)

pour la bonne raison que ce code si je ne me trompe affiche ce qui est passé dans la variable value.
Or, moi je ne veux pas ça, je veux seulement que ça s'affiche en bulle.
Don pas besoin du super .... tu es d'accord avec ça ?

Autrement dit, il faudrait faire comme ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 // imports public class InfosCellRenderer extends DefaultTableCellRenderer { //TODO Supprimes l'attribut non utilisé ImageIcon icon; public InfosCellRenderer(){ super(); this.setText(""); //TODO Remplaces l'attribut par une variable ImageIcon icon = new ImageIcon("Organizator_Ressources/medias/images/prog/screenActivity/infos.png"); this.setIcon(icon); } @Override public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) { this.setToolTipText(value.toString()); return this; } }
c'est bien cela ?
Merci encore.

Citation:

Envoyé par Dominique49

d'abord, merci beaucoup pour tous ces éclairages ;)

De rien le forum est là pour ça :p

Citation:

Envoyé par Dominique49

Sinon, je fais quelque chose avec la valeur ! Je l'affiche en bulle :

Code:

this.setToolTipText(value.toString());

Je parlais des exemples fournis ;-)

Citation:

Envoyé par Dominique49

par contre, pas tout à fait d'accord avec ceci :

Code:

super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column)

pour la bonne raison que ce code si je ne me trompe affiche ce qui est passé dans la variable value.
Or, moi je ne veux pas ça, je veux seulement que ça s'affiche en bulle.
Don pas besoin du super .... tu es d'accord avec ça ?

Non car le super ne fait pas que ça, il gère également la font, le focus, etc.

Citation:

Envoyé par Dominique49

Autrement dit, il faudrait faire comme ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 // imports public class InfosCellRenderer extends DefaultTableCellRenderer { //TODO Eviter ce code crado { this.setIcon(new ImageIcon("Organizator_Ressources/medias/images/prog/screenActivity/infos.png")); } @Override public void setValue(Object value) { this.setToolTipText(value != null ? value.toString() : ""); } }
Citation:

Envoyé par Dominique49

c'est bien cela !

d'accord ...
en effet, j'avais remarqué que la cellule ne changeait pas d'aspect quand la ligne était sélectionnée. En fait, ça m'arrangeait sinon on n'aurais pas beaucoup vu l’icône qui est de couleur plutôt bleue :lol:

Juste au passage, es-ce qu'un super (et donc un setText("")) dans la méthode getTableCellRendererComponent prend des ressources ... ce serait logique ;)
Enfin, ça en prend plus que si on mais rien quoi, mais il faut ce qu'il faut !
En fait, j'ai beaucoup d'icône ... et ça m'arrange que ces cellules reste en blanc au focus, sinon ça ne serait pas sympa :)

Merci encore, et à bientôt.

NB
Citation:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 // imports public class InfosCellRenderer extends DefaultTableCellRenderer { //TODO Eviter ce code crado { this.setIcon(new ImageIcon("Organizator_Ressources/medias/images/prog/screenActivity/infos.png")); } @Override public void setValue(Object value) { this.setToolTipText(value != null ? value.toString() : ""); } }
je suppose que c'est getTableCellRendererComponent au lieu de setValueAt ? ;)
j'ai bien pensé à la condition pour le cas ou value serait égal à null, mais quand c'est le cas, aucune bulle ne s'affiche, c'est pour ça que j'ai rien mis.

a+

31/08/2011, 12h10
Logan Mauzaize

setValue est la méthode appelée par le DefaultTableCellRenderer pour gérer la valeur.

value.toString() te lancera une NullPointerException si value est null !

Tout code exécuté prend des ressources :) Mais sur quelques millions de lignes tu ne verras quasiment pas de différences et de toutes façons ca exploserait ailleurs !

Il y a également la bordure à gérer.
31/08/2011, 15h28
Dominique49

ok, merci beaucoup.