[JTable] Tri problème suppression

Bonjour,

Je possède dans mon application une JTable sur laquelle j'utilise le tri via TableRowSorter.

Lorsque je supprime une de mes lignes sans trier, tout fonctionne parfaitement, cela supprime la ligne dans l'affichage et supprime l'élément dans la base de données.
Lorsque je supprime une de mes lignes en utilisant le tri, ce n'est pas le bon élément qui est supprimé dans l'affichage mais c'est le bon élément qui est supprimé dans ma base de données.
Donc si je ferme et rouvre ma JTable le problème n'est plus là, je pourrais faire un simple rafraîchissement en somme mais je ne trouve pas ça propre.

(J'ai cru comprendre que ça venait du fait que lors de la suppression, la taille de mon tableau reste le même avec un élément en moins et le tri s'effectue donc encore sur la taille totale.)

Merci par avance pour votre aide !

Voici le 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

super(); setTitle("Liste des clients"); setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);   modele = new ListeClientsModele(); cDAO = new ClientDAO(); tableClients = new JTable(modele); tableClients.getColumnModel().getColumn(0).setMinWidth(0); tableClients.getColumnModel().getColumn(0).setMaxWidth(0); getContentPane().add(new JScrollPane(tableClients)); tableClients.addMouseListener(new MouseAdapter() { @Override public void mouseClicked(MouseEvent click) { if(click.getClickCount()==1 && click.getButton()==click.BUTTON3) { ligne = tableClients.rowAtPoint(click.getPoint()); id = (int) tableClients.getValueAt(ligne, 0); popupMenu.show(tableClients, click.getX(),click.getY()); } } });   TableRowSorter<TableModel> sorter = new TableRowSorter<TableModel>(tableClients.getModel()); tableClients.setRowSorter(sorter); sorter.setSortsOnUpdates(true);

---

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

popupMenu = new JPopupMenu(); item = new JMenuItem("Supprimer"); item.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent arg0) { try { if(aDesReservEnCours(id) == false) { cDAO.delete(id); afficherMessage("Suppression effectuée avec succès"); modele.removeClient(ligne); } else { afficherMessage("Le client a des réservations en cours"); } } catch(Exception e) { afficherMessage("Erreur dans la suppression du client."); } } }); popupMenu.add(item);

---

tu dois ajouter a la fin de ton remove ceci:

Code:

---

1 2 3

modele.removeClient(ligne); modele.fireTableDataChanged()

---

Eric

Non ça ne marche pas car je l'ai déjà dans mon modele :

Code:

---

1 2 3 4 5

public void removeClient(int rowIndex) { listeClients.remove(rowIndex); fireTableRowsDeleted(rowIndex, rowIndex); fireTableDataChanged(); }

---

J'ai testé quand même mais ça ne change rien :(

essaies de mettre ceci :

Code:

---

modele.removeRow(ligne)

---

Euh... je n'ai pas cette méthode O_o

ton model extends quelle model?

Extends AbstractTableModel.

essaies donc ceci:

Code:

---

fireTableRowsDeleted(rowIndex - 1, listeClients.size() - 1);

---

J'ai testé, cela me fait exactement la même chose x)

voici un exemple qui marche:

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

import java.awt.Point; import java.util.ArrayList; import java.util.Arrays; import java.util.HashMap; import java.util.List; import java.util.Map;   import javax.swing.table.AbstractTableModel;   public class LiveSearchTableModel extends AbstractTableModel {   /**          *          */ private static final long serialVersionUID = 1L; private List<String> names = new ArrayList<String>(); private Map<Point, Object> data = new HashMap<Point, Object>(); private Point[] keys;   public LiveSearchTableModel() { this.names = Arrays.asList("Point", "Name"); data.put(new Point(1, 1), "One"); data.put(new Point(2, 2), "Two"); data.put(new Point(3, 3), "Three"); this.keys = data.keySet().toArray(new Point[data.size()]); }   public void remove(int row) { data.remove(keys[row]); keys = data.keySet().toArray(new Point[data.size()]); this.fireTableRowsDeleted(row, row); }   @Override public int getColumnCount() { return names.size(); }   @Override public String getColumnName(int col) { return names.get(col); }   @Override public int getRowCount() { return data.size(); }   @Override public Object getValueAt(int row, int col) { if (col == 0) { return keys[row]; } else { return data.get(keys[row]); } } }

---

et

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

import model.LiveSearchTableModel;   public class TableAbstractModelTest extends JPanel { public TableAbstractModelTest() { super(new BorderLayout()); final LiveSearchTableModel model = new LiveSearchTableModel(); JTable table = new JTable(model); RowSorter<LiveSearchTableModel> sorter = new TableRowSorter<LiveSearchTableModel>(model); table.setRowSorter(sorter); this.add(new JScrollPane(table)); this.add(new JButton(new AbstractAction("Delete") {     @Override public void actionPerformed(ActionEvent e) { model.remove(0); } }), BorderLayout.SOUTH); }   private void display() { JFrame f = new JFrame("TableModelTest"); f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); f.add(this); f.pack(); f.setLocationRelativeTo(null); f.setVisible(true); }   public static void main(String[] args) { EventQueue.invokeLater(new Runnable() {   @Override public void run() { new TableAbstractModelTest().display(); } }); } }

---

Eric

C'est tout de même étrange que dans ton actionPerformed les variables id et ligne ne soit pas recalculées.

La logique voudrait qu'il faudrait au moment du clic sur ton bouton

1 Récupérer la (ou les lignes) sélectionnée(s) sur la JTable (getSelectedIndex)
2 Convertir ce numéro de ligne (qui est dans l'ordre défini par la JTable) en numéro de ligne dans l'ordre défini dans le modèle via la méthode convertRowIndexToModel(int viewRowIndex) de la JTable
3 Récupérer ton identifiant depuis ton modèle avec le numéro de ligne obtenu en 2
4 Appeler ton DAO (attention, celui ci ne devrait pas s'exécuter dans le thread Swing)
5 Appeler ta méthode de suppression de la ligne avec le numéro de ligne obtenu en 2 (qui elle doit bien sur faire appel au fireTableRowsDeleted