JComboBox作爲帶有Overriden的Jtable CellEditor stopCellEditing修改錯誤的表格單元格

Question

我有一個使用JComboBox作爲單元格編輯器的自定義TableModel的自定義JTable。 ComboBox也有一個自定義ComboBoxModel ComboBox模型包含多個字段，這些字段將用於更新JTable後面的數據並隨後更新數據庫。

以下是顯示我遇到的問題的簡單示例。重現步驟：

1. 單擊一個細胞
2. 從ComboBox中選擇下拉列表
3. 點擊不同的細胞
4. 點擊後面的第一個選擇的小區上

值

第二個單元格將獲得第一個單元格的值。

這是怎麼發生的？爲什麼ComboBox模型在stopCellEditing存在之前更改？

import javax.swing.DefaultCellEditor; 
import javax.swing.DefaultComboBoxModel; 
import javax.swing.JComboBox; 
import javax.swing.JFrame; 
import javax.swing.JScrollPane; 
import javax.swing.JTabbedPane; 
import javax.swing.JTable; 
import javax.swing.table.DefaultTableModel; 

public class TestComboCellEditor { 

    public static void main(String[] args) { 

     TestComboCellEditor test = new TestComboCellEditor(); 
     test.go(); 
    } 

    public void go() { 

     //create the frame 
     JFrame frame = new JFrame(); 
     frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 

     // create and add a tabbed pane to the frame 
     JTabbedPane tabbedPane = new JTabbedPane(); 
     frame.getContentPane().add(tabbedPane); 
     //create a table and add it to a scroll pane in a new tab 
     final JTable table = new JTable(new DefaultTableModel(new Object[]{"A", "B"}, 5)); 
     JScrollPane scrollPane = new JScrollPane(table); 
     tabbedPane.addTab("test", scrollPane); 

     // create a simple JComboBox and set is as table cell editor on column A 
     Object[] comboElements = {"aaaaa1", "aaaaaa2", "b"}; 
     final JComboBox comboBox = new JComboBox(comboElements); 
     comboBox.setEditable(true); 
     table.getColumn("A").setCellEditor(new DefaultCellEditor(comboBox) { 
      @Override 
      public boolean stopCellEditing() { 
       if (comboBox.isEditable()) { 
        DefaultComboBoxModel comboModel = (DefaultComboBoxModel) comboBox.getModel(); 
        String selectedItem = (String) comboModel.getSelectedItem(); 
        int selectedIndex = comboModel.getIndexOf(selectedItem); 
        if (!(selectedIndex == -1)) { 
         // the selected item exists as an Option inside the ComboBox 
         DefaultTableModel tableModel = (DefaultTableModel) table.getModel(); 
         int selectedRow = table.getSelectedRow(); 
         int selectedColumn = table.getSelectedColumn(); 
         tableModel.setValueAt(selectedItem, selectedRow, selectedColumn); 
        } else if (selectedItem != null) { 
         // missing code - adding new info to a custom JComboBox model and to alter info inside a custom table model 
        } 
       } 
       return super.stopCellEditing(); 
      } 
     }); 

     // pack and show frame 
     frame.pack(); 
     frame.setVisible(true); 

    } 
} 

Answer 1

好的，我做了一些改變，我想我有些工作。如果這不是最佳做法，並且您得到了更好的實施，請發佈答案。

編輯：不要按照下面的例子！根據kleopatra的評論，這是一個錯誤的實施。我在這裏離開它，所以你知道如何不是來做到這一點。

import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener; 
import javax.swing.DefaultCellEditor; 
import javax.swing.DefaultComboBoxModel; 
import javax.swing.JComboBox; 
import javax.swing.JFrame; 
import javax.swing.JScrollPane; 
import javax.swing.JTabbedPane; 
import javax.swing.JTable; 
import javax.swing.table.DefaultTableModel; 

public class TestComboCellEditor { 

    public static void main(String[] args) { 

     TestComboCellEditor test = new TestComboCellEditor(); 
     test.go(); 
    } 

