les mystères de JTable.

Salut à tous,
Je me suis fait un petit exemple d'utilisation de JTable pour voir un peu comment cela fonctionnait. Mon but était de faire une JTable permettant l'ajout et la suppression de lignes.
J'ai donc implementé mon TableModel , un renderer pour mes boutons permettant la suppresion (eux meme contenu à la fin de chaque lignes) et un cell Editor.

Je me retrouve avec le code suivant:

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
 
 
 
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.GridLayout;
import java.awt.ScrollPane;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Vector;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.event.TableModelEvent;
import javax.swing.event.TableModelListener;
import javax.swing.table.DefaultTableCellRenderer;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.TableModel;
 
import fr.dyade.jdring.AlarmEntry;
 
/**
 * Class testTable
 * 
 * DESCRIPTION : Test d'une JTable
 * avec ajout et suppression de lignes dynamique.
 *
 */
 
 
public class testTable extends JFrame implements ActionListener
{
	// la JTable
	private JTable actionTable;
	// Le model de la table
	private ActionTableModel model;
	// une référence sur la liste d'alarm
	private ArrayList<AlarmEntry> alarmList;
	// Le panneau qui contiendra la JTable
	private JScrollPane scrollPane;
	// Ce bouton permet l'ajout
	private JButton addButton = new JButton("add");
 
 
 
	public testTable()
	{
		super();
 
		// Initialisation du model,
		// Instance de la classe ActioNt
		model = new ActionTableModel();
 
		// creation de la table
		actionTable = new JTable(model);
		/// Le scrollPane de la table
		scrollPane = new JScrollPane(actionTable);
 
		// le panneau contenant le bouton 
		// d'ajout
		JPanel panelButton = new JPanel();
		// Ajout du bonton dans le panel
		panelButton.add(addButton);
		// On autorise la selection d'une seule
		// ligne dans la table
		actionTable.setSelectionMode(0);
 
		// configuration du renderer pour les JButton
		actionTable.getColumnModel().getColumn(2).setCellRenderer(new JButtonRenderer());
		// l'Editor pour les JButton
		actionTable.getColumnModel().getColumn(2).setCellEditor(new JButtonCellEditor());
 
 
		// Layout du contentPane de la JFrame
		this.getContentPane().setLayout(new GridLayout());
 
		// ajout du scrollPane contenant la JTAble
		this.getContentPane().add(scrollPane);
		// ajout du panel contenant le JButton pour l'ajout 
		//de ligne dans la jtable.
		this.getContentPane().add(panelButton);
 
 
		// On ajoute le listener
		// au bouton
		addButton.addActionListener(this);	
 
	}
 
	public void addRow()
	{
		Vector row = new Vector();
		row.add("data");
		row.add(true);
		JButton b = new JButton("Delete");
		b.addActionListener(this);
		row.add(b);
 
		model.addRows(row);
	}
 
	public void actionPerformed(ActionEvent event) 
	{
		if(event.getSource() == addButton)
		{
			// le JButton qui envoie l'évenements est
			// le addButton
			System.out.println(" Ajout d'une valeur ");
			// On ajoute une ligne.
			addRow();
			// puis on sort de la fonction
			return;
 
		}
 
		if(event.getSource() instanceof JButton)
		{
			// Lorsque qu'un bouton de suppression est 
			// actionné
			System.err.println("ligne="+actionTable.getSelectedRow());
			if(model != null)
			model.removeRow(actionTable.getSelectedRow());
			else System.out.println("model =Null");
 
		}
	}
 
	public static void main(String[] args)
	{
		testTable table = new testTable();
		table.setVisible(true);
		table.setSize(400,300);
		table.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	}
 
}

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
 
 
 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Vector;
 
import javax.swing.JButton;
import javax.swing.JCheckBox;
import javax.swing.table.AbstractTableModel;
 
/**
 * Description: Le TAbleModel  
 */
 
public class ActionTableModel extends AbstractTableModel{
 
	private String[] nomsColonnes = {"ActionName","continuous","Next Run"};
	Class[]  dataClass = {String.class,Boolean.class,JButton.class};
 
	// Le vecteur qui contient les lignes
	// les lignes sont réprénté comme un vecteur 
	Vector<Vector> rows;
 
 
	public ActionTableModel()
	{
		rows = new Vector<Vector>();
 
	}
 
	// Ce constructeur prend en parametre un vecteur 
	// contenant les lignes du tableau
	public ActionTableModel(Vector rowsVector)
	{
		rows = new Vector(rowsVector);
	}
 
	public int getColumnCount() {
 
		return nomsColonnes.length;
	}
 
 
	public String getColumnName(int col)
	{
 
		return nomsColonnes[col];
	}
 
	public int getRowCount() {
		return rows.size();
	}
 
	public Class getColumnClass(int c)
	{
		return dataClass[c];
	}
 
	public boolean isCellEditable(int row, int col)
	{
		if( col == 2) return true;
		return false;
	}
 
