Bonjour,

lors d'un column dragging, quand je déplace la colonne 3 (cellule éditée) en 2 (donc une permutation), la valeur qui était en 3 vient bien en 2, mais la valeur qui était en 2 est écrasée par la valeur qui était en 3.
En mettant des traces dans la méthode setValueAt(Object value, int row, int col) de mon tableModel, je constate lors de mon drag and drop
  • que cette méthode est appelée pour mettre à jour la colonne 3 du modèle (à la limite pourquoi pas, de toute façon je ne change pas les valeurs des cellules)
  • puis qu'elle est appelée avec col = 2 (là ça ne devrait carrément pas) avec la valeur de l'ancienne colonne 3.


Au final je me retrouve avec un modèle faux : les colonnes 2 et 3 contiennent l'ancienne valeur de 3

si col1 col2 col3 sont le contenu de mes colonnes, ça donne :

Avant le drag column 3 vers 2:
écran : col1 col2 col3
modèle : col1 col2 col3

Après le drag column:
écran : col1 col3 col3
modèle : col1 col3 col3

Je ne sais pas déjà à quel endroit est appelée setValueAt, je ne trouve aucun appel dans mon code.
Peut être que c'est moi qui déclenche cette commande via un autre bout de code, mais comment le retrouver ?

Pour info, j'utilise
Code : Sélectionner tout - Visualiser dans une fenêtre à part
putClientProperty("terminateEditOnFocusLost", Boolean.TRUE);
dans ma jtable, et même sans cette instruction la méthode continue d'être appelée je ne sais pas par qui.

J'utilise aussi un
Code : Sélectionner tout - Visualiser dans une fenêtre à part
PersoEditor extends AbstractCellEditor implements TableCellEditor
et je fais quelques bidouilles sur le focus dans ma jtable, je ne peux pas envoyer mon code il est pas simple.

Par contre voici le table model et la 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
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
 
public PersoTable(final PgTable pgt, PersoTableModel persotablemodel) {
 
        super(persotablemodel);
        if (persotablemodel.isModelFiltre()) {
            tableFiltre = true;
        } else {
            tableFiltre = false;
        }
 
        /*
         * parcours des colonnes utiles (action et inforow comptent pas)
         */
        sorter = new TableRowSorter<>(this.getModel());
        this.setRowSorter(sorter);
        for (int i = 0; i < pgt.getNbCols(); i++) {
            sorter.setComparator(i, new PersoTriable());
        }
        DocumentListener dl = new DocumentListener() {
            /*
             * on créé ici car un seul suffit pour la tableFiltre. 
             * Dans l'Editot il serait instancié plusieurs fois
             */
            @Override
            public void insertUpdate(DocumentEvent e) {
                filtre(pgt);
            }
 
            @Override
            public void removeUpdate(DocumentEvent e) {
                filtre(pgt);
            }
 
            @Override
            public void changedUpdate(DocumentEvent e) {
                filtre(pgt);
            }
        };
        ge = new PersoEditor(pgt, dl);
        gr = new PersoRender(pgt);
        setDefaultRenderer(String.class, gr);
        setDefaultEditor(String.class, ge);
 
        removeColumn(getColumnModel().getColumn(INFO_ROW_COL));
 
        setShowGrid(false);
        setIntercellSpacing(new Dimension(0, 0));
 
        setFillsViewportHeight(true);
        /**
         * pour mise en édit qd on vient sur la jtable par une tabulation
         */
        addFocusListener(new GfocusAdapter());
        getTableHeader().addMouseListener(PersoTableTriEcouteur.getInstance());
 
        /*
         * mouseMotion car besoin du move
         */
        addMouseMotionListener(PersoMouseAdapter.getInstance());
        addMouseListener(PersoMouseAdapter.getInstance());
        putClientProperty("terminateEditOnFocusLost", Boolean.TRUE);
}
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
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
 
public class PersoTableModel extends AbstractTableModel {
 
    boolean DEBUG = true, modelFiltre;
    private String[] entetes;
    private List<ParDesc> listePardesc;
 
    public PersoTableModel(List<ParDesc> data, PgTable pgtable, PersoTable.IS_FILTRE isFiltre) {
        if (isFiltre == PersoTable.IS_FILTRE.Oui) {
            modelFiltre = true;
        } else {
            modelFiltre = false;
        }
        // dans pgt on a déclaré x colonnes.
        // mais columnNames contient en plus :
        // - la 1ere colonne contenant les InfoRow.
        // - la colonne action
        entetes = new String[pgtable.getNbCols() + 2];// 2 premieres colonnes sans entetes
        entetes[0] = "InfoRow";
        entetes[1] = "";
        for (int i = 0; i < pgtable.getNbCols(); i++) {
            entetes[i + 2] = pgtable.getColName(i);
        }
        this.listePardesc = data;
    }
 
    public PersoTableModel(List<ParDesc> data, PgTable pgtable) {
        this(data, pgtable, PersoTable.IS_FILTRE.Non);
    }
 
    public boolean isModelFiltre() {
        return modelFiltre;
    }
 
    public void addParDesc(int row, ParDesc parDesc) {
        listePardesc.add(row, parDesc);
        fireTableRowsInserted(row, listePardesc.size() - 1);
    }
 
    public void addParDesc(ParDesc parDesc) {
        listePardesc.add(parDesc);
        fireTableRowsInserted(listePardesc.size() - 1, listePardesc.size() - 1);
    }
 
    @Override
    public int getColumnCount() {
        return entetes.length;
    }
 
    @Override
    public int getRowCount() {
        return listePardesc.size();
    }
 
    @Override
    public String getColumnName(int col) {
        return entetes[col];
    }
 
    @Override
    public Object getValueAt(int rowIndex, int columnIndex) {
        switch (columnIndex) {
            case 0:
                return listePardesc.get(rowIndex).getInfoRow();
            case 1:
                return listePardesc.get(rowIndex).getAction();
            case 2:
                return listePardesc.get(rowIndex).getRang();
            case 3:
                return listePardesc.get(rowIndex).getFld_key();
            case 4:
                return listePardesc.get(rowIndex).getLib();
            case 5:
                return listePardesc.get(rowIndex).getDebut();
            case 6:
                return listePardesc.get(rowIndex).getLg();
            case 7:
                return listePardesc.get(rowIndex).getDt_cre_maj();
            default:
                return null; //Ne devrait jamais arriver
        }
    }
 
    public InfoRow getInfoRow(int row) {
        return listePardesc.get(row).getInfoRow();
    }
 
    @Override
    public Class getColumnClass(int columnIndex) {
        switch (columnIndex) {
            case 0:
                return InfoRow.class;
            default:
                return String.class;
        }
    }
 
    @Override
    public boolean isCellEditable(int row, int col) {
        //Note that the data/cell address is constant,
        //no matter where the cell appears onscreen.
        if (col == 7 || (col == PersoTable.ACTION_COL_MODEL1 && isModelFiltre())) {
            return false;
        } else {
            return true;
        }
    }
 
    @Override
    public void setValueAt(Object value, int row, int col) {
        if (value == null) {
            return;
        }
        ParDesc pardesc = listePardesc.get(row);
        if (DEBUG) {
            System.out.println("On sette " + row + "," + col
                    + " à " + value
                    + " (instance de "
                    + value.getClass() + ")");
        }
        String newVal = "";
        if (value instanceof String) {
            newVal = ((String) value).trim();
        }
        switch (col) {
            case 0:
                pardesc.setInfoRow((InfoRow) value);
                break;
            case 1:
                pardesc.setAction(newVal);
                break;
            case 2:
                pardesc.setRang(newVal);
                break;
            case 3:
                pardesc.setFld_key(newVal);
                break;
            case 4:
                pardesc.setLib(newVal);
                break;
            case 5:
                pardesc.setDebut(newVal);
                break;
            case 6:
                pardesc.setLg(newVal);
                break;
            case 7:
                pardesc.setDt_cre_maj(newVal);
                break;
        }
         fireTableCellUpdated(row, col);
        if (DEBUG) {
            printDebugData();
        }
    }
 
    private void printDebugData() {
        int numRows = getRowCount();
        int numCols = getColumnCount();
 
        System.out.print("Après set,");
        for (int i = 0; i < numRows; i++) {
            System.out.print(" row " + i + " =");
            for (int j = 0; j < numCols; j++) {
                String toto;
                if (getValueAt(i, j) instanceof InfoRow) {
                    toto = ((InfoRow) getValueAt(i, j)).toString();
                } else {
                    toto = (String) getValueAt(i, j);
                }
                if (toto.trim().isEmpty()) {
                    toto = "_";
                }
                toto+=" |";
                System.out.print(" " + j + " " + toto);
            }
            System.out.println();
        }
    }
}