    public void go() { 

     //create the frame 
     JFrame frame = new JFrame(); 
     frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 

     // create and add a tabbed pane to the frame 
     JTabbedPane tabbedPane = new JTabbedPane(); 
     frame.getContentPane().add(tabbedPane); 
     //create a table and add it to a scroll pane in a new tab 
     final JTable table = new JTable(new DefaultTableModel(new Object[]{"A", "B"}, 5)); 
     JScrollPane scrollPane = new JScrollPane(table); 
     tabbedPane.addTab("test", scrollPane); 

     // create a simple JComboBox and set is as table cell editor on column A 
     Object[] comboElements = {"aaaaa1", "aaaaaa2", "b"}; 
     final JComboBox comboBox = new JComboBox(comboElements); 
     comboBox.setEditable(true); 
     table.getColumn("A").setCellEditor(new DefaultCellEditor(comboBox) { 
      @Override 
      public boolean stopCellEditing() { 
       if (comboBox.isEditable()) { 
        DefaultComboBoxModel comboModel = (DefaultComboBoxModel) comboBox.getModel(); 
        String selectedItem = (String) comboModel.getSelectedItem(); 
        int selectedIndex = comboModel.getIndexOf(selectedItem); 
        if (!(selectedIndex == -1)) { 
         comboBox.actionPerformed(new ActionEvent(this, selectedIndex, "blabla")); 
        } else if (selectedItem != null) { 
         // missing code - adding new info to a custom JComboBox model and to alter info inside a custom table model 
        } 
       } 
       return super.stopCellEditing(); 
      } 
     }); 

     comboBox.addActionListener(new ActionListener() { 
      @Override 
      public void actionPerformed(ActionEvent e) { 
       // the selected item exists as an Option inside the ComboBox 
       if (e.getActionCommand().equals("blabla")) { 
        DefaultComboBoxModel comboModel = (DefaultComboBoxModel) comboBox.getModel(); 
        String selectedItem = (String) comboModel.getSelectedItem(); 
        DefaultTableModel tableModel = (DefaultTableModel) table.getModel(); 
        int selectedRow = table.getSelectedRow(); 
        int selectedColumn = table.getSelectedColumn(); 
        tableModel.setValueAt(selectedItem, selectedRow, selectedColumn); 
       } 
      } 
     }); 

     // pack and show frame 
     frame.pack(); 
     frame.setVisible(true); 

    } 
} 

Answer 2

這裏是保持所有的代碼編輯器的方法：

import java.awt.*; 
import java.awt.event.*; 
import javax.swing.*; 
import javax.swing.table.*; 

public class TestComboCellEditor { 

    public static void main(String[] args) { 

     TestComboCellEditor test = new TestComboCellEditor(); 
     test.go(); 
    } 

    public void go() { 

     //create the frame 
     JFrame frame = new JFrame(); 
     frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 

     // create and add a tabbed pane to the frame 
     JTabbedPane tabbedPane = new JTabbedPane(); 
     frame.getContentPane().add(tabbedPane); 
     //create a table and add it to a scroll pane in a new tab 
     final JTable table = new JTable(new DefaultTableModel(new Object[]{"A", "B"}, 5)); 
     JScrollPane scrollPane = new JScrollPane(table); 
     tabbedPane.addTab("test", scrollPane); 

     // create a simple JComboBox and set is as table cell editor on column A 
     Object[] comboElements = {"aaaaa1", "aaaaaa2", "b"}; 
     final JComboBox comboBox = new JComboBox(comboElements); 
     comboBox.setEditable(true); 
     table.getColumn("A").setCellEditor(new DefaultCellEditor(comboBox) 
     { 
      private Object originalValue; 

      @Override 
      public Component getTableCellEditorComponent(JTable table, Object value, boolean isSelected, int row, int column) 
      { 
       originalValue = value; 
       return super.getTableCellEditorComponent(table, value, isSelected, row, column); 
      } 

      @Override 
      public boolean stopCellEditing() 
      { 
       JComboBox comboBox = (JComboBox)getComponent(); 
       DefaultComboBoxModel comboModel = (DefaultComboBoxModel) comboBox.getModel(); 
       Object editingValue = getCellEditorValue(); 

       // Needed because your TableModel is empty 

       if (editingValue == null) 
        return super.stopCellEditing(); 

       int selectedIndex = comboModel.getIndexOf(editingValue); 

       // Selecting item from model 

       if (! (selectedIndex == -1)) 
        return super.stopCellEditing(); 

       // Confirm addition of new value 

       int result = JOptionPane.showConfirmDialog(
        comboBox.getParent(), 
        "Add (" + editingValue + ") to table?", 
        "Update Model", 
        JOptionPane.YES_NO_OPTION); 

       if (result == JOptionPane.YES_OPTION) 
       { 
        comboBox.addItem(editingValue); 
        return super.stopCellEditing(); 
       } 
       else 
       { 
        comboBox.removeItem(editingValue); 
        comboBox.setSelectedItem(originalValue); 
        return false; 
       } 
      } 
     }); 

     // pack and show frame 
     frame.pack(); 
     frame.setVisible(true); 

    } 
}