Problème d'actualisation d'une JTable

Version imprimable

Bonjour,

Voilà, j'ai un problème d'actualisation dans un JTable...
Je m'explique :
J'ai une JTable qui possède un cell editor, un renderer et un model (les trois compris dans une seule et unique classe !
Code:

1 2 3 4 getTable().setCellEditor(getCtrTableHonoraireMedecin()); getTable().setDefaultEditor(Object.class, getCtrTableHonoraireMedecin()); getTable().setDefaultRenderer(Object.class, getCtrTableHonoraireMedecin()); getTable().setModel(getCtrTableHonoraireMedecin());
Dans ce CtrTableHonoraireMedecin, j'ai redéfinie les méthodes standards pour avoir un affichage personnalisé avec des labels et des listes.

Là où ça se complique, c'est que pour pouvoir selectionner des élements dans la liste (pour pouvoir les supprimer ou modifier), j'ai dû rendre les champs de liste (de la table) éditables.
En gros, ce qui se passe actuellement, c'est que lorsqu'on clique sur une des listes, elle est remplacée par une nouvelle qui est disponible en édition et sur laquelle on peut travailler (redéfinition des méthodes getTableCellEditorComponent(...) et getTableCellRendererComponent(...) ).

Là où mon problème apparait, c'est lorsque je selectionne une colonne non éditable alors qu'une colonne éditable était préalablement selectionnée (une liste quoi).
==> Dans ce cas, ma liste en édition est rendue invisible (redéfinition du stopCellEditing() ) mais la case reste vide après celà... La valeur 'par défaut' n'est pas remise après le stopCellEditing().
Par contre, si je selectionne une autre colonne éditable, le problème n'apparait pas.

Si quelqu'un a rencontré le problème ou a une idée de la provenance de celui-ci, merci de me l'indiquer !
Et en tout cas, merci d'avoir porté attention à mon problème !

22/03/2007, 17h36
adiGuba

Salut,

Déjà tu ne devrais pas mélanger le cell editor, le renderer et le model dans la même classe !!! A la rigueur le cell editor et le renderer cela pourrait aller même si je pense que c'est mieux de les séparer...

Ensuite je te conseille de te baser sur les classes existantes (AbstractCellEditor/DefaultCellEditor, DefaultTableCellRenderer et AbstractTableModel/DefaultTableModel) qui propose déjà pas mal de fonctionnalité...

Quand à ton problème exact... c'est dur de t'aider sans code... mais je suis presque sûr que le problème vient de ta "super-classe" qui fait tout ;)

a++

Merci pour ta réponse !

Si tout est dans la même classe, c'est que j'ai suivi un principe qui était déjà utilisé dans l'application et en fait j'ai surtout repris et adapté le code qui avait déjà été fait.

Citation:

Ensuite je te conseille de te baser sur les classes existantes (AbstractCellEditor/DefaultCellEditor, DefaultTableCellRenderer et AbstractTableModel/DefaultTableModel) qui propose déjà pas mal de fonctionnalité...

Pour les classes existantes, je me base déjà dessus. En effet, un grand nombre de classes ont étés implémentés pour 'faciliter' le codage des suivante et ainsi, il existe une classe CTRTable :

Code:

public class CTRTable extends javax.swing.table.AbstractTableModel implements javax.swing.table.TableCellRenderer, TableCellEditor

Ma classe étend donc celle-ci...

Enfin, pour les exemples de code, je peux éventuellement en mettre, mais le problème, c'est que (comme je l'ai dis plus haut) j'utilise des classes redéfinies et j'ai peur que les noms ne vous parlent pas (auquel cas la masse de code à laisser serait impressionnante)...

Je vais tout de même laisser cela :
Voilà les colonnes de ma table :

Code:

protected final static int DATE_HONORAIRE = 0, BENEFICIAIRE = 1, ACTES = 2, MONTANT_HONORAIRE = 3, FSES = 4, REGLEMENTS = 5;

Mes colonnes contenant des listes sont FSES et REGLEMENTS.

Le isCellEditable :

Code:

1
2
3
4
5
6
7
8
9
10
11
@Override
public boolean isCellEditable(int row, int col)
{
	ligneSelected = row;
	if ( col == FSES )
		return true;
	if ( col == REGLEMENTS )
		return true;
 
	return false;
}

Le getTableCellRendererComponent :

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
@Override
public Component getTableCellRendererComponent(JTable jTable, Object value, boolean selected, boolean hasFocus, int row, int column)
{
	if (value instanceof METHonoraire)
	{
		METHonoraire honoraire = (METHonoraire) value;
		switch (column)
		{
			case DATE_HONORAIRE :
			{
				...
			}
			case BENEFICIAIRE :
			{
				...
			}
			case ACTES :
			{
				...
			}
			case MONTANT_HONORAIRE :
			{
				...
			}
			case FSES :
			{
				IHMListeFseHonoraire retour = getListeFses(honoraire);
				return retour;
			}
			case REGLEMENTS :
			{
				IHMListeReglementsHonoraire retour = getListeReglements(honoraire);
				return retour;
			}
		}
	}
	return super.getTableCellRendererComponent(jTable, value, selected, hasFocus, row, column);
}

Le getTableCellEditorComponent :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
@Override
public Component getTableCellEditorComponent(JTable table, Object value, boolean isSelected, int row, int column)
{
	METHonoraire honoraire = (METHonoraire) value;
	if( column == FSES )
	{
		getListeFseEdition().setHonoraire(honoraire);
		getListeFseEdition().actualiser();
		getListeFseEdition().setVisible(true);
 
		return getListeFseEdition();
	}
	if( column == REGLEMENTS )
	{
		getListeReglementEdition().setHonoraire(honoraire);
		getListeReglementEdition().actualiser();
		getListeReglementEdition().setVisible(true);
 
		return getListeReglementEdition();
	}
	return super.getTableCellEditorComponent(table, value, isSelected, row, column);
}

