[cellRenderer] problème de focus

Version imprimable

Salut,

j'ai un petit souci, j'ai fait un cellRenderer qui ne fonctionne pas comme je le veux j'ai dû faire une erreur quelque part mais je ne vois pas où, du coup je suis allé voir des tutos et la javadoc mais j'ai pas trouvé de solution.
Pourtant mon code est ultra simple.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 public class Montant_CellRenderer extends DefaultTableCellRenderer { public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column){ setHorizontalAlignment(SwingConstants.RIGHT); BigDecimal bdMtt = ActionUtils.parsetoBigDecimal(value); setText(value.toString()); if (bdMtt.signum() < 0){ setForeground(Color.red); }else{ setForeground(Color.black); } return this; } }
Ce qu'il fait : si le montant est négatif la couleur est rouge seulement quand on sélectionne la cellule.
Du coup j'ai essayé avec les hasFocus et isSelected dans mon if mais au lieu de n'afficher la valeur en rouge quand il est selectionné il affiche tout le temps en noir.

:merci:

21/06/2007, 15h35
grabriel

personne ne sait???

:help:

Et si tu appelais la méthode super() ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
class Montant_CellRenderer extends DefaultTableCellRenderer {
	public Component getTableCellRendererComponent(JTable table, Object value,
			boolean isSelected, boolean hasFocus, int row, int column) {
		super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
		setHorizontalAlignment(SwingConstants.RIGHT);
		BigDecimal bdMtt = ActionUtils.parsetoBigDecimal(value);
		setText(value.toString());
		if (bdMtt.signum() < 0) {
			setForeground(Color.red);
		} else {
			setForeground(Color.black);
		}
		return this;
	}
}

22/06/2007, 11h12
grabriel

:cry: ca change rien... le contenu de la cellule est rouge seulement si la cellule est sélectionnée, comme avant.

Merci pour cette réponse je commencais à désepérer.
22/06/2007, 11h48
Gueritarish

Salut,

Si j'ai bien compris ton problème, tu veux afficher la cellule en rouge si, quand elle est sélectionné, le montant est négatif.
Petit exemple explicatif de ce que j'ai compris:
Cell1 a un montant négatif
Cell1 est en noir au début --> sélection de Cell1 --> Cell1 passe en rouge
sélection de Cell2 --> Cell1 passe en noir

C'est ça?
22/06/2007, 12h42
grabriel

Non je veux afficher la cellule en rouge si le montant est négatif, que la cellule soit sélectionnée ou pas. Bref tout le temps rouge si montant < 0.

Mon problème c'est que ca affiche en rouge seulement quand je sélectionne.

Si je reprends ton exemple.

Ce que je veux :

Cell1 est négatif --> cell 1 est en rouge.

Ce que j'ai déjà et que je ne veux pas avoir :

Cell1 a un montant négatif
Cell1 est en noir au début --> sélection de Cell1 --> Cell1 passe en rouge
sélection de Cell2 --> Cell1 passe en noir
22/06/2007, 13h58
Gueritarish

Ok.
C'est toi qui rentres les valeurs des Cell où c'est l'utilisateur?

Tu peux poster ce code?

Citation:

Ok.
C'est toi qui rentres les valeurs des Cell où c'est l'utilisateur?

L'utilisateur

Citation:

Tu peux poster ce code?

Le code pour le cellEditor est au dessus en ce qui concerne la table :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 JXTable tableDetail = new JXTable(new SaisieAchatTableModel(ecrituresType)); tableDetail.getTableHeader().setReorderingAllowed(false); final JScrollPane scrollPane = new JScrollPane(tableDetail); tableDetail.setPreferredScrollableViewportSize(new Dimension(600, 100)); final GBC affTableDetail = new GBC(0,7,6,1).setInsets(10,0,0,0).setAnchor(GBC.CENTER); //pour aligner certaines colonnes à droite DefaultTableCellRenderer renderer = new DefaultTableCellRenderer(); renderer.setHorizontalAlignment(SwingConstants.RIGHT); //pour aligner à droite et afficher en rouge si négatif MontantCellRenderer montantRenderer = new MontantCellRenderer(); //applique le renderer aux colonnes tableDetail.getColumnModel().getColumn(SaisieAchatTableModel.MONTANT_COL).setCellRenderer(montantRenderer); tableDetail.getColumnModel().getColumn(SaisieAchatTableModel.QTE_COL).setCellRenderer(renderer); ...
Je construis une Jxtable toute bête avec un model, j'applique le cellrenderer dont tu as le code au dessus. J'ai vérifié je n'applique pas deux cellRenderer sur la même colonne :aie: (ca peux m'arriver des fois je suis fatigué), et je n'ai pas de renderer sur la table, rien qui fasse doublon avec le cellRenderer qui affiche en rouge.

