修改Table Model后JTable单元格编辑获取列返回-1的问题

首先咱们拆解下你遇到的核心问题：切换列表项后，TableModelListener里调用table.getSelectedColumn()返回-1，这主要是两个关键原因导致的：

1. 监听器引用了已失效的旧JTable对象 #

每次调用SetTable时，你都会创建新的JTable实例覆盖成员变量table，但之前添加到model上的TableModelListener还持有旧的table引用——这个旧table已经被从界面移除，调用它的getSelectedColumn()自然会返回-1（因为它不在显示状态，没有任何选中列）。而且你每次给model加新监听器却不删除旧的，导致多个监听器叠加，旧监听器的错误输出会干扰判断。

2. 依赖JTable选择状态获取变更位置不可靠 #

TableModelEvent本身已经包含了模型变更的行和列信息，没必要依赖JTable的选择状态——模型变更事件和表格选择事件是独立触发的，两者没有同步保证，尤其是在替换整个表格的场景下，选择状态可能还没更新就触发了模型事件。

具体修复步骤 #

步骤1：重构监听器，改用TableModelEvent获取变更位置 #

不要再通过table.getSelectedColumn()拿位置，直接从事件对象里取：

private void addTableChangeListener(JTable targetTable) {
    TableModel model = targetTable.getModel();
    // 先移除旧的监听器，避免叠加导致的混乱
    TableModelListener[] existingListeners = model.getTableModelListeners();
    for (TableModelListener listener : existingListeners) {
        model.removeTableModelListener(listener);
    }
    // 添加新的监听器，直接从事件中获取变更位置
    model.addTableModelListener(new TableModelListener() {
        @Override
        public void tableChanged(TableModelEvent e) {
            int changedRow = e.getFirstRow();
            int changedColumn = e.getColumn();
            
            // ALL_COLUMNS表示整行/整个模型变更，这里只处理单元格编辑的情况
            if (changedColumn != TableModelEvent.ALL_COLUMNS) {
                String columnName = model.getColumnName(changedColumn);
                System.out.printf("Something changed at row %d, column %d (%s)%n", 
                    changedRow, changedColumn, columnName);
            }
        }
    });
}

步骤2：避免重复添加ListSelectionListener #

你每次调用SetList都会给list加新的选择监听器，导致点击列表项时逻辑执行多次，加重混乱。把监听器移到初始化逻辑里（比如构造函数），只添加一次：

// 在类的构造函数或初始化方法里添加一次即可
public void initListListener() {
    list.addListSelectionListener(new ListSelectionListener() {
        @Override
        public void valueChanged(ListSelectionEvent e) {
            if (!e.getValueIsAdjusting()) {
                Object selectedValue = list.getSelectedValue();
                if (selectedValue == null) return;
                
                SelectedTab = selectedValue.toString();
                try {
                    csvToArray(SelectedTab);
                    updateTableContent(); // 抽离表格更新逻辑
                } catch (Exception ex) {
                    ex.printStackTrace();
                }
            }
        }
    });
}

步骤3：规范表格更新逻辑，避免重复创建JTable（可选但推荐） #

其实没必要每次都销毁重建JTable，直接更新模型即可。如果一定要重建，也要正确清理旧组件：

private void updateTableContent() {
    // 1. 清空并重新配置模型
    model.setRowCount(0);
    Object[] columnNames = getColumnNames();
    model.setColumnIdentifiers(columnNames);
    
    // 2. 填充新数据
    if (Students != null && !StudentList.isEmpty()) {
        for (int i = 0; i < StudentList.size(); i++) {
            Object[] rowData = new Object[columnsNumber1];
            rowData[0] = Students[i];
            for (int j = 1; j < columnsNumber1; j++) {
                rowData[j] = Grades[j-1];
            }
            model.addRow(rowData);
        }
    }
    
    // 3. 如果一定要重建JTable：
    scrollPane_table.removeAll();
    table = new JTable(model);
    // 给新表格添加监听器
    addTableChangeListener(table);
    scrollPane_table.add(table);
    // 强制刷新布局
    scrollPane_table.revalidate();
    scrollPane_table.repaint();
}

// 抽离列名生成逻辑，提高可读性
private Object[] getColumnNames() {
    Object[] columnsName = new Object[columnsNumber1];
    columnsName[0] = "Name";
    for (int i = 1; i < columnsNumber1; i++) {
        if (Assignments[i] != null) {
            columnsName[i] = Assignments[i].toString();
        }
    }
    return columnsName;
}

为什么这样能解决问题？ #

• 每次添加新的TableModelListener前先移除旧的，确保只有当前有效的监听器在工作，且监听器不再依赖JTable对象，直接用Model的信息。
• 从TableModelEvent中获取变更位置，完全摆脱对JTable选择状态的依赖，这才是监听模型变更的正确姿势。
• 避免重复添加列表选择监听器，防止逻辑多次执行导致的状态混乱。

内容的提问来源于stack exchange，提问作者BimBim SalaBim