Mauvaise valeur dans JTable avec FireTableCellUpdated

Version imprimable

10/06/2013, 14h35
Cdébryha

Mauvaise valeur dans JTable avec FireTableCellUpdated

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 :aie:
10/06/2013, 14h52
Népomucène

ça doit être tout bête.

poste ton code
10/06/2013, 14h52
andry.aime

Citation:

Envoyé par Cdébryha

Sauriez-vous d'où cela peut venir?

De ton code.

A+.
10/06/2013, 14h59
Népomucène

Citation:

Envoyé par andry.aime

De ton code.

A+.

:bravo:

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

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:

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:

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:

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

10/06/2013, 15h24
Népomucène

Citation:

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

Mais c'est pas grave, il faut bien qu'on rigole un peu ... :D
ç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
10/06/2013, 15h32
Cdébryha

Citation:

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 :

Citation:

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.

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 :mrgreen:)
Voilà comment ça marche :

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

10/06/2013, 16h10
Cdébryha

J'ai oublié quelque chose d'important : la variable var, que j'envoie dans le setValueAt, est une ArrayList<Object>. C'est la raison pour laquelle j'envoie tout d'un coup. Je pourrais faire ce que tu me dis, mais avec ça c'est coton. :?

Mais normalement, si je suis pas con comme un balai sans manche (ce qui peut être le cas ici ;)), je devrai, à partir du setValueAt, enregistrer toutes les valeurs que je souhaite. C'est dailleurs ce qui se passe ici :

Code:

System.out.println("res ["+a+"]["+b+"] = "+res[a][b]);

Là, quand je boucle autour, j'ai toutes mes valeurs, que j'insère après une à une dans data[][] :

Code:

data[a][b] = res[a][b];

Mais c'est le coup de fusil juste après qui m'est pas compréhensible :

Code:

fireTableCellUpdated(a, b);

Même avec la doc je comprends pas ça : Comment mettre une valeur dans une cellule avec FireTableCellUpdated() si on ne donne pas la valeur à mettre dedans? 8O
(je vais sûrement devoir faire un autre topic pour ça...)

Je sais que ces questions peuvent sembler idiotes et que d'autres ont déjà dû y répondre, mais je n'ai pas trouvé où...

Ce problème, ça fait deux semaines que je travaille dessus. Je trouve une bonne solution, et j'ai fini mon programme... C'est quand même con, non? :lol:

Je sens que je vais mal finir... :aie:
10/06/2013, 16h15
Népomucène

Citation:

la variable var, que j'envoie dans le setValueAt, est une ArrayList<Object>

Dans ce cas, pourquoi ne pas parcourir var et déclencher setValueAt pour chaque objet ?

Citation:

fireTableCellUpdated

pourquoi en as-tu besoin puisqu'il ne s'agit que d'un problème d'affichage ?
10/06/2013, 16h29
Cdébryha
Citation:

Dans ce cas, pourquoi ne pas parcourir var et déclencher setValueAt pour chaque objet ?

Tu veux dire, avec un iterator, comme ça? :
Code:

1 2 3 4 5 6 7 Iterator<Object> it = var.iterator(); while(it.hasNext()) { Object o = it.next(); tableau.setValueAt(o, i, j); j++; }
Je reviens à mon problème initial. Dans ce cas là, ça ne m'affiche que la dernière valeur de chaque ligne. Donc j'en reviens au même qu'avant.:calim2:

Code:

pourquoi en as-tu besoin puisqu'il ne s'agit que d'un problème d'affichage ?

C'est FireTableCellUpdated qui, si j'ai bien compris, me mettrait les valeurs dans le data de ma JTable. Donc j'en ai besoin. Enfin, je pense, parce que ce truc là, si on me demandait, je dirais qu'il sert à suicider les neurones en boucle. :mouarf:

pour l'itération, je pense que ça ira mieux avec une boucle for.
en supposant qu'il y a 3 objets dans var et que tu dois les mettre en ligne n° 7 :
Code:

1 2 3 4 5 table.setValueAt("7", 7, 0); // numérotation de la ligne for (int col = 0; col <3; col ++) { table.setValueAt(var.get(col), 7, col +1); // inscription des données }
Citation:

ça ne m'affiche que la dernière valeur de chaque ligne

donc, je crois qu'il s'agit d'un problème de boucle et non de JTable.

Citation:

FireTableCellUpdated

si je ne dis pas de sottise, cela sert surtout quand on tape au clavier une valeur directement dans la table et donc il faut notifier au dataModel qui est derrière.

@Népomucène :

Alors, j'ai essayé ce que tu m'as dit (arrangé pour mon cas) :
Code:

1 2 3 4 5 6 7 8 9 10 11 for(int ligne=0;ligne<i;ligne++) { for(int col=0;col<=j;col++) { tableau.setValueAt(var.get(col), ligne, col); System.out.println(var.get(col)); System.out.println("i"+ligne+"j"+col); } }
En déroulant le programme, j'obtiens bien comme suis sous Ecplise :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 1 //valeur de var.get(0) i0j0 80.0 //valeur de var.get(1) i0j1 etc... 13 // dernière valeur de ma première ligne i0j4 //on arrive à la fin de la ligne 1 2 // deuxième ligne i1j0 // tout va bien etc... 31 // dernière valeur de ma deuxième ligne i1j4 // fin
Pourtant, au niveau de mon tableau, j'obtiens ça :

13|13|13|13|13|
13|13|13|13|13|

