17

I've tried this:

public void removeSelectedFromTable(JTable from)
{
    int[] rows = from.getSelectedRows();
    TableModel tm= from.getModel();

    while(rows.length>0)
    {
        ((DefaultTableModel)tm).removeRow(from.convertRowIndexToModel(rows[0]));

        rows = from.getSelectedRows();
    }
    from.clearSelection();
}

But, it sometimes leaves one still there. What can be the problem?

1
  • This didn't worked for me
    – chamzz.dot
    Nov 12, 2017 at 17:27

18 Answers 18

25

It doesn't work, this is better:

public void removeSelectedRows(JTable table){
   DefaultTableModel model = (DefaultTableModel) this.table.getModel();
   int[] rows = table.getSelectedRows();
   for(int i=0;i<rows.length;i++){
     model.removeRow(rows[i]-i);
   }
}
4
  • 2
    So you subtract i to account for index difference from the rows which have already been removed, but that assumes the rows which have been removed already are above the row that's "about to be" removed. Is that always a safe assumption? May 5, 2011 at 3:24
  • it's not. 'for' should go other way, with "i--"
    – Bugari
    Sep 24, 2012 at 16:05
  • This does not account for table sorting.
    – SingleShot
    Jul 7, 2013 at 18:55
  • 2
    -1 Doesn't work with sort. You should use table.convertRowIndexToModel(index);
    – barwnikk
    Sep 1, 2014 at 12:31
11
int[] selectedRows = getTable().getSelectedRows();
        if (selectedRows.length > 0) {
            for (int i = selectedRows.length - 1; i >= 0; i--) {
                tableModel.removeRow(selectedRows[i]);
            }
        }
7

This worked for me

int numRows = table.getSelectedRows().length;
for(int i=0; i<numRows ; i++ ) {

    m_tableModel.removeRow(table.getSelectedRow());
}
1
  • Voted up as this works perfectly! Because getSelectedRow() returns the index of the first selected row...
    – trinity420
    Jul 3, 2016 at 21:06
5

Java 8+ streaming solution:

JTable table;
DefaultTableModel model = (DefaultTableModel) table.getModel();

IntStream.of(table.getSelectedRows())
            .boxed()
            .sorted(Collections.reverseOrder())
            .map(table::convertRowIndexToModel) // support for sorted table
            .forEach(model::removeRow);
1
  • 1
    I really liked this solution but it didn't account for sorted tables.. I've edited the answer to also give this solution. Jan 31, 2020 at 15:24
3
removeRows: {
    int[] lines = t.getSelectedRows();
    if(lines.length==0)
        break removeRows;
    for(int i=0; i<lines.length; i++) {
        lines[i]=t.convertRowIndexToModel(lines[i]);
    }
    Arrays.sort(lines);
    for(int i=lines.length-1; i>=0; i--) {
        dtm.removeRow(lines[i]);    
    }
}

Working with sorting, moving columns, multiple selection, etc.

1
  • The trick is to sort the rows to remove and work from bottom up!
    – David I.
    Dec 2, 2015 at 20:36
2

try this is perfect work and simple

DefaultTableModel df1 = (DefaultTableModel) jTable1.getModel();
    int rs[] = jTable1.getSelectedRows();
    for (int i = rs.length-1; i >=0 ; i--) {

        int k = rs[i];

        df1.removeRow(k);

    }
1

Why not just use this,

public void removeSelectedFromTable(JTable from)
{
        int[] rows = from.getSelectedRows();
        TableModel tm = (DefaultTableModel) from.getModel();


        for (int row : rows) {
            tm.removeRow(from.convertRowIndexToModel(row));
        }

        from.clearSelection();
}
0
1

This code work for me!

DefaultTableModel model = (DefaultTableModel) Table.getModel();
    int nbrrow=Table.getRowCount();

        if(nbrrow>=1){
            int[] rows = Table.getSelectedRows();
            if(rows.length>=1){
            if(Table.getModel().isCellEditable(Table.getSelectedRow(),Table.getSelectedColumn())
                       &&Table.isEditing()==true)
               {
                   Table.getCellEditor().stopCellEditing();
                    for(int i=0;i<rows.length;i++){ 
                     model.removeRow(rows[i]-i);
                   }
               }else{
                            for(int i=0;i<rows.length;i++){ 
                             model.removeRow(rows[i]-i);
                           }

                }
            for(int i=0;i<Table.getRowCount();i++){
                           model.setValueAt(i, i, 0);
                       }
              Table.revalidate();
              Table.repaint();
            }
        }
