Discussion :

[momop72]

Bonjour,

J'essaie depuis plusieurs jours de supprimer des lignes avec JTable mais ça ne fonctionne pas. J'ai déjà vu qu'il existait des sujets sur le problème mais je reste tout de même bloquer.

Dans un premier temps je souhaite juste supprimer la première ligne en appuyant sur un bouton. Je veux impéritavement utiliser une AbstractTableModel par contre.

Voici mon code :

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
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
import java.util.Collection;
 
import javax.swing.table.AbstractTableModel;
 
 
public class MaTableModel extends AbstractTableModel{
 
	private static final long serialVersionUID = 1L;
 
	private Collection<Personne> data;
	private String[] columnNames = {"Nom", "Prenom"};
 
	public MaTableModel (Collection<Personne> data) {
		this.data = data;
	}
 
	@Override
	public int getColumnCount() {
		return columnNames.length;
	}
 
	@Override
	public int getRowCount() {
		if (this.data == null) return 0;
		else return this.data.size();
	}
 
	@Override
	public Object getValueAt(int rowIndex, int columnIndex) {
		Personne per = (Personne) data.toArray()[rowIndex];
		switch (columnIndex) {
		case 0 : return per.getNom();
		case 1 : return per.getPrenom();
		default : throw new IllegalArgumentException ("Colonne inconnue" + columnIndex);
		}
	}
 
	/* (non-Javadoc)
	 * @see javax.swing.table.AbstractTableModel#isCellEditable(int, int)
	 */
	@Override
	public boolean isCellEditable(int rowIndex, int columnIndex) {
		//Permet de rendre toutes les colonnes non-éditables
		return false;
	}
 
	/* (non-Javadoc)
	 * @see javax.swing.table.AbstractTableModel#getColumnName(int)
	 */
	@Override
	public String getColumnName(int arg0) {
		return this.columnNames[arg0];
	}
 
	/* (non-Javadoc)
	 * @see javax.swing.table.AbstractTableModel#getColumnClass(int)
	 */
	@Override
	public Class<?> getColumnClass(int columnIndex) {
		if (data == null) return Object.class;
		else {
			Object objetRecupere = getValueAt(0, columnIndex);
			if (objetRecupere == null) return Object.class;
			else return objetRecupere.getClass();
		}
	}
}

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
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
 
import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.ListSelectionModel;
 
public class PanelListe extends JPanel {
 
	private static final long serialVersionUID = 1L;
 
	private MaTableModel maTableModele = null;
	private JTable maTable = null;
	private final JButton jbSupprimer;
 
	public PanelListe() {
 
		jbSupprimer = new JButton("Supprimer");
 
		// Mise en place du Modèle de données à afficher
		// ///////////////////////////////////////////////
		ArrayList<Personne> maListe = new ArrayList<Personne>();
		maListe.add(new Personne("Michel", "Dumont"));
		maListe.add(new Personne("Michou", "Patrick"));
 
		// /////////////////////////////////////////////////////////////////////////
		// Affichage la JTable
		// que l'on doit placer au préalable dans un conteneur de type
		// JScrollPane
		// /////////////////////////////////////////////////////////////////////////
		maTableModele = new MaTableModel (maListe);
		maTable = new JTable(maTableModele);
 
		maTable.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
 
		//Utilisation d'une classe interne locale
		class ActionListenerSupprimer implements ActionListener {
 
			private PanelListe monPanelListe;
 
			@Override
			public void actionPerformed(ActionEvent arg0) {
				monPanelListe.maTable.removeRowSelectionInterval(0, 1);
				monPanelListe.maTableModele.fireTableRowsDeleted(0, 1);
				monPanelListe.maTableModele.fireTableDataChanged();
 
				repaint();
			}
 
			ActionListenerSupprimer (PanelListe monPanelListe) {
				this.monPanelListe = monPanelListe;
			}
		}
 
		jbSupprimer.addActionListener(new ActionListenerSupprimer (this));
 
		JScrollPane monScrollPane = new JScrollPane(maTable);
 
		this.setLayout(new BorderLayout());
		add(monScrollPane, BorderLayout.CENTER);
		add(jbSupprimer, BorderLayout.SOUTH);
	}
 
	public static void main(String[] args) {
		JFrame fenetre = new JFrame();
		fenetre.add(new PanelListe());
		fenetre.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		fenetre.setTitle("Test");
		fenetre.setSize(700, 500);
		fenetre.setVisible(true);
	}
 
}

Merci de votre aide.

[joel.drigo]

Salut,

La méthode removeRowSelectionInterval() de JTable ne supprime pas de lignes dans les données, mais désélectionne des lignes (modification du SelectionModel, pas du TableModel).

Tu dois implémenter la méthode de suppression dans le modèle (enfin tu n'es pas obligé, mais c'est plus clean). Par contre, une Collection n'est pas ce qui est le plus adapté : il te faudrait utiliser une List, parce qu'il faut pouvoir accéder aux éléments par leur index, pour optimiser les évènements, ce que ne permet pas une Collection.

Mais tu peux quand même faire une suppression par référence, en lançant l’évènement pour prévenir la vue fireTableDataChanged() :

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
16
17
18
19
20
21
22
23
24
25
public class MaTableModel extends AbstractTableModel{
 
	private static final long serialVersionUID = 1L;
 
	private Collection<Personne> data;
	private String[] columnNames = {"Nom", "Prenom"};
 
	public MaTableModel (Collection<Personne> data) {
		this.data = data;
	}
 
        public void remove(Personne personne) {
                 if ( data.remove(personne) {
                      fireTableDataChanged();
                }
        }
        public void removeAll(Collection<Personne> personnes) {
                 if ( data.removeAll(personnes) {
                      fireTableDataChanged();
                }
        }
 
        ...
 
}

[momop72]

Merci ! Ça fonctionne parfaitement maintenant