Bug in Java? - Problem beim Ändern von Spaltentiteln

[p.h.o.e.n.i.x]

Hallo,

mir ist durch Zufall aufgefallen, dass die Spaltentitel einer

JTable

nach einer Änderung nicht aktualisiert werden, wenn ich das Verschieben von Spalten durch

setReorderingAllowed(false)

deaktiviere.

Ich habe ein kleines Beispiel geschrieben, damit es sich jeder selber ansehen kann:

package swing.table;

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;
import java.util.Vector;

import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.WindowConstants;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.JTableHeader;
import javax.swing.table.TableColumn;
import javax.swing.table.TableColumnModel;

/**
 * @author jens goetz
 */
public class ReorderingAllowed {
 // ======
 // FIELDS
 // ======
    private final static int ASCENDING = 0;
    private final static int DESCENDING = 1;
    
    
    private static class DefaultTableModelComparator implements Comparator {
        
        private final int m_order;
        private final int m_col;
        
        
        public DefaultTableModelComparator(final int order, final int col) {
            m_order = order;
            m_col = col;
        }
        
        
        /**
         * @see java.util.Comparator#compare(java.lang.Object, java.lang.Object)
         */
        public int compare(final Object o1, final Object o2) {
            Vector row_1 = (Vector)o1;
            Vector row_2 = (Vector)o2;
            
            String valRow_1 = (String)row_1.get(m_col);
            String valRow_2 = (String)row_2.get(m_col);
            
            int result = valRow_1.compareToIgnoreCase(valRow_2);
            
            return (m_order == ASCENDING) ? result : -result;
        }
    };
 // ============
 // CONSTRUCTORS
 // ============
 // =======
 // METHODS
 // =======
    /**
     * @param args
     */
    public static void main(String[] args) {
        // init columns
        final String[] tblColumns = new String[]{
                "A",
                "B",
                "C"
        };
        
        // init rows
        String[][] tblData = new String[][]{
                {"1", "2", "3"},
                {"4", "5", "6"},
                {"7", "8", "9"}
        };
        
        // init table model and table
        DefaultTableModel tblModel =
            new DefaultTableModel(tblData, tblColumns);
        final JTable tbl = new JTable(tblModel);
        
        JTableHeader tblHeader = tbl.getTableHeader();
        
        /*
         * if line is uncommented and flag is setted to false, then changing a
         * column header title do not result in repainting the table
         */
        tblHeader.setReorderingAllowed(false);
        
        tblHeader.addMouseListener(new MouseAdapter() {            
            private int m_lastSortedCol = -1;
            private int m_lastSortedOrder = -1;
            
            /**
             * @see java.awt.event.MouseAdapter#
             *          mouseClicked(java.awt.event.MouseEvent)
             */
            public void mouseClicked(MouseEvent e) {
                DefaultTableModel tblModel = (DefaultTableModel)
                    tbl.getModel();
                
                // get column index
                TableColumnModel columnModel = tbl.getColumnModel();
                int col = columnModel.getColumnIndexAtX(e.getX());
                
                assert (col > -1
                        && col < columnModel.getColumnCount());
                
                /*
                 * if another column is clicked, then remove order text
                 */
                if (m_lastSortedCol != -1
                    && m_lastSortedCol != col) {
                    TableColumn column =
                        columnModel.getColumn(m_lastSortedCol);
                    
                    column.setHeaderValue(tblColumns[m_lastSortedCol]);
                }
                
                TableColumn column = columnModel.getColumn(col);
                
                int isAscendingDescending;
                
                /*
                 * if column is clicked the first time or another column is
                 * clicked, then sort column ascending - else sort column
                 * descending or switch between ascending and descending when
                 * column is clicked more than one time
                 */
                if (m_lastSortedCol == -1
                    || m_lastSortedCol != col) {
                    isAscendingDescending = ASCENDING;
                    m_lastSortedOrder = -1;
                } else {
                    isAscendingDescending =
                        (m_lastSortedOrder == -1
                         || m_lastSortedOrder == ASCENDING) ? DESCENDING
                                 : ASCENDING;
                }
                
                // sort table data
                List tblData = tblModel.getDataVector();
                Collections.sort(tblData, new DefaultTableModelComparator(
                        isAscendingDescending, col));
                
                tblModel.fireTableDataChanged();
                
                // set order text
                String appendix =
                    isAscendingDescending == ASCENDING
                        ? " Up"
                        : " Down";
                
                column.setHeaderValue(tblColumns[col] + appendix);
                
                /*
                 * try this - it will not help, when
                 * tblHeader.setReorderingAllowed(false) was called, before
                 */
                tbl.repaint();
                
                m_lastSortedCol = col;
                m_lastSortedOrder = isAscendingDescending;
            }
        });
        
        JScrollPane scr = new JScrollPane();
        scr.setViewportView(tbl);
        
        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        
        frame.setTitle("Reordering Allowed");
        
        Container content = frame.getContentPane();
        content.setLayout(new BorderLayout());
        content.add(scr, BorderLayout.CENTER);
        
        frame.pack();
        frame.setVisible(true);
    }
}

Wenn ihr die Zeile

tblHeader.setReorderingAllowed(false);

auskommentiert, dann werdet ihr sehen, dass nach einem Klick auf einen Spaltentitel der Zusatz "Up" oder "Down" im Spaltentitel angezeigt wird. Ansonsten wird die Ansicht nur aktualisiert, wenn ihr z.B. eine Spalte in der Größe verändert.

Auch ein

tbl.repaint();

(oder meinetwegen auch

tbl.revalidate();

) behebt dieses Problem nicht.

Kann mir jemand weiterhelfen?

Viele Grüße



Jens

PS: Ich verwende Java 1.4

 

[Michael...]

vielleicht hilft da ein tbl.getTableHeader().repaint()
m.W ist der TableHeader ja nicht Bestandteil des Tables sondern der ScrollPane

 

[p.h.o.e.n.i.x]

Danke Michael,

so funktioniert es.

Allerdings verstehe ich immer noch nicht, warum sich das repaint() vom TableHeader anders verhält, wenn ich das Verschieben von Spalten durch

setReorderingAllowed(false)

deaktivere. Ich habe mal eine Suche nach Referenzen im Workspace auf den getter ausgeführt - und der getter wird eigentlich nur bei einem mousePressed event in der Klasse

javax.swing.plaf.basic.BasicTableHeaderUI$MouseInputHandler

ausgelesen ...

Naja, hauptsache es geht erstmal ...

Viele Grüße

Jens