13 est la dernière valeur de la première ligne de mon tableau, que j'obtiens sur les deux lignes.

Je suis complètement perdu...:(

J'ai remarqué quelque chose. Lorsque je fais :

Code:

tableau.setValueAt(var.get(col), ligne, col);

dans ma classe principale, je vois que dans la méthode :
Code:

1 2 3 4 5 6 public void setValueAt(Object value, int row, int col) throws ClassCastException { data[row][col] = value; System.out.println("row"+row+"col"+col); fireTableCellUpdated(row, col); }
le System.out.println me renvoit col = 0 tout le temps.

Vous y comprenez quelque chose?

10/06/2013, 17h29
Népomucène

Tu as redéfini la méthode setValueAt ?

Essaye sans la redéfinir !
(met-la en commentaire)

Je viens de tenter sans la redéfinir. Je l'ai mis en commentaire. Mais le tableau reste vide. je me suis rendu compte d'une chose entre temps :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 int varlength =0; //En ajoutant ça, je vais jusqu'au bout de mon ArrayList for(int ligne=0;ligne<i;ligne++) { System.out.println("Pour ligne = "+ligne); for(int col=0;col<=j;col++) { System.out.println("Pour colonne = "+col); tableau.setValueAt(var.get(varlength), ligne, col); System.out.println("COLONNE : "+col); System.out.println("VALEUR DE GETvarl : "+var.get(varlength)); varlength++; } }
Déjà comme ça je vais jusqu'au bout de mon arrayList. C'est ça de gagné. Et lorsque je redéfinissais le setValueAt, j'obtenais la dernière valeur de chaque ligne cette fois. :mrgreen:

Ca tient vraiment à peu de chose. Ca doit être con comme la lune, mais kécécé? :? Je vais checker le coup de la colonne qui reste à 0. Doit y avoir quelque chose avec ça.

Tu devrais essayer en repartant d'un squelette qui fonctionne :

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
public class NewJFrame extends javax.swing.JFrame {

    public NewJFrame() {
        javax.swing.JTable jTable1 = new javax.swing.JTable();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jTable1.setModel(new javax.swing.table.DefaultTableModel(
                new Object[][]{
            {null, null, null, null},
            {null, null, null, null},
            {null, null, null, null},
            {null, null, null, null}
        },
                new String[]{
            "Title 1", "Title 2", "Title 3", "Title 4"
        }));
        getContentPane().add(jTable1, java.awt.BorderLayout.CENTER);
        pack();
        for (int i = 0; i < 4; i++) {
            for (int j = 0; j < 4; j++) {

                // injecte ici ton tableau
                jTable1.setValueAt("val " + i + " " + j, i, j);
            }
        }
    }
                       

    public static void main(String args[]) {
        new NewJFrame().setVisible(true);
    }
         
}

10/06/2013, 17h55
Cdébryha

Je vais tenter ça. Mais mon programme faisant déjà plus de 1000 lignes, ça risque de prendre du temps. Je reprendrai contact avec vous demain. Merci pour tout.

Note aux modérateurs : Je laisse ce topic ouvert, vu qu'il n'est pas terminé. ( je préviens pour que vous n'ayez pas à vous fatiguer à tout lire, y'a presque un après midi de discussion. ^_^).

Merci encore! Et a demain!
11/06/2013, 09h47
Cdébryha
Bonjour!

J'ai tenté de redéfinir mon programme en repartant d'un squelette saint. C'est pas mieux...

Devant obtenir un tableau dynamique, je ne peux pas faire le DefaultTableModel dans le setModel. Du coup, ce dernier plante. :cry:

Je dois dire que là, je me sens mal... Je vois les deadlines qui approchent et j'arrive pas à terminer ça. Pour ma déclaration, j'ai utilisé ça :
Code:

1 2 3 Object[][] data = new Object[13][5]; MaTableModel mtm = new MaTableModel(ta, data); JTable tableau = new JTable(mtm);
Mais en passant comme ça, ça marche pas.

J'ai complètement compris à quoi servait le FireTableCellUpdated(). Il sert à mettre à jour les informations de la méthode tableChanged(TableModelEvent e).

Je pense que le problème vient du col qui se met pas à jour partout. Ce que je trouve bizarre aussi. Je continue. Tenez moi au courant si vous y comprenez quelque chose. :)
11/06/2013, 10h56
Cdébryha

J'ai ENFIN trouvé une solution viable!

Et... C'était con comme la lune! :D

Voilà ma solution :

Lorsque je mettais mes informations dans mes data, j'oubliais une ligne :

Code:

tableau = new JTable(data, ta);

Cette ligne me permet de mettre à jour ma table. Il me suffisait donc de la mettre pour remplir mon tableau.

Mais, à ce moment, autre problème :

Lorsque j'avais les informations, ils ne restaient pas en place. Pourquoi? Réponse : parce que je faisais un this.repaint(). Je l'ai enlevé, et maintenant mes données sont persistentes.

Cependant, je n'obtenais pour chaque ligne que la répétition de ma première ligne...
En même temps, je redéclarai ce qui me permettait de parcourir mon ArrayList à chaque boucle de réception de data de ma table. :mrgreen:

Au final, tout fonctionne! J'ai plus qu'à généraliser ce fonctionnement et mon programme sera prêt! Je vous remercie beaucoup de l'aide que vous m'avez apporté, surtout toi, Népomucène!:ccool:

Merci beaucoup!!!! :aie: