Автосортировка JTable после dataModelUpdate

Давайте на секунду отвлечемся от проблем с нарушением потока.

Вызов setDataVector для DefaultTableModel запускает TableStructureChanged, что заставляет JTable, среди прочего, отбрасывать/сбрасывать rowSorter, вместо этого просто напрямую обновлять значения в модели, например

for (int x = 0; x < 3; x++) {
    for (int y = 0; y < table.getColumnCount(); y++) {
        model.setValueAt(dice.nextInt(10), x, y);
    }
}

Вместо этого я проверил это, используя Swing Timer, чтобы убедиться, что обновления происходят правильно EDT.

import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;
import java.util.Vector;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.SwingUtilities;
import javax.swing.Timer;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.TableRowSorter;

public class Main {

    public static void main(String[] args) {
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                new Main();
            }
        });
    }

    public Main() {
        Vector<String> columNames = new Vector<>();
        columNames.add("index");
        columNames.add("years");
        columNames.add("weight");
        Vector<Vector<Object>> data = new Vector<Vector<Object>>();
        Random dice = new Random();

        JTable table = new JTable(data, columNames);
        MyFrame frame = new MyFrame(table);
        DefaultTableModel model = (DefaultTableModel) table.getModel();
        TableRowSorter<DefaultTableModel> sorter = new TableRowSorter<>(model);
        table.setRowSorter(sorter);
        sorter.setSortsOnUpdates(true);//<---------------<<<<<

        for (int x = 0; x < 3; x++) {
            model.addRow(new Vector());
            for (int y = 0; y < table.getColumnCount(); y++) {
                model.setValueAt(dice.nextInt(10), x, y);
            }
        }

        Timer timer = new Timer(2000, new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                for (int x = 0; x < 3; x++) {
                    for (int y = 0; y < table.getColumnCount(); y++) {
                        model.setValueAt(dice.nextInt(10), x, y);
                    }
                }
            }
        });
        timer.start();
    }

    public static class MyFrame extends JFrame {

        JTable table;
        Dimension dim = new Dimension(300, 500);
        JScrollPane pane;
        Vector<String> columNames = new Vector<>();
        Vector<Vector<Object>> dataVector;
        int[] i = {0, 1, -1};

        public MyFrame(JTable table) {
            pane = new JScrollPane(table);
            table.setFillsViewportHeight(true);
            setSize(300, 500);
            setPreferredSize(dim);
            setMinimumSize(dim);
            setTitle("Failing autoupdate of JTable");
            setBackground(Color.BLACK);
            add(pane);
            setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            setVisible(true);
        }
    }
}

Я решил изменить данные с помощью DefaultTableModel.setDataVector() вместо строки за строкой, которая кажется громоздкой, если меняется много строк.

Хорошо, это кажется разумным. Единственное решение, которое я, кажется, могу заставить работать, это заменить RowSorter, но использовать sortKeys из старого RowSorter

Простое использование того же экземпляра RowSorter у меня не сработало

import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;
import java.util.Vector;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.RowSorter;
import javax.swing.SwingUtilities;
import javax.swing.Timer;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.TableModel;
import javax.swing.table.TableRowSorter;

public class Main {

