Discussion :

[ulquiorra]

Bonjour,

Je me suis mis aux JTable , et j ' essaie de supprimer une ligne que j ' aurai selectionne dans mon tableau( rempli avec des noms de fichiers)

Le probleme , c ' est que ca ne fait aucune action et me renvoi aucune erreur(je pense pourtant que la logique de ma fonction est bonne)

Un peu d ' aide svp

creation de ma JTable

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 JPanel creerPanelCentre()
	{
 
		JPanel panneauCentre = new JPanel();
 
		panneauCentre.setPreferredSize(new Dimension(300,300));
		panneauCentre.setLayout(new BorderLayout());
		listeMessage= new JTable(new TableauPersonnalise());
 
		listeMessage.setBackground(Color.BLACK);
		listeMessage.setForeground(Color.WHITE);
		listeMessage.setPreferredSize(new Dimension(this.getWidth(),this.getHeight()));
		listeMessage.setRowHeight(30);
		listeMessage.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		listeMessage.addMouseListener(new Evenement(1,this));
		JScrollPane defilant = new JScrollPane(listeMessage);
		defilant.setPreferredSize(new Dimension(this.getWidth(),this.getHeight()));
		defilant.setWheelScrollingEnabled(true);
 
		//panneauCentre.setBackground(Color.LIGHT_GRAY);
		panneauCentre.add(defilant);
		panneauCentre.setVisible(true);
 
		return panneauCentre;
	}

son modele:

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
public class TableauPersonnalise extends DefaultTableModel{
private RecupererMusique musiques=new RecupererMusique(null);
//private final String[] entetes = {"Prénom", "Nom", "Couleur favorite", "Homme", "Sport"};
 
 
 
private List <String> songs= new ArrayList<String>();
 
public TableauPersonnalise() {
    super();
    songs=musiques.updateSongList();
    //System.out.println("liste de musiques"+ musiques.updateSongList());
    //System.out.println("taille"+ musiques.updateSongList().size());
 
}
 
 
 
 
	@Override
	 public int getRowCount() {
 
		return 9;
 
    }
 
    public int getColumnCount() {
        return 1;
    }
 
    public String getColumnName(int columnIndex) {
        return null;
    }
 
    public Object getValueAt(int rowIndex, int columnIndex) {
        return songs.get(rowIndex);
 
    }

mon evenement:

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
 public void removeRows() {
		   TableauPersonnalise tableau= (TableauPersonnalise) fenetre.getListeMessage().getModel();
 
		    int[] selectedRows = fenetre.getListeMessage().getSelectedRows();
		    if (selectedRows.length != 0) {
		       for (int i = 0; i < selectedRows.length; ++i)
		    	   tableau.removeRow(i);
		    }
		    tableau.fireTableStructureChanged();
		    return;
		}
 
