Discussion :

[voyager75]

Bonjour tous le monde,

J'ai de nouveau un soucis. Promis j'ai vérifiié ce que je pouvais.

Mon problème c'est que les sélection dans ma JTable sont complètement délirantes et pire encore, absolument aléatoires.

Parfois la ligne est correctement sélectionnée, parfois la sélection va "prendre" une ligne ici + une ou deux cellules plus haut, ou plus bas.
Parfois aucune sélection ne se fait et parfois ce sont des cellules dispersées qui sont sélectionnées.

Bref, c'est un grand n'importe quoi.

Ce n'est qu'un problème d'affichage, car systématiquement, la ligne est bien sélectionnée (j'ai un formulaire avec les données de la ligne qui s'affiche correctement à coté).

Pourtant je veux une sélection tout ce qu'il y a de plus simple, une ligne à la fois.

Voici comment j'ai paramétré ma JTable

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	public MatosTab(final MatosModel md) {
		super(md);
		setDefaultRenderer(Object.class, new TabCellRenderer());
		getTableHeader().setReorderingAllowed(false);
		getTableHeader().setResizingAllowed(true);
		setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		setRowSelectionAllowed(true);
		setColumnSelectionAllowed(false);
		setCellSelectionEnabled(false);
		setShowVerticalLines(false);
		setShowHorizontalLines(false);
		setSelectionBackground(new Color(240, 216, 168));
		setAutoResizeMode(AUTO_RESIZE_OFF);

Merci

[sinok]

A mon avis le problème vient plus de ton Renderer qui ne marque pas forcément bien la sélection. Est-ce que le retour de la méthode getSelectedRows est correct?

[voyager75]

Le résultat est toujours correct.

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
setHorizontalAlignment(SwingConstants.LEFT);
setIcon(null);
setText("");
switch(col) {
	case 0:	setIcon(Define.matosType[((_Matos) getValueAt(row, 0)).getRef()].getIcone()); break;
	case 1:
	case 2:
	case 3:	super.getTableCellRendererComponent(table, val, isSelected, false, row, col); break;
	case 4:	setHorizontalAlignment(SwingConstants.RIGHT);
		setText(Define.EuroFormat.format(getValueAt(row, col))); break;
	case 5:
	case 6:
	case 7:	setText(((Long) getValueAt(row, col)) == 0? "":Define.dateFormat.
			format(((Long) getValueAt(row, col)))); }
return this;

[sinok]

Pour essayer de faire propre, il est préférable en général de s'appuyer sur le composant retourné par le super.getTableCellRendererComponent.

Dans ton cas ça donnerait ça (les cas 1,2,3 n'ayant plus vraiment d'utilité):

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
JLabel l = (JLabel)     super.getTableCellRendererComponent(table, val, isSelected, false, row, col);
 
l.setHorizontalAlignment(SwingConstants.LEFT);
l.setIcon(null);
l.setText("");
switch(col) {
    case 0:    l.setIcon(Define.matosType[((_Matos) getValueAt(row, 0)).getRef()].getIcone()); break;    
    case 4:    l.setHorizontalAlignment(SwingConstants.RIGHT);
        l.setText(Define.EuroFormat.format(getValueAt(row, col))); break;
    case 5:
    case 6:
    case 7:    l.setText(((Long) getValueAt(row, col)) == 0? "":Define.dateFormat.
            l.format(((Long) getValueAt(row, col)))); 
}
return l;

[voyager75]

Tu as raison, c'est certainement plus propre.
Je ferais comme ça dorénavant, merci.

Mais ça ne règle pas le problème
C'est toujours n'importe quoi lorsque je clique.

Je viens de relancer l'exécution une dizaine de fois.

Au dernier essai:
1er clic ligne 2, colonne 3, ok
2ème clic sur la ligne 2, colonne 3 et la:
la ligne 2 reste sélectionnée sauf la cellule colonne 3 et 4
et sélection de la colonne 3 et 4 de la ligne 3

Mais ce n'est pas un comportement constant.
Le plus souvent, aucune sélection n'apparait alors que mon formulaire se remplit correctement.

[sinok]

As tu essayé de reformuler ton case en if/else?

[voyager75]

Drôle d'idée

Code :

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

1
2
3
4
5
6
7
if(col == 0)
	laBel.setIcon(Define.matosType[((_Matos) getValueAt(row, 0)).getRef()].getIcone());
else if(col == 4) {
	laBel.setHorizontalAlignment(SwingConstants.RIGHT);
	laBel.setText(Define.EuroFormat.format(getValueAt(row, col))); }
else if(col > 4 && col < 8)
	laBel.setText(((Long) getValueAt(row, col)) == 0? "":Define.dateFormat.format(((Long) getValueAt(row, col))));

Ça ne pourrait pas venir d'ailleurs plutôt?

Voici 3 exemples d'un 1er et unique clic.

[sinok]

Pour l'instant je ne vois pas de piste probante, donc j'essaie de trouver un truc ...


sinon enlève ton appel à

Code :

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

setCellSelectionEnabled(false);

Il est en conflit avec le

Code :

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

setRowSelectionAllowed(true);

En effet la javadoc annonce la chose suivante:

This method is equivalent to setting both the rowSelectionAllowed property and columnSelectionAllowed property of the columnModel to the supplied value.

[voyager75]

Noté

J'ai essayé en vidant le constructeur du JTable

Code :

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

1
2
public MatosTab(final MatosModel md) {
	super(md); }

D'autres soucis, normal, mais même comportement.

[voyager75]

Dans ma JTable j'avais redéfinit valueChanged pour mettre à jour mon formulaire.
Et....j'avais tout simplement zappé l'appel à super.valueChanged(e)

[sinok]

Pour cela mieux vaudrait passer par un TableModelListener.

Aucun besoin de redéfinir la JTable.

[voyager75]

Je me suis mal exprimé.
En fait, je ne veux pas détecter de changement dans la JTable, rien n'est éditable et l'ajout/suppression d'enregistrement est géré par une méthode du modèle.
Je veux juste être prévenu lorsque l'utilisateur sélectionne une ligne, pour afficher le formulaire ad-hoc.

[sinok]

Dans ce cas il faut plutôt passer par un Listener sur le SelectionModel de la JTable (il faut jouter un ListSelectionListener sur le selection model de ta JTable)

[voyager75]

C'est bien ce que j'ai fais.
Ma JTable implémente ListSelectionListener et dans son constructeur j'ai

Code :

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

getSelectionModel().addListSelectionListener(this);

et donc cela me permet de redéfinir valueChanged.