Discussion :

[Cdébryha]

Bonjour à tous,

J'ai un problème qu'il me faudrait résoudre concernant ma JTable :

J'utilise un DataModel perso, que j'utilise pour ma JTable tableau. Pour l'instant, tout est simple.

J'ai un listener sur le tableau. Ce listener m'écrit sous Eclipse chaque fois qu'une cellule du tableau est changée.

Quand je demande à ce qu'il écrive, le listener m'écrit les bonnes choses. Là où ça se gatte, c'est au niveau du tableau à l'affichage.

Dans toutes les cases modifiées du tableau, il ne m'écrit que la première valeur écrite dans la ligne du tableau qu'il parcourt.

un exemple concret :

Je veux entrer des informations diverses dans un tableau de 2 lignes et 5 colonnes.

Colonne 1 : numéro de la ligne.
Colonnes suivantes : autres valeurs
Lorsque je fais un System.out.println dans mon tableChanged, il m'écrit bien toutes les valeurs que j'insère dans mon tableau.

Là où ça coince, c'est pour l'affichage. Car à l'affichage, pour la ligne 1, dans le numéro de ligne, j'ai bien le numéro de ligne, mais pour les 4 colonnes suivantes, j'ai aussi le numéro de ligne, et non pas les valeurs que je devrai avoir.
Et bien sûr, pareil pour la ligne 2. Que le numéro de ligne répété, et pas le reste.

A l'affichage, je devrai avoir (ligne 1 par exemple) : 1, 80, 500, 18, 3

Mais j'ai : 1, 1, 1, 1, 1

Sauriez-vous d'où cela peut venir? j'ai pensé à mon fireTableCellUpdated que j'ai dans mon setValueAt(...), mais ça n'a pas l'air d'être ça.

je vous remercie beaucoup pour l'aide que vous pourrez m'apporter.


Cordialement,

Cdébryha

[Népomucène]

ça doit être tout bête.

poste ton code

[andry.aime]

Sauriez-vous d'où cela peut venir?

De ton code.

A+.

[Népomucène]

De ton code.

A+.

@andry.aime tu te laisses allez là ! c'était un peu facile !

[Cdébryha]

Malheureusement, je ne peux pas poster tout mon code pour des raisons de sécurité.

Cependant, voilà les morceaux de codes qui concernent l'insertion des valeurs au sein du tableau :

