JTable неправильно возвращает выбранную строку

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

Второе изменение:

вместо этого:

  if(model==null)
    model = new AchievementTableModel(cells, columns);
  else
    model.replace(cells, columns);
  if(table==null) {
    table = new JTable(model);
    table.setFillsViewportHeight(true);
    table.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);
    table.getTableHeader().setReorderingAllowed(false);
    table.setSelectionMode(DefaultListSelectionModel.SINGLE_SELECTION);
    table.getColumnModel().setColumnSelectionAllowed(false);
    table.getTableHeader().setResizingAllowed(false);
  } else
    table.setModel(model);

  column = table.getColumn(columns[0]);
  column.setPreferredWidth(25);
  column = table.getColumn(columns[1]);
  column.setPreferredWidth(225);
  column = table.getColumn(columns[2]);
  column.setPreferredWidth(40);
  table.doLayout();

  add(new JScrollPane(table), BorderLayout.CENTER);

сделайте это вместо этого:

 if(model==null) {
    model = new AchievementTableModel(cells, columns);
 } else {
    model.setDataVector(cells, columns);
 }
 if(table==null) {
    table = new JTable(model);
    table.setFillsViewportHeight(true);
    table.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);
    table.getTableHeader().setReorderingAllowed(false);
    table.setSelectionMode(DefaultListSelectionModel.SINGLE_SELECTION);
    table.getColumnModel().setColumnSelectionAllowed(false);
    table.getTableHeader().setResizingAllowed(false);

    column = table.getColumn(columns[0]);
    column.setPreferredWidth(25);
    column = table.getColumn(columns[1]);
    column.setPreferredWidth(225);
    column = table.getColumn(columns[2]);
    column.setPreferredWidth(40);
    table.doLayout();

    add(new JScrollPane(table), BorderLayout.CENTER);
   } else {
    table.setModel(model);
   }

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

Хорошо, теперь мне интересно

Похоже, вам действительно нужно почистить свой код, потому что вокруг много ссылок.

Причина, по которой вы не видите таблицу с выбранным индексом, заключается в том, что каждый раз, когда вы создаете новый JTable, метод, в котором вы печатаете выбранную запись, по-прежнему указывает на оригинал. Поскольку теперь вы показываете «вновь» созданную таблицу, старая печатает -1.

Причина, по которой вы получаете пустую таблицу при использовании DefaultTableModel, заключается в том, что векторы являются null (возможно, полученными из комбо), и, таким образом, и данные, и заголовки исчезают из таблицы.

Вам не нужен подкласс, если вы все равно используете Object[][] в качестве данных.

Итак, вот более простой тестовый класс, который вы можете увидеть, чтобы исправить свой.

Я тестирую его как с вашим пользовательским TableModel, так и с DefaultTableModel

Это не имеет ничего общего с вашей пользовательской моделью таблицы, а с тем, как вы используете свои ссылки.

Надеюсь, это поможет.

import javax.swing.*;
import java.awt.*;
import javax.swing.table.*;
import java.util.*;
import java.awt.event.*;
public class Test { 

    private DefaultTableModel tableModel = null;
    //private AchievementTableModel tableModel = null;
    private Object []   headers = new Object[]{"Name", "Last Name"};
    private Object [][] data;
    private Object [][] dataA = new Object[][]{{"Oscar","Reyes"},{"John","Doe"}};
    private Object [][] dataB = new Object[][]{{"Color","Green"},{"Thing","Car"}};
    private JTable table;


    public static void main( String [] args ) { 
        Test test = new Test();
        test.main();
    }
    public void main() { 
        // Create the frame
        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );

        // Create the unique table.
        table = new JTable();
        frame.add(new JScrollPane( table ));

        // Add two buttons
        frame.add( new JPanel(){{ 
            // swap table model button ( simulates combo )
            add(new JButton("Change Table model"){{
                addActionListener( new ActionListener() { 
                    public void actionPerformed( ActionEvent e ) { 
                        if( tableModel == null ) { 
                            data = dataA;
                            tableModel = new DefaultTableModel( data, headers );
                            //tableModel = new AchievementTableModel( data, headers );
                            table.setModel( tableModel );
                        } else { 
                            data = data == dataA ? dataB : dataA;
                            tableModel.setDataVector( data, headers );
                            //tableModel.replace( data ); // not needed DefaultTableModel already has it.

                        }
                    }
                });
            }});
            // and print selectedRow button
            add( new JButton("Print selected row"){{
                addActionListener( new ActionListener() { 
                    public void actionPerformed( ActionEvent e ) { 
                        System.out.println(table.getSelectedRow());
                    }
                });
            }});

        }}, BorderLayout.SOUTH);

        // show the frame
        frame.pack();
        frame.setVisible( true );
    }

}

Ваш подкласс не изменился.

class AchievementTableModel extends DefaultTableModel {

    public AchievementTableModel(Object[][] c, Object[] co) {
        super.dataVector = super.convertToVector(c);
        super.columnIdentifiers = super.convertToVector(co);
    }
    public int getColumnCount() {return super.columnIdentifiers.size();}
    public int getRowCount() {return super.dataVector.size();}
    public String getColumnName(int c) {return (String)super.columnIdentifiers.get(c);}
    @SuppressWarnings("unchecked")
    public Object getValueAt(int r, int c) {return ((Vector<Object>)super.dataVector.get(r)).get(c);}
    public boolean isCellEditable(int r, int c) {return false;}
    public void replace(Object[][] c) {
        super.dataVector = super.convertToVector(c);
        super.fireTableDataChanged();
    }
}

Попробуйте и посмотрите, как он не теряет ссылку на таблицу и всегда печатает правильные selectedRow.

Сравните его с вашим кодом и исправьте его оттуда.

Может быть, попробуйте использовать

super.setDataVector (вектор данных, векторные имена столбцов);

javax.​swing.​table.​DefaultTableModel
public void setDataVector(Vector dataVector, Vector columnIdentifiers)

Из JavaDoc

Заменяет текущую переменную экземпляра dataVector новым вектором строк, dataVector. Каждая строка представлена ​​в dataVector как вектор значений объекта. columnIdentifiers — это имена новых столбцов. Первое имя в columnIdentifiers сопоставляется со столбцом 0 в dataVector. Каждая строка в dataVector настраивается в соответствии с количеством столбцов в columnIdentifiers либо путем усечения вектора, если он слишком длинный, либо путем добавления нулевых значений, если он слишком короткий. Обратите внимание, что передача значения null для dataVector приводит к неуказанному поведению, возможно, к исключению. Параметры: dataVector - новый вектор данных columnIdentifiers - имена столбцов

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

Я укажу, что по моему опыту, именно поэтому я склонен расширять AbstractTableModel или правильно реализовывать свой собственный TableModel интерфейс с нуля. Изменение ссылки на резервные данные, как здесь, всегда вызывает миллион проблем ИМХО.

Когда вы переупорядочиваете JTable, вам нужно отслеживать исходные индексы в TableModel для ваших данных, а не текущие индексы в JTable. Визуально таблица могла сместиться, но базовая модель данных — нет.

Похоже, при изменении ваш выбор теряется.

Возвращает ли "getSelectedRow()" что-либо "ДО" изменения модели?

Если это так, то сохраните этот индекс, измените модель и снова установите этот индекс.

Вероятно, вам нужно использовать пользовательскую ListSelectionModel. для этого

Еще одна вещь, о которой я подумал, когда вы делаете table= new JTable(model);, вы изменяете таблицу, на которую ссылается переменная «таблица», однако это может не вызвать автоматическое отображение новой таблицы.

Если ваша таблица содержится в ScrollPane, вам может потребоваться вызвать ScrollPane.setViewportView(table);

У меня была такая же проблема с получением -1 всегда для getSelectedRow(). Проблема уже могла быть решена. Тем не менее, размещение кода, который устранил мою проблему:

final int selectedRowIndex = table.rowAtPoint(mouseEvent.getPoint());
final int modelRowIndex = table.convertRowIndexToModel(selectedRowIndex);