    public static void main(String[] args) {
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                new Main();
            }
        });
    }

    Random dice = new Random();

    public Main() {
        Vector<String> columNames = new Vector<>();
        columNames.add("index");
        columNames.add("years");
        columNames.add("weight");
        Vector<Vector<Object>> data = new Vector<Vector<Object>>();

        JTable table = new JTable();
        makeModel(table, columNames);
        MyFrame frame = new MyFrame(table);

        Timer timer = new Timer(2000, new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                makeModel(table, columNames);
            }
        });
        timer.start();
    }

    protected void makeModel(JTable table, Vector columnNames) {
        DefaultTableModel model = new DefaultTableModel(columnNames, 0);

        for (int x = 0; x < 3; x++) {
            Vector row = new Vector();
            for (int y = 0; y < model.getColumnCount(); y++) {
                row.add(dice.nextInt(10));
            }
            model.addRow(row);
        }
        TableRowSorter<DefaultTableModel> sorter = new TableRowSorter<>(model);
        RowSorter<? extends TableModel> oldSorter = table.getRowSorter();
        if (oldSorter != null) {
            sorter.setSortKeys(oldSorter.getSortKeys());
        }

        table.setModel(model);
        table.setRowSorter(sorter);
    }

    public static class MyFrame extends JFrame {

        JTable table;
        Dimension dim = new Dimension(300, 500);
        JScrollPane pane;
        Vector<String> columNames = new Vector<>();
        Vector<Vector<Object>> dataVector;
        int[] i = {0, 1, -1};

        public MyFrame(JTable table) {
            pane = new JScrollPane(table);
            table.setFillsViewportHeight(true);
            setSize(300, 500);
            setPreferredSize(dim);
            setMinimumSize(dim);
            setTitle("Failing autoupdate of JTable");
            setBackground(Color.BLACK);
            add(pane);
            setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            setVisible(true);
        }
    }
}

Просто обратите внимание, что этот тип обновления также удалит выбор столбца/строки.

Я решил изменить данные с помощью DefaultTableModel.setDataVector() вместо строки за строкой, которая кажется громоздкой, если меняется много строк.

Другим вариантом было бы не использовать DefaultTableModel, а реализовать свой собственный с использованием AbstractTableModel, где вы выполняете «пакетное» обновление базовых данных модели, а затем запускаете событие tableDataChanged. Вы должны быть осторожны, чтобы убедиться, что вы не изменяете данные, которые могут быть запрошены JTable, пока вы не завершите пакетное обновление.

Вы хотите сохранить сортировку при установке нового предпочтительного индекса? у вас есть только три заголовка столбца (ограничено), почему бы не установить приоритет для заголовков столбцов? когда вы нажимаете на один заголовок, измените его приоритет на первый, уменьшите другие на один. и снова рассчитать общую сортировку, чтобы показать их.

Я решил изменить данные с помощью DefaultTableModel.setDataVector() вместо строки за строкой, которая кажется громоздкой, если меняется много строк.

Кажется разумным, если все данные меняются, поскольку вы будете генерировать только одно событие для перерисовки всей таблицы вместо одного события для каждой строки.

Однако, если вы измените данные только в нескольких случайных строках, я бы просто обновил отдельную строку, и тогда у вас не будет проблемы с сортировкой.

Для первого подхода попробуйте следующее:

JTable table = new JTable(data, columNames); 
table.setAutoCreateColumnsFromModel(false);

это должно предотвратить повторное создание TableColumnModel при вызове setDataVector(...), что приводит к генерации события TableStructureChanged, что приводит к потере RowSorter. (или, может быть, вы все еще теряете RowSorter, я никогда не пробовал, но я знаю, что вы не потеряете пользовательские рендереры/редакторы с таким подходом).

//model.fireTableRowsUpdated(0, model.getRowCount() -1 );

Это никогда не нужно. TableModel отвечает за вызов соответствующего события при изменении данных.

Я столкнулся с той же проблемой, и после просмотра исходного кода JDK самым простым способом восстановить сортировку, по-видимому, является подклассификация JTable со следующим кодом (однако это не устраняет другую проблему - выбор также сбрасывается и должен быть восстановлен).

RowSorter<? extends TableModel> lastRowSorter;
List<? extends RowSorter.SortKey> lastSortKeys;

@Override
public void tableChanged(TableModelEvent e) {
    lastRowSorter = getRowSorter();
    lastSortKeys = (lastRowSorter == null) ? null : lastRowSorter.getSortKeys();

    super.tableChanged(e);
}

@Override
public void createDefaultColumnsFromModel() {
    super.createDefaultColumnsFromModel();
    if( lastSortKeys != null ) {
        setRowSorter(lastRowSorter);
        lastRowSorter.setSortKeys(lastSortKeys);
    }
}