1
int[] selectedRows = table.getSelectedRows();
List<Object> selectedObjects = new ArrayList<Object>();
for (int row : selectedRows) {
    selectedObjects.add(tableModel.getObject(row));
}
for (Object obj : selectedObjects) {
    tableModel.removeObject(obj);
}

// My TableModel is a subclass of this:

public abstract class ListTableModel<T> extends AbstractTableModel {

    private List<T> data;

    public void setData(List<T> data) {
        this.data = data;
    }

    public T getObject(int row) {
        return data.get(row);
    }

    @Override
    public int getRowCount() {
        return data.size();
    }

}
1
  • DefaultTableModel doesn't have method "getObject" :(
    – barwnikk
    Sep 1, 2014 at 12:14
0

Try this

    public void removeSelectedFromTable(JTable table) {

        DefaultTableModel model = (DefaultTableModel) table.getModel();
        int indexes[] = table.getSelectedRows(); 
        int res = 0;
        for(int i = 0; i < indexes.length; i++) {
            res += (i>0)?(indexes[i]-indexes[i-1]-1):0;
            int index = table.convertRowIndexToModel(indexes[0]+res)
            model.removeRow(index);
        }
    }
0

Try this:

int selectedIndex = table_.getSelectedRow();
while (selectedIndex >= 0) {
    int modelIndex = table_.convertRowIndexToModel(selectedIndex);
    tableModel_.removeRow(modelIndex);

    selectedIndex = table_.getSelectedRow();
}
0

It is woked for me.enter code here

public void removeSelectedFromTable(JTable from)
    {
        int[] rows = from.getSelectedRows();
        DefaultTableModel tm = (DefaultTableModel) from.getModel();
        for (int i = rows.length-1; i >= 0; i--) {
            tm.removeRow(rows[i]);
        }
}
0

The code below will work:

do {
    ((DefaultTableModel) jXTable1.getModel()).removeRow(jXTable1.getSelectedRows()[0]);
} while (jXTable1.getSelectedRowCount() > 0);
0

That works for me:

if(tablaNotif.getSelectedRows().length>0){
    int[] rows = tablaNotif.getSelectedRows();
    if(tablaNotif.getSelectedRows().length<2) {
        modelo.remove(tablaNotif.convertRowIndexToModel(rows[0]));
    } else{
        for(int i = 0; i<tablaNotif.getSelectedRows().length;i++) {
            modelo.remove(tablaNotif.convertRowIndexToModel(rows[i]));
        }
        modelo.remove(tablaNotif.convertRowIndexToModel(rows[(tablaNotif.getSelectedRows().length-1)]));
    }
}
0

The stream solution above doesn't take into account sorting, it may be fixed this way:

        IntStream.of(table.getSelectedRows())
            .boxed().map(i -> table.convertRowIndexToModel(i))
            .sorted(Collections.reverseOrder())
            .forEach(((DefaultTableModel)table.getModel())::removeRow);
-2

I looked at all the solutions, but there are some exceptions, so I would like to add this answer.

for (int i = 0; i < raws.length; i++) {
    System.out.println("I : " + i);
    for (int j = 1; j <= 1; j++) {
        defaultTableModel.removeRow(0);
        jTStockJournal.clearSelection();
        defaultTableModel.fireTableDataChanged();
    }
}
1
  • -1 for firing events on behalf of the model (it's the model's own job)
    – kleopatra
    Mar 13, 2012 at 15:22
-2

That solution works with header sorting and updates automatically the table (i.e.: it is not required to rebuid it).

The "Collecions.sort in reverse order" ennables to avoid IndexOutOfBondException.

private void deleteSelectedRows() {
        int[] selectedRows = table.getSelectedRows();
        table.clearSelection();

        // get model rows
        List<Integer> selectedModelRows = new LinkedList<Integer>();
        for (int i =`enter code here` 0; i < selectedRows.length; i++) {
            selectedModelRows.add(table.convertRowIndexToModel(selectedRows[i]));
        }

        Collections.sort(selectedModelRows, Collections.reverseOrder());

        for (int selectedModelRow : selectedModelRows) {
            tableModel.removeRow(selectedModelRow);
            tableModel.fireTableRowsDeleted(selectedModelRow, selectedModelRow);
        }
}
1
  • 1
    -1 for firing events on behalf of the model (it's the model's own job)
    – kleopatra
    Mar 13, 2012 at 15:22
-2

private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) {
DefaultTableModel defaultModel = (DefaultTableModel) jTable1.getModel(); Vector newRow = new Vector(); defaultModel.removeRow(jTable1.getSelectedRow()); i = i - 1; }

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.