22/06/2007, 14h55
Gueritarish

T'as essayé de faire un CellEditorListener et dans la méthode editingStopped() tu vérifies si le chiffre rentré est négatif, puis tu mets la cellule à rouge.
22/06/2007, 16h21
grabriel

J'ai cherché un peu mais j'ai pas trouvé d'exemple, ca marche comment le CellEditorListener?
22/06/2007, 16h30
Gueritarish

Bah dans la classe où tu fais la saisie des cellules, tu implémentes CellEditorListener avec les méthodes qui vont bien : CellEditorListener
Et dans la méthode editingStopped() (c'est à dire quand l'utilisateur a fini d'éditer la cellule) tu regardes si le montant est négatif et s'il l'est avec ton Renderer, tu passes la cellule à rouge..
Sans garantie, c'est pas testé et ça reste théorique ^^
Voilà

J'ai fait ca comme ca :

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
 
 
import java.awt.Color;
import java.awt.Component;
import java.math.BigDecimal;
 
import javax.swing.JTable;
import javax.swing.SwingConstants;
import javax.swing.event.CellEditorListener;
import javax.swing.event.ChangeEvent;
import javax.swing.table.DefaultTableCellRenderer;
 
 @SuppressWarnings("serial") public class MontantCellRenderer extends DefaultTableCellRenderer implements CellEditorListener{
 
	 BigDecimal bdMtt = new BigDecimal(0);
 
	 public Component getTableCellRendererComponent(JTable table,
	 Object value,
	 boolean isSelected,
	 boolean hasFocus,
	 int row,
	 int column){
		 super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
 
		 setHorizontalAlignment(SwingConstants.RIGHT);
 
		 BigDecimal bdMtt = ActionUtils.parsetoBigDecimal(value);
 
		 setText(value.toString());
 
		 this.bdMtt = bdMtt;
 
	 return this;
	 }
 
	public void editingCanceled(ChangeEvent arg0) {
 
	}
 
	public void editingStopped(ChangeEvent arg0) {
		if (bdMtt.signum() < 0){
			 setForeground(Color.red);
		 }else{
			 setForeground(Color.black);
		 }		
	}
}

Ca fait rien!!!!!!!:cry:
[EDIT]
Ca fait rien = montant positif ou négatif ca reste noir d'ailleurs et si je mets un system.out.println il n'affiche rien.

Dans la version actuelle (code tout en haut) si je rajoutais un system.out.println et qu'une cellule avait un montant <0 il affichait mais la couleur reste noir sauf si on sélectionne la cellule (sélectionne pas édite).
[/EDIT]

J'ai trouvé la réponse la : http://java.sun.com/docs/books/tutor...rRenderer.java

j'ai rajouté cette ligne là au commentaire très explicite, et maintenant ca fonctionne.

Citation:

setOpaque(true); //MUST do this for background to show up.

Merci à Gueritarish et mikemikemike pour votre aide.

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
public class MontantCellRenderer extends JLabel implements TableCellRenderer {
 
	 public MontantCellRenderer(){
	        setOpaque(true); //MUST do this for background to show up.
	 }
 
	 BigDecimal bdMtt = new BigDecimal(0);
 
	 public Component getTableCellRendererComponent(JTable table,
	 Object value,
	 boolean isSelected,
	 boolean hasFocus,
	 int row,
	 int column){
 
		 setHorizontalAlignment(SwingConstants.RIGHT);
 
		 BigDecimal bdMtt = ActionUtils.parsetoBigDecimal(value);
 
		 setText(value.toString());
 
		 if (bdMtt.signum() < 0){
			 setForeground(Color.red);
		 }else{
			 setForeground(Color.black);
		 }
	 return this;
	 }
}

25/06/2007, 14h07
Gueritarish

Mais c'est avec plaisir si j'ai pu t'aider ^^