[swing] problème de TableModelListener

problème de TableModelListener [swing] - Java - Programmation

Marsh Posté le 07-11-2005 à 19:49:48    

Bonjour à tous
 
je cherche à implémenter une JTable, écoutée par mon propre TableModelListener. Pour cela j'ai une classe AtmTableModel, qui hérite de AbstractTableModel, et qui implémente TableModelListener.
mais plutôt qu'un long discours, voici le code :

Code :
  1. package atmgui;
  3. import javax.swing.table.AbstractTableModel;
  4. import main.AllTaskManager;
  5. import javax.swing.event.TableModelListener;
  6. import javax.swing.event.TableModelEvent;
  8. @SuppressWarnings("serial" )
  9. public class AtmTableModel extends AbstractTableModel implements
  10.  TableModelListener {
  11. private Object[][] _data;
  12. private String[] _columnNames;
  13. private AllTaskManager _atm;
  15. public AtmTableModel(AllTaskManager atm) {
  16.  super();
  17.  _data = atm.getData();
  18.  _columnNames = atm.getColumnNames();
  19.  _atm = atm;
  21.  addTableModelListener(this);
  22. }
  24. public int getRowCount() {
  25.  return _data.length;
  26. }
  28. public int getColumnCount() {
  29.  return _columnNames.length;
  30. }
  32. public Object getValueAt(int row, int col) {
  33.  return _data[row][col];
  34. }
  36. public String getColumnName(int col) {
  37.  return _columnNames[col];
  38. }
  40. public boolean isCellEditable(int row, int col) {
  41.  return true;
  42. }
  44. public void contentChanged() {
  45.  _data = _atm.getData();
  46.  fireTableRowsInserted(getRowCount(), getRowCount());
  47. }
  49. public void tableChanged(TableModelEvent e) {
  50.  System.out.println("tableChanged" );
  51.  int row = e.getFirstRow();
  52.  int column = e.getColumn();
  54.  if (e.getType() == TableModelEvent.UPDATE)
  55.   _atm.changeTask(this, row, column, getValueAt(row, column));
  56.  else
  57.   return;
  58. }
  60. public String toString() {
  61.  String result = "Value of data:\n";
  62.  for (int i = 0; i < getRowCount(); i++) {
  63.   result += "    row " + i + ":";
  64.   for (int j = 0; j < getColumnCount(); j++)
  65.    result += "  " + getValueAt(i, j);
  66.   result += "\n";
  67.  }
  68.  return result += "--------------------------";
  69. }
  71. }


 
mon but premier est d'attrapper une modification faite dans la table par l'utilisateur, lors de l'exécution du programme.
MAIS VOILA, ça ne marche pas. je pensais pourtant avoir tout bien fait comme il faut, comme dans le how to de sun.
quelqu'un sait d'où vient le problème ?
 
merci

Reply

Marsh Posté le 07-11-2005 à 19:49:48   

Reply

Marsh Posté le 08-11-2005 à 08:56:48    

As tu bien enregistré ton listener sur ta table ?


---------------
Light is right
Reply

Marsh Posté le 08-11-2005 à 14:00:22    

dans le constructeur, il y a " addTableModelListener(this); "
cela ne suffit pas ?

Reply

Marsh Posté le 08-11-2005 à 16:47:35    

ca, c'est louche:

Code :
  1. fireTableRowsInserted(getRowCount(), getRowCount());


---------------
The Million Dollar Screenshot  (seo v7ndotcom elursrebmem paesys wifi)
Reply

Marsh Posté le 08-11-2005 à 17:43:21    

pourquoi est-ce louche ?
sans ça, la JTable qui utilise le modele n'est pas prévenue de l'insertion d'une nouvelle ligne dans le modele. nan ?
en fait je débute en swing.

Message cité 1 fois
Reply

Marsh Posté le 09-11-2005 à 07:20:17    

up :-/

Reply

Marsh Posté le 10-11-2005 à 14:26:48    

up :-(

Reply

Marsh Posté le 10-11-2005 à 23:52:57    

psychotek a écrit :

pourquoi est-ce louche ?
sans ça, la JTable qui utilise le modele n'est pas prévenue de l'insertion d'une nouvelle ligne dans le modele. nan ?
en fait je débute en swing.


 
Salut,
ce qui est surtout louche, c'est que tu appelle avec getRowCount() en parametre or:
 

Citation :

public void fireTableRowsInserted(int firstRow, int lastRow)
 
    Notifies all listeners that rows in the range [firstRow, lastRow], inclusive, have been inserted.


 
et les index des lignes de ta table vont de 0 à getRowCount() -1.
 
Je pense donc que ton modèle n'est jamais prévenu puisque cette ligne n'existe pas.
Tu devrais donc essayer de faire tes appels ainsi:

Code :
  1. fireTableRowsInserted(0, getRowCount() -1);


 
 
Si ça ne solutionne pas le problème, moi je ferais plutot comme ca:
 
Utiliser un DefaultTableModel à la place d'un AbstractTableModel qui n'est pas une classe abstraite: donc tu n'as pas besoin de gérer les notifications de changement (fireTableRows****), c'est déja fait.
 

Citation :

mon but premier est d'attrapper une modification faite dans la table par l'utilisateur, lors de l'exécution du programme


 
Ensuite, la JTable répercute automatiquement les changements sur le DefaultTableModel donc si l'utilisateur change une valeur, ajoute des lignes, etc. le modèle est automatiquement mis à jour. Tu n'as donc plus qu'à interroger le modèle suite à un changement:
 

Code :
  1. AllTaskManager atm = new AllTaskManager(/* tes paramètres */);
  3. DefaultTableModel dtm = new DefaultTableModel(atm) {
  4.      private AllTaskManager _atm = null;
  6.      public DefaultTableModel(AllTaskManager atm) throwsNullPointerException {
  7.           if(atm == null)
  8.                throw(new NullPointerException("ATM IS NULL" ));
  9.           _atm = atm;
  10.           // appel du constructeur avec Object[][] et Object[]
  11.           this(atm.getData(), atm.getColumnNames());
  13.           addTableModelListener(
  14.                 new TableModelListener() {
  15.                      public void tableChanged(TableModelEvent tme) {
  16.                           // ici tu mets le même code que ce que tu avais
  17.                      }
  18.                 }
  19.           );
  20.      }
  22.      public String toString() {
  23.           // ici pareil, tu remets ton code
  24.      }
  26.      // et ainsi de suite, tu redefinis seulement les méthodes dont tu as besoin.
  28. };


 

Reply

Marsh Posté le 11-11-2005 à 17:30:33    

merci pour ta réponse. je vais essayer ce que tu conseilles, dès que le cvs de sourceforge sera reparti (parce que là j'ai un timeout, c'est pénible)

Reply

Marsh Posté le 12-11-2005 à 11:30:14    

booon, super, finalement mon AtmTableModel dérive de DefaultTableModel, et effectivement ça marche, tout est bien géré. merci à toi.

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed