problème de tri dans un JTable après une suppression et vice versa

Version imprimable

Bonjour,

J'ai un JTable. Les opérations de tri et de suppression marchent bien séparement. MAIS si je fais un tri, puis je décide de supprimer un élément (et vice versa) ça ne marche plus , ça fait un ArrayIndexOutOfBoundException...pourtant je croyais avoir mis à jour mon modèle...Je ne sais pas ce qui ne va pas.
Je vous donne ci-après la classe de tri, puis la classe de suppression, et enfin la méthode de mise à jour dans ma Vue.
Merci d'avance pour tout!!!!!!!!

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public class TrierTableau_v4 extends AbstractAction implements ActionListener {
    private VueTableau_v4 fen;
 
    public TrierTableau_v4(VueTableau_v4 fen,String texte){
        super(texte);
        this.fen = fen;
    }
 
    public void actionPerformed(ActionEvent e) { 
        NgramModel_v4 m = fen.getNgramModel();
        m.sort(1); // on trie la colonne où il y a les occurences
        System.out.println("Vous avez cliqué sur Trier");
        System.out.println("premier élément après tri: "+ m.getValueAt(0, 0));
        //on raffraichit la Jtable
        fen.miseAJour();
    }
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public class SupprimerTableau_v4 extends AbstractAction implements ActionListener{
    private VueTableau_v4 fen;
 
    public SupprimerTableau_v4(VueTableau_v4 fen,String texte){
        super(texte);
        this.fen=fen;
    }
 
    public void actionPerformed(ActionEvent e) { 
        NgramModel_v4 m = fen.getNgramModel();
        ArrayList<String> maListe = m.getNgramSelected();
        System.out.println("vous voulez supprimer: "+Outils_v4.contenu(maListe));
        m.supprimer(maListe);
        System.out.println("Vous avez cliqué sur Supprimer");
        // raffraichissement des données dans le modèle
        fen.miseAJour();
    } 
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
    public void miseAJour() {
        // on raffraichit la table
        System.out.println("je mets à jour l'affichage du tableau");
        table.setModel(model_v4);
        if(table.getRowCount()>0){
            TableColumn col = table.getColumnModel().getColumn(2);
            col.setCellEditor(new MyCellEditor_v4());
            col.setCellRenderer(new MyCellRenderer_v4());
            etiquetteTotalN.setText("total Ngrames : " + model_v4.getTotalNgram());
            etiquetteTotalNDist.setText("total Ngrames distincts : "+ model_v4.getTotalNgramDist());
 
        }        
    }

03/11/2008, 05h21
sinok

Perso ça me ferait penser à un problème dans le modèle, qui ne répercute pas la suppression de la ligne dans le getRowCount, de fait la JTable essaie de trier une ligne qui n'existe plus, d'où l'ArrayOutOfBoundsException lancée.

Si c'est un problème dans le modèle qui ne repercute pas sur le getRowCount(), pourquoi la suppression seule sans le tri marche alors? Car je peux supprimer individuellement une ligne ou supprimer en groupe ça marche sans problème. C'est à partir du moment où je décide de faire un tri après suppression que ça ne marche plus.
J'ai rajouté dans ma classe suppression et tri un "this.modele.update(this.modele);" afin de faire un Update() du modèle mais ça ne marche toujours pas :(
Voici mon modèle :

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
142
143
144
 
package v4;
 
import javax.swing.event.EventListenerList;
import javax.swing.table.*;
import java.util.Arrays;
import java.util.ArrayList;
import v4.Outils_v4;
 
// le modÃ¨le
public class NgramModel_v4 extends AbstractTableModel {
	DefaultTableModel model; 
	Ligne [] lignes;
	int colonneTri;
	private EventListenerList listeners;
 
	public NgramModel_v4(){
		this.model = new DefaultTableModel();
		this.listeners = new EventListenerList(); 
		this.lignes = new Ligne[20];
	      for( int i = 0; i<lignes.length; ++i)
	         lignes[i] = new Ligne(i);// voir la classe Ligne plus bas
	}
 
	public NgramModel_v4(DefaultTableModel m){
		this.model = m;
		this.listeners = new EventListenerList(); 
		if(model==null){
			this.lignes = new Ligne[20];
		      for( int i = 0; i<lignes.length; ++i)
		         lignes[i] = new Ligne(i);
		}
		else {this.lignes = new Ligne[m.getRowCount()];
	      for( int i = 0; i<lignes.length; ++i){
	         lignes[i] = new Ligne(i);// voir la classe Ligne plus bas
	      }
		}
	}
 
	   public int getRowCount() {
		      return model.getRowCount();
	   }	   
	   public int getColumnCount() {
	      return model.getColumnCount();
	   }	   
	   public Object getValueAt(int rowIndex, int columnIndex) {
	      return model.getValueAt(lignes[rowIndex].index,  columnIndex);
	   }	   
	   public Class<?> getColumnClass( int i){
	      return model.getColumnClass(i);
	   }	   
	   public String getColumnName(int i){
	      return model.getColumnName(i);
	   }	   
	   public boolean isCellEditable(int row, int col) {
	      return col>1;
	   }
 
	   public void setValueAt(Object value, int row, int col) {
		// mise Ã* jour des valeurs dans le tableau aprÃ¨s le tri
	      model.setValueAt(value, lignes[row].index, col);
	      fireTableCellUpdated(lignes[row].index, col);
	   }
	   public void sort(int c){
	      colonneTri = c; 
	      try{
	    	  Arrays.sort(lignes);
	          fireTableDataChanged();
	          fireTableStructureChanged();       
 
	      }catch (RuntimeException e){}  // les donnÃ©es ne sont pas comparables !
	   }
 
	   public int getNumLigne(String ch){
		   //renvoie le numÃ©ro de ligne de ch dans le tableau
		   if (this.getRowCount()<1){
			   return 0;
		   }
		   else{
			    int i=0;
			   while(i<this.getRowCount()&(String)model.getValueAt(lignes[i].index,0)!=ch){
				   i++;
			   }
			    return i;
		   }
	   }
 
	   public ArrayList<String> getNgramSelected(){
			  int nbLignes = model.getRowCount();
			  ArrayList<String> lesSelectionnes = new ArrayList<String>();
			  for(int i=0; i<nbLignes; i++){
				 Object valeur= model.getValueAt(i, 2);
				 if (valeur.equals(true)){
					 String leNgram = (String)model.getValueAt(i, 0);
					 lesSelectionnes.add(leNgram);
				 }
			  }
			  return lesSelectionnes;
		   } 
 
	   public void supprimer(ArrayList<String> listeASupprimer){
			  for(int i=0; i<listeASupprimer.size(); i++){
				  int laLigne = getNumLigne(listeASupprimer.get(i));
				  model.removeRow(laLigne);
				  fireTableRowsDeleted(laLigne,laLigne);
				  fireTableDataChanged();
				  System.out.println("les Ã©lÃ©ments Ã* supprimer sont: "+Outils_v4.contenu(listeASupprimer));
			  }	
 
	   }
 
	   public DefaultTableModel getTableModel(){
		   return this.model;
	   }
 
	   //mise Ã* jour du modÃ¨le
	   public void update(NgramModel_v4 newModel){
		   this.model= newModel.getTableModel();
	   }
		private class Ligne implements Comparable{
		      int index;
		      public Ligne (int i){
		    	  index = i; }		     
		      public int compareTo(Object o) {
		            Ligne autreLigne = (Ligne)o;
		            Object cellule = model.getValueAt(index, colonneTri);
		            Object autreCellule = model.getValueAt(autreLigne.index, colonneTri);
		            return((Comparable)cellule).compareTo(autreCellule);
		      }
		}
		public int getTotalNgramDist(){
			//affiche le nombre total de ngrames distincts
			return this.model.getRowCount();
		}
		public int getTotalNgram(){
			//affiche le nombre total de ngrames distincts
			int total = 0;
			for(int i=0; i<this.model.getRowCount(); i++){
				total = total+ Integer.parseInt((this.model.getValueAt(i,1)).toString());
			}
			return total;
		}
 
}

Merci d'avance pour tout!!!!!!!!!!!

05/11/2008, 10h30
dinobogan
L'erreur provient de la désynchronisation entre ton tableau de "Ligne" et ton modèle.

Tu fais un tri sur le tableau de 20 "Ligne". Même lorsque tu supprimes une ligne de ta JTable, la taille de ton tableau de "Ligne" reste fixe.

Lorsque tu supprimes une ligne de la JTable, ton tableau de lignes "lignes" possède une ligne de plus que ton modèle. Or le tri du tableau de ligne s'effectue toujours sur les 20 éléments. J'imagine que le ArrayIndexOfBoundsException est levé sur une des deux lignes :
Code:

1 2 Object cellule = model.getValueAt(index, colonneTri); Object autreCellule = model.getValueAt(autreLigne.index, colonneTri);
c'est une très mauvaise idée d'utiliser un tableau pour tes "Ligne". Utilise plutôt la fonctionnalité de tri de la JTable. Ca fait la même chose que ton code mais.... en moins buggé :mrgreen:

Oui, c'est la ligne

Code:

Object cellule = model.getValueAt(index, colonneTri);

qui est responsable de ArrayIndexOutOffBondsException...
Alors, j'ai ignoré ma classe "Ligne" et j'ai utilisé TableRowSorter de Java à la place. J'ai juste modifié un peu dans actionPerformed de ma classe TrierTableau et ça marche très bien!!!!!!!!!! Un grand Merci à toi dinobogan:king::king: . Je mets donc en résolu!
Je mets ici la portion de code que j'ai changé, ça pourrait éventuellement aider quelqu'un d'autre!
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public void actionPerformed(ActionEvent e) { NgramModel_v4 m = fen.getNgramModel(); this.modele=m; SortKey[] sks = new SortKey[]{ new RowSorter.SortKey(1, SortOrder.ASCENDING) }; TableRowSorter nouvTable = new TableRowSorter(this.modele); fen.getTable().setRowSorter(nouvTable); nouvTable.setSortKeys(Arrays.asList(sks)); nouvTable.sort(); //on raffraichit la Jtable fen.setModele(modele); fen.miseAJour(); }