	public void actionPerformed(ActionEvent e){
 
		switch(code){
 
		case 2 :
			removeRows();
			 break;
		}
	}

merci de votre aide

[OButterlin]

Dans ton cas, c'est plutôt tableau.fireTableDataChanged()

[ulquiorra]

salut,

Merci de ton aide
Même apres cette modification de la méthode , il n y a pas de changement.
C ' est assez bizarre , car mon action deselectionne la ligne que je veux supprimer

[ulquiorra]

up svp

[MaitreKaio]

Tu n'as visiblement pas donné tout le code. Quand on essaie de suivre ce qui se passe, il manque visiblement des morceaux...

As-tu essayé de tracer ce qui se passe ? De ce que j'en vois, ton code peut ne rien faire pour plusieurs raisons:

• actionPerformed n'est pas appelé
• le code (dont on ne voit pas l'origine) n'est pas égal à 2
• selectedRows est vide (d'ailleurs, que fait la méthode listeMessage.getSelectedRows(), que tu ne donnes pas non plus ?

[ulquiorra]

Salut

Merci de ta reponse.

Mon action performed est bien appelé dans ma classe graphique

Code :

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

boutonSupprimer.addActionListener(new Evenement(2,this));

Correspond a l' indice code=2 que j ' ai spécifié dans le constructeur de ma classe chargé d ' écouter les évènements

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
public class Evenement  implements ActionListener, MouseListener{
private int code;
private GUI fenetre;
private int currentPosition = 0; 
private AudioClip ding;
final String MEDIA_PATH=new String("C:/Documents and Settings/EddayanH/Desktop/test SD/");
private RecupererMusique songs=new RecupererMusique(fenetre);
private TableauPersonnalise tableau=new TableauPersonnalise();
private boolean bruitFond=true;
 
 //
 
 
	public Evenement(int i, GUI fenetre) {
		super();
		this.code=i;
		this.fenetre=fenetre;
		// TODO Auto-generated constructor stub
	}
 
	 public void removeRows() {
		   TableauPersonnalise tableau= (TableauPersonnalise) fenetre.getListeMessage().getModel();
 
		    int[] selectedRows = fenetre.getListeMessage().getSelectedRows();
		    if (selectedRows.length != 0) {
		       for (int i = 0; i < selectedRows.length; ++i)
		    	   tableau.removeRow(i);
		    }
		    tableau.fireTableDataChanged();
		    return;
		}
 
	public void actionPerformed(ActionEvent e){
 
		switch(code){
 
		case 2 :
			removeRows();
			 break;
		}
	}

listeMessage correspond a mon JTable et getselectedRows() est une méthode deja implémentée dans la classe JTable et qui permet de récupérer les lignes sélectionnées.

Ce qui est étrange , c est que mon listener semble fonctionner , mais quant je sélectionne une ligne de mon tableau et clique sur le bouton supprimer , ca me deselectionne la ligne( je ne vois pas pourquoi ca fait ca d ailleurs)au lieu de la supprimer

Voila, merci de ton aide

[MaitreKaio]

Je crois que les fireXXX, qui impliquent un rechargement de la table, entrainent une réinitialisation de la sélection.
As-tu débuggé la méthode removeRows() pour voir ce qui s'y passe exactement ?

Tiens d'ailleurs je m'aperçois en le lisant que tu veux probablement faire

Code :

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

1
2
3
 
for (int i = 0; i < selectedRows.length; ++i)
		    	   tableau.removeRow(selectedRows[i]);

et non pas

Code :

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

1
2
3
 
for (int i = 0; i < selectedRows.length; ++i)
		    	   tableau.removeRow(i);

En effet, si ta table a 5 lignes, et que les ligne d'indices 3 et 4 sont sélectionnées, avec ton code tu vas effacer les lignes 0 et 1 (enfin quand ca marchera )

[ulquiorra]

Merci de ton aide, je commence a y voir plus clair
car en effet , imaginons que j ai une liste de 6 , si je selectionne les elements 4,5,6 , j ai un ArrayIndexOutOfBoundsException(3>=3 puis 4 >=3 etc)

D ' ou peut venir le probleme?
Si je selectionne uniquement les 1,2,3 , j ai pas d exception levees

Merci encore

edit: apres debug, rien d anormal a signaler , mes variables sont bien remplies

[ulquiorra]

Bon , j ' ai reussi et c ' etait plus bete que ce que je croyais ...
J ' ai insiste sur la methode removerows du defautTableModel alors que je n ' avais qu ' a utiliser la methode remove de la collection ArrayList.

Code :

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

1
2
3
4
public void deleteRow(int rowIndex) {
        songs.remove(rowIndex);
        fireTableRowsDeleted(rowIndex,rowIndex); /
    }

En gros , au lieu de supprimer une ligne , je supprime un element de ma liste , ce qui revient au meme( car je suppose que DefautTableModel ne travaille pas avec les listes d ou le probleme).
Merci a tous de votre aide