Le listener :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public void tableChanged(TableModelEvent e) {
        int row = 0;
        row = e.getFirstRow();
        int column = 0;
        column = e.getColumn();
        TableModel model = (TableModel)e.getSource();
 
        Object data = model.getValueAt(row, column);
        System.out.println("Valeur qui devraient s'insérer dans le tableau : "+data);

Dans ma classe principale :

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
while(rs.next())
					{
						j=0;
						nbm = rs.getInt(1);
						pmu = rs.getFloat(2);
						tmu = rs.getFloat(3);
						tmv = rs.getFloat(4);
						date = rs.getString(5);
 
					//Thread.sleep(1500);
						var.add(nbm);
						tableau.getTableHeader().getColumnModel().getColumn(j).setMaxWidth(200);
						j++;
						//Thread.sleep(1500);
						var.add(pmu);
						tableau.getTableHeader().getColumnModel().getColumn(j).setMaxWidth(200);
						j++;
						//Thread.sleep(1500);
						var.add(tmu);
						tableau.getTableHeader().getColumnModel().getColumn(j).setMaxWidth(200);
						j++;
						//Thread.sleep(1500);
						var.add(tmv);
						tableau.getTableHeader().getColumnModel().getColumn(j).setMaxWidth(200);
						j++;
						var.add(date);
						tableau.getTableHeader().getColumnModel().getColumn(j).setMaxWidth(200);
						i=i+1;
						System.out.println("Values i et j :"+i+" "+j);
 
 
					}
					SetVal(i,j);
										tableau.setValueAt(var, i, j);

Le SetValueAt de mon AbstractTableModel :

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
    public void setValueAt(Object value, int row, int col) throws ClassCastException
    {
    	a = 0;
    	b = 0;
    	row = Fenetre.geta();
    	col = Fenetre.getb();
    	String test = value.toString();
    	test = test.replace("[", " ");
    	test = test.replace("]", "");
    	System.out.println("Test = "+test);
    	StringTokenizer st = new StringTokenizer(test, ",");
    	String[][] res = new String[row+1][col+1];
    	System.out.println("row et col : "+row+" "+col);
    	while(st.hasMoreTokens()&&a<row)
    	{
    		while(b<=col)
    		{
    			System.out.println(a+"<=a : b=>"+b);
    			res[a][b] = st.nextToken();
    			System.out.println("res ["+a+"]["+b+"] = "+res[a][b]);
    			b++;
    		}
    		a++;
    		b=0;
    	}
    	a=0;
    	b=0;
    	while(a<row)
    	{
    		while(b<=col)
    		{
    	        data[a][b] = res[a][b];
    	        System.out.println("a et b"+a+" "+b);
    	        fireTableCellUpdated(a, b);
    	        b++;
    		}
    		a++;
    		b=0;
    	}        
    }

Note : Je trouve dommage la réponse du modérateur.

[Népomucène]

Note : Je trouve dommage la réponse du modérateur.

Mais c'est pas grave, il faut bien qu'on rigole un peu ...
ça m'arrive aussi de me faire charrier quand j'écris une bonne tautologie.

Sinon, pour ton code, je pense que tu essaies de changer plusieurs cellules à la fois
en utilisant un seul setValueAt(Object value, int row, int col)
Ce n'est pas fait pour cela. Il faut appeler cette méthode pour chaque cellule à modifier.

Tu dois boucler sur la méthode et non pas mettre cette méthode dans une boucle.
(tiens il faudra que je la ressorte celle-là)

exemple :
http://www.java2s.com/Tutorial/Java/...odofJTable.htm

[Cdébryha]

je pense que tu essaies de changer plusieurs cellules à la fois
en utilisant un seul setValueAt(Object value, int row, int col)

Il est vrai que je n'utilise qu'un seul setValueAt, mais j'avais déjà essayé avant de mettre à jour chaque cellule à chaque boucle, mais ça n'avait pas marché. je me retrouvais toujours avec la dernière valeur entrée dans tout le tableau...

En gros :

Tu dois boucler sur la méthode et non pas mettre cette méthode dans une boucle.

J'ai fait les deux.
Premier cas : Je n'obtiens que la dernière valeur du tableau, mais dans tout le tableau.
Deuxième cas : J'obtiens ce que j'ai actuellement.

[Népomucène]

setValueAt est fait pour une seule cellule.
Si tu as eu des problèmes dans la version précédente, c'est que le bug était ailleurs.

J'ai repris le code que j'avais mis en lien et j'ai ajouté une bouclette (petite boucle )
Voilà comment ça marche :

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
 
import java.awt.BorderLayout;
import java.util.Vector;
 
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JScrollPane;
import javax.swing.JTable;
 
@SuppressWarnings("UseOfObsoleteCollectionType")
public class SetEditableTableCell {
 
    public static void main(String args[]) {
        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
        Vector<String> rowOne = new Vector<String>();
        rowOne.addElement("Row1-Column1");
        rowOne.addElement("Row1-Column2");
        rowOne.addElement("Row1-Column3");
 
        Vector<String> rowTwo = new Vector<String>();
        rowTwo.addElement("Row2-Column1");
        rowTwo.addElement("Row2-Column2");
        rowTwo.addElement("Row2-Column3");
 
        Vector<String> rowThree = new Vector<String>();
        rowThree.addElement("Row3-Column1");
        rowThree.addElement("Row3-Column2");
        rowThree.addElement("Row3-Column3");
 
        Vector<Vector> rowData = new Vector<Vector>();
        rowData.addElement(rowOne);
        rowData.addElement(rowTwo);
        rowData.addElement(rowThree);
 
        Vector<String> columnNames = new Vector<String>();
        columnNames.addElement("Column One");
        columnNames.addElement("Column Two");
        columnNames.addElement("Column Three");
        JTable table = new JTable(rowData, columnNames);
 
        JScrollPane scrollPane = new JScrollPane(table);
        frame.add(scrollPane, BorderLayout.CENTER);
        frame.setSize(300, 150);
        frame.setVisible(true);
        frame.setLocation(50, 50);
 
        JOptionPane.showMessageDialog(null, "On change ?");
 
        // la fameuse boucle !!!
        for (int i = 0; i < 3; i++) {
            table.setValueAt("changé !", i, i);
        }
    }
 
}

Oui, je sais, ça utilise un Vector mais la question n'est pas là.