Le stopCellEditing :

Code:

1
2
3
4
5
6
7
8
9
10
@Override
public boolean stopCellEditing()
{
	if ( getListeFseEdition().isVisible() )
		getListeFseEdition().setVisible(false);
	if ( getListeReglementEdition().isVisible() )
		getListeReglementEdition().setVisible(false);
 
	return true;
}

Getter de mes listes en édition :

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
48
49
50
51
52
53
54
55
 
private IHMListeReglementsHonoraire listeReglementEdition = null;
private IHMListeFseHonoraire listeFseEdition = null;
 
/**
 * Renvoie la liste des fse pour le mode édition. 
 */
private IHMListeFseHonoraire getListeFseEdition() 
{
	if ( listeFseEdition == null )
	{
		listeFseEdition = new IHMListeFseHonoraire(null){
			private final static long serialVersionUID = fr.crosstalk.infos.InfosCommun.serialVersionUID;
				@Override
			public void addNotify()
			{
				super.addNotify();
				addEcouteurListeFseHonoraire(getEcouteur());
			}
				@Override
			public void removeNotify()
			{
				super.removeNotify();
				removeEcouteurListeFseHonoraire(getEcouteur());
			}
		};
	}
	return listeFseEdition;
}
 
/**
 * Renvoie la liste des règlements pour le mode édition. 
 */
private IHMListeReglementsHonoraire getListeReglementEdition() 
{
	if ( listeReglementEdition == null )
	{
		listeReglementEdition = new IHMListeReglementsHonoraire(null){
			private final static long serialVersionUID = fr.crosstalk.infos.InfosCommun.serialVersionUID;
				@Override
			public void addNotify()
			{
				super.addNotify();
				addEcouteurListeReglementsHonoraire(getEcouteur());
			}
				@Override
			public void removeNotify()
			{
				super.removeNotify();
				removeEcouteurListeReglementsHonoraire(getEcouteur());
			}
		};
	}
	return listeReglementEdition;
}

Et Enfin les getters pour les liste en affichage :

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
private IHMListeReglementsHonoraire listeReglementEdition = null;
private IHMListeFseHonoraire listeFseEdition = null;
 
/**
 * Renvoie la liste de fse de l'honoraire en paramètres
 */
private IHMListeFseHonoraire getListeFses(METHonoraire honoraire)
{
	if ( listeFses == null )
		listeFses = new HashMap<METHonoraire, IHMListeFseHonoraire>();
 
	if ( !listeFses.containsKey(honoraire) )
	{
		IHMListeFseHonoraire liste = new IHMListeFseHonoraire( honoraire );
		listeFses.put(honoraire, liste);
	}
 
	return listeFses.get(honoraire);
}
/**
 * Renvoie la liste de reglements de l'honoraire en paramètres
 */
private IHMListeReglementsHonoraire getListeReglements(METHonoraire honoraire)
{
	if ( listeReglements == null )
		listeReglements = new HashMap<METHonoraire, IHMListeReglementsHonoraire>();
 
	if ( !listeReglements.containsKey(honoraire) )
	{
		IHMListeReglementsHonoraire liste = new IHMListeReglementsHonoraire( honoraire );
		listeReglements.put(honoraire, liste);
	}
 
	return listeReglements.get(honoraire);
}

Voilà pour le code...
Merci d'avoir lu jusque là et de bien vouloir m'aider !

23/03/2007, 10h29
adiGuba

Citation:

Envoyé par feasil

Pour les classes existantes, je me base déjà dessus. En effet, un grand nombre de classes ont étés implémentés pour 'faciliter' le codage des suivante et ainsi, il existe une classe CTRTable :

Code:

public class CTRTable extends javax.swing.table.AbstractTableModel implements javax.swing.table.TableCellRenderer, TableCellEditor

Non : tu ne te bases que sur la classe AbstractTableModel !

En implémentant les interfaces TableCellRenderer et TableCellEditor tu es obligé de réécrire toutes leurs méthodes... alors que cela n'aurait pas été le cas si tu avais hérité des classes Abstract/Default qui propose une implémentation de la plupart des méthodes...

Par exemple, le premier problème que je vois, c'est que ta méthode stopCellEditing() cache le composant ! Or il se trouve que tu ne devrais pas faire cela car les cellules des tableaux ne sont pas des composants "standards". Je veux dire par là qu'il ne sont pas réellement ajouter à la JTable, mais que la JTable se contente de les afficher...

La méthode stopCellEditing() devrait plutôt lancer un évènement si je ne me trompe pas...

Bref il serait plus simple d'utiliser les implémentations existante plutôt que de recoder toutes les méthodes...

Citation:

Envoyé par feasil

Si tout est dans la même classe, c'est que j'ai suivi un principe qui était déjà utilisé dans l'application et en fait j'ai surtout repris et adapté le code qui avait déjà été fait.

Oui mais pourquoi ? Je pense que le fait d'avoir une "grosse classe qui fait tout" est plus propice à augmenter le risque d'erreur...

a++
23/03/2007, 10h39
feasil

Citation:

Envoyé par adiGuba

Je veux dire par là qu'il ne sont pas réellement ajouter à la JTable, mais que la JTable se contente de les afficher...

Ca c'est interressant, je ne savais pas et ça explique certains bugs que j'ai eu...

En tout cas merci, je vais chercher de ce côté là !