	public Object getValueAt(int lignes, int colonnes)
	{
		Object value = null;
		if(lignes < rows.size()){
			value = rows.get(lignes).get(colonnes);
		}
		return value;
	}
 
 
	public void setValueAt(Object o,int i, int j)
	{
		// On recupère une alarmEntry
		// on ajoute les champs dans la table.
 
	}
 
 
	public void addRows(Vector row)
	{
		rows.add(row);
		this.fireTableDataChanged();
	}
 
 
	public Vector getList()
	{
		return rows;
	}
 
 
 
	public void removeRow(int row)
	{
		System.out.println("!!! removeRow("+row+"); !!!!!!!!!!!!!!!");;
		if(rows.isEmpty()) return;
		if(row<0) return;
		rows.remove(row);
		System.out.println("NOmbre de ligne = "+this.getRowCount());
		System.out.println();
		this.fireTableRowsDeleted(0,getRowCount());
 
	}
 
 
 
 
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
import javax.swing.JButton;
import javax.swing.JTable;
import javax.swing.table.DefaultTableCellRenderer;
import javax.swing.table.TableCellRenderer;
 
 
public class JButtonRenderer  implements TableCellRenderer
{
 
 
	public JButton getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column){
		return (JButton)value;
 
 
	}
}

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
56
57
58
59
60
61
62
63
64
 
import java.awt.Component;
import java.util.EventObject;
 
import javax.swing.JButton;
import javax.swing.JTable;
import javax.swing.event.CellEditorListener;
import javax.swing.table.TableCellEditor;
 
 
public class JButtonCellEditor implements TableCellEditor {
 
	@Override
	public JButton getTableCellEditorComponent(JTable arg0, Object arg1,
			boolean arg2, int row, int column) {
		// TODO Auto-generated method stub
 
		return (JButton)arg1;
	}
 
	@Override
	public void addCellEditorListener(CellEditorListener arg0) {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void cancelCellEditing() {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public Object getCellEditorValue() {
		// TODO Auto-generated method stub
		return null;
	}
 
	@Override
	public boolean isCellEditable(EventObject arg0) {
		// TODO Auto-generated method stub
		return true;
	}
 
	@Override
	public void removeCellEditorListener(CellEditorListener arg0) {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public boolean shouldSelectCell(EventObject arg0) {
		// TODO Auto-generated method stub
		return true;
	}
 
	@Override
	public boolean stopCellEditing() {
		// TODO Auto-generated method stub
		return true;
	}
 
 
}

J'avais plutot l'impression de m'etre bien débrouillé, et que ce code marcherait sans trop broncher...
Seulement voilà, l'ajout et la suppression fonctionne, mais il existe un petit bug: lors de la suppression d'une ligne, la ligne en dessous n'est plus selectionné lors de l'appuie sur le bouton Delete...et donc n'est plus supprimé directement , il faut reselectionner la ligne pour pouvoir la supprimer....

Ce bug n'est pas forcément vitale à résoudre ... mais cela fait deux jours que je suis dessus à essayer de comprendre pourquoi le programme agit ainsi, et je ne comprends absolument pas , et çà me tient vraiment à coeur !!!

Donc si un curieux ayant un peu plus d'expérience en JTAble passe par là, je
serais ravis de pouvoir en discuter!

Merci !

tu dois faire manuellement la sélection de la ligne au dessus.
essaye ce code:

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
public void actionPerformed(ActionEvent event) 
	{
		if(event.getSource() == addButton)
		{
			// le JButton qui envoie l'évenements est
			// le addButton
			System.out.println(" Ajout d'une valeur ");
			// On ajoute une ligne.
			addRow();
			// puis on sort de la fonction
			return;
			
		}
		
		if(event.getSource() instanceof JButton)
		{
			// Lorsque qu'un bouton de suppression est 
			// actionné
			System.err.println("ligne="+actionTable.getSelectedRow());
			if(model != null)
			model.removeRow(actionTable.getSelectedRow());
			else System.out.println("model =Null");
                        selectionnerLigneTable(actionTable.getSelectedRow() -1);
			
		}
	}
               private void selectionnerLigneTable(int ligneASel) {
                   if (ligneASel < 0) ligneASel = 0;
        
                   if ( actionTable.getRowCount() > 0 )
                         actionTable.setRowSelectionInterval(ligneASel,ligneASel);
              }

salut et merci de ta réponse.
Ta solution permet de remettre la ligne = 0 lorsque getRowCount renvoie -1, et donc ne donne pas la bonne ligne lorsque que l'on clique sur la ligne juste au dessous de la ligne supprimé. Le probleme est en fait que une fois la ligne supprimé celle dessous ne peut apparement plus etre séléctionnée lorsque que l'on appuie son bouton delete...
En fait je ne comprends pas pourquoi le code ce comporte comme çà, plus qu'une solution je voudrais une simple explication...

Merci a toi en tous les cas !