Discussion :

[oelydina]

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 :

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
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 :

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
 
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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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());
 
        }        
    }

[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.

[oelydina]

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 :

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
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!!!!!!!!!!!

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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é

[oelydina]

Oui, c'est la ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 . 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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();
	}