Exception when I delete the last rwo of the JTable using the button I put in one of the cells.

[raminaa niilian]

Hi
Thank you for reading my post.
My problem is tat when I am trying to delete the last row in a JTable using a button that I placed in one of its columns I receive an exception which cripple the JTable.
I was dealing with this error for one day and I am nowhere. Please let me know if you have any suggestion about it.
I developed following two classes to add a button to one cell of a JTable:

The TableCellRenderer implementation:

public class ButtonRenderer implements TableCellRenderer {

    JButton button = new JButton();

    public Component getTableCellRendererComponent(JTable table,
            Object value,
            boolean isSelected,
            boolean hasFocus,
            int row, int column) {
        button.setText(value.toString());
        return button;
    }
}

The TableCellEditor implementation:

public class ButtonEditor extends AbstractCellEditor implements TableCellEditor,
        ActionListener {

    JTable table;
    JButton button = new JButton();
    int clickCountToStart = 1;

    public ButtonEditor(JTable table) {
        this.table = table;
        button.addActionListener(this);
    }

    public void actionPerformed(ActionEvent e) {

        int row = table.getEditingRow();
        int col = table.getEditingColumn();
        DefaultTableModel mod = (DefaultTableModel) table.getModel();
        mod.removeRow(row);
        mod.fireTableDataChanged();
        stopCellEditing();

    }

    public Component getTableCellEditorComponent(JTable table,
            Object value,
            boolean isSelected,
            int row, int column) {
        button.setText(value.toString());
        return button;
    }

    public Object getCellEditorValue() {
        return button.getText();
    }

    public boolean isCellEditable(EventObject anEvent) {
        if (anEvent instanceof MouseEvent) {
            return ((MouseEvent) anEvent).getClickCount() >= clickCountToStart;
        }
        return true;
    }
}

And finally here is how I load the data and use these two classes to add the button to the Jtable:

for (Iterator<Employee> it = l.iterator(); it.hasNext();) {
            Employee employee = it.next();

            String[] row = {"Delete", employee.getGrade(), employee.getSalary(), Integer.toString(employee.getID())};

            ((DefaultTableModel) t.getModel()).addRow(row);
        }



        TableColumnModel colModel = t.getColumnModel();
        colModel.getColumn(0).setCellRenderer(new ButtonRenderer());
        colModel.getColumn(0).setCellEditor(new ButtonEditor(t));
        ((DefaultTableModel) t.getModel()).fireTableDataChanged();

It works perfect when I press delete button for all rows except for the last row. either it is the only row in the table or there are tens of rows in the table, when I press the delete button for the last row it return the followin exception: The exception happened when my table had 7 rows and I delete the 7th row.

SEVERE [global]
java.lang.ArrayIndexOutOfBoundsException: 6 >= 6
        at java.util.Vector.elementAt(Vector.java:427)
        at javax.swing.table.DefaultTableModel.setValueAt(DefaultTableModel.java:648)
        at javax.swing.JTable.setValueAt(JTable.java:2719)
        at javax.swing.JTable.editingStopped(JTable.java:4721)
        at javax.swing.AbstractCellEditor.fireEditingStopped(AbstractCellEditor.java:125)
        at javax.swing.AbstractCellEditor.stopCellEditing(AbstractCellEditor.java:68)
        at com.bsm.utils.ButtonEditor.actionPerformed(ButtonEditor.java:36)
        at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
        at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
        at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
        at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
        at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
        at java.awt.Component.processMouseEvent(Component.java:6263)
        at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
        at java.awt.Component.processEvent(Component.java:6028)
        at java.awt.Container.processEvent(Container.java:2041)
        at java.awt.Component.dispatchEventImpl(Component.java:4630)
        at java.awt.Container.dispatchEventImpl(Container.java:2099)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at javax.swing.plaf.basic.BasicTableUI$Handler.repostEvent(BasicTableUI.java:924)
        at javax.swing.plaf.basic.BasicTableUI$Handler.mouseReleased(BasicTableUI.java:1138)
        at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:273)
        at java.awt.Component.processMouseEvent(Component.java:6263)
        at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
        at java.awt.Component.processEvent(Component.java:6028)
        at java.awt.Container.processEvent(Container.java:2041)
        at java.awt.Component.dispatchEventImpl(Component.java:4630)
        at java.awt.Container.dispatchEventImpl(Container.java:2099)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
        at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4238)
        at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
        at java.awt.Container.dispatchEventImpl(Container.java:2085)
        at java.awt.Window.dispatchEventImpl(Window.java:2475)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
        at org.netbeans.core.TimableEventQueue.dispatchEvent(TimableEventQueue.java:104)
[catch] at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
        at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

[pete stein]

Question answered in OP's cross-post at forums.sun.com.

[N. Priore]

pete stein wrote:Question answered in OP's cross-post at forums.sun.com.

And the link to that page would be what? For those of us that would like to see the answer.

[Rob Spoor]

Raminaa, http://faq.javaranch.com/java/BeForthrightWhenCrossPostingToOtherSites

[Rob Camick]

N. Priore wrote:

pete stein wrote:Question answered in OP's cross-post at forums.sun.com.

And the link to that page would be what? For those of us that would like to see the answer.

Actually no answer to the question was given.

The answer provided, explains why people have lost interest in attempting to help this individual. Anway here is the
link

[N. Priore]

Rob Camick wrote:

N. Priore wrote:

pete stein wrote:Question answered in OP's cross-post at forums.sun.com.

And the link to that page would be what? For those of us that would like to see the answer.

Actually no answer to the question was given.

The answer provided, explains why people have lost interest in attempting to help this individual. Anway here is the
link

Thank you. I appreciate the response.

[Maneesh Godbole]

Thank you Pete, Rob.
One more user to my personal ignore list.

[Doua Beri]

#     public void actionPerformed(ActionEvent e) {  
#   
#         int row = table.getEditingRow();  
#         int col = table.getEditingColumn();  
#         DefaultTableModel mod = (DefaultTableModel) table.getModel();  
#         mod.removeRow(row);  
#         mod.fireTableDataChanged();  
#         stopCellEditing();  
#   
#     } 

try to add before you delete the row

fireEditingStopped();

hope this helps you.