DefaultTableModelのremoveRow（）メソッドに関して

Question

実現したいこと

• JTableに表示されている表から行を選択して削除したい

前提

ここに質問の内容を詳しく書いてください。
Jframeでテーブルを表示させて操作させる簡単なシステムを作っています。
テーブルに表示されている行を選択して、ボタンを押下することで選択行が削除される
機能を搭載しているのですが、「java.lang.ArrayIndexOutOfBoundsException:」が発生しました。

テーブルの行数と選択行がうまく「removeRow()」に送られていないのでしょうか？

発生している問題・エラーメッセージ

【エラーメッセージ】
Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 2 >= 0
	at java.base/java.util.Vector.removeElementAt(Vector.java:549)
	at java.desktop/javax.swing.table.DefaultTableModel.removeRow(DefaultTableModel.java:469)
	at practice.no14.ListPractice.actionPerformed(ListPractice.java:163)
	at java.desktop/javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1972)
	at java.desktop/javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2313)
	at java.desktop/javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:405)
	at java.desktop/javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:262)
	at java.desktop/javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:279)
	at java.desktop/java.awt.Component.processMouseEvent(Component.java:6620)
	at java.desktop/javax.swing.JComponent.processMouseEvent(JComponent.java:3398)
	at java.desktop/java.awt.Component.processEvent(Component.java:6385)
	at java.desktop/java.awt.Container.processEvent(Container.java:2266)
	at java.desktop/java.awt.Component.dispatchEventImpl(Component.java:4995)
	at java.desktop/java.awt.Container.dispatchEventImpl(Container.java:2324)
	at java.desktop/java.awt.Component.dispatchEvent(Component.java:4827)
	at java.desktop/java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4948)
	at java.desktop/java.awt.LightweightDispatcher.processMouseEvent(Container.java:4575)
	at java.desktop/java.awt.LightweightDispatcher.dispatchEvent(Container.java:4516)
	at java.desktop/java.awt.Container.dispatchEventImpl(Container.java:2310)
	at java.desktop/java.awt.Window.dispatchEventImpl(Window.java:2780)
	at java.desktop/java.awt.Component.dispatchEvent(Component.java:4827)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:775)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:720)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:714)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:97)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:747)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:745)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:744)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

### 該当のソースコード

```ここに言語名を入力
【ソースコード】
public class ListPractice extends JFrame implements ActionListener {

	final JTable table;
	final TableModel tableModel = new TableModel();
	final TableRowSorter<TableModel> sorter = new TableRowSorter<TableModel>(tableModel);
	final JTextField filterText = new JTextField();
	final DefaultTableModel model = new DefaultTableModel();

	ListPractice() {
		// フレームのセットアップ
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		// 画面サイズの決定
		setSize(400, 300);
		// 画面位置の設定（null指定で画面中央）
		setLocationRelativeTo(null);

		table = new JTable(model);

		ListSelectionModel lsModel = table.getSelectionModel();
		lsModel.addListSelectionListener(new ListSelectionListener() {

			@Override
			public void valueChanged(ListSelectionEvent e) {
				// TODO 自動生成されたメソッド・スタブ
				if (e.getValueIsAdjusting()) {
					return;
				}
				ListSelectionModel lsm =
						(ListSelectionModel) e.getSource();
			}
		});

		// フィルターパネル
		JPanel filterPanel = new JPanel();
		// 横レイアウトを指定（X_AXIS）
		BoxLayout boxLayout = new BoxLayout(filterPanel, BoxLayout.X_AXIS);
		filterPanel.setLayout(boxLayout);


		JLabel filterLabel = new JLabel("検索");
		filterPanel.add(filterLabel);

		// 検索処理
		filterText.addActionListener(new ActionListener() {

			@Override
			public void actionPerformed(ActionEvent e) {
				// TODO 自動生成されたメソッド・スタブ
				RowFilter<TableModel, Object> filter = null;
				try {
					filter = RowFilter.regexFilter(filterText.getText(), 0);
				} catch (Exception ex) {
					// TODO: handle exception
				}
				sorter.setRowFilter(filter);
			}
		});
		filterLabel.setLabelFor(filterText);
		filterPanel.add(filterText);

		add(filterPanel,BorderLayout.NORTH);


		// フィルターパネル
		JPanel filterPanel2 = new JPanel();
		// 縦レイアウトを指定（Y_AXIS）
		BoxLayout boxLayout2 = new BoxLayout(filterPanel2, BoxLayout.Y_AXIS);
		filterPanel2.setLayout(boxLayout2);

		// 削除ボタン作成
		JButton Rbutton = new JButton("削除");
		Rbutton.setAlignmentX(RIGHT_ALIGNMENT);
		//Rbutton.addActionListener(new ClickAction());
		// filterPanel2にボタンを登録
		filterPanel2.add(Rbutton);

		// 削除処理
		Rbutton.setActionCommand("DELETE");
		Rbutton.addActionListener(this);

		// テーブルの設定
		table.setRowSorter(sorter);
		table.setModel(tableModel);

		add(filterPanel2,BorderLayout.CENTER);

		TableColumn col = table.getColumnModel().getColumn(0);

		JScrollPane sp = new JScrollPane(table);
		filterPanel2.add(sp);

		//getContentPane().add(sp, BorderLayout.CENTER);
		getContentPane().add(filterPanel, BorderLayout.NORTH);
		getContentPane().add(filterPanel2, BorderLayout.CENTER);
	}

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
		SwingUtilities.invokeLater(new Runnable() {

			@Override
			public void run() {
				// TODO 自動生成されたメソッド・スタブ
				ListPractice app = new ListPractice();
				app.setVisible(true);
			}
		});

	}

**	@Override
	public void actionPerformed(ActionEvent e) {
		// TODO 自動生成されたメソッド・スタブ
		String command = (String) e.getActionCommand();
		// 削除ボタンが押下された時の処理
		if ("DELETE".equals(command)) {
			ListSelectionModel lsModel = table.getSelectionModel();
			for (int i=0; i< table.getRowCount(); i++) {
				if (lsModel.isSelectedIndex(i)) {
					model.removeRow(i);
				}
			}
			table.repaint();
		}
	}**
}

【ソースコード】
public class TableModel extends AbstractTableModel implements Serializable {
	Object[][] data = {
		{"田中", "大阪", true},
		{"長谷川", "東京", false},
		{"藤原", "奈良", false},
		{"橋本", "和歌山", false},
		{"山田", "北海道", false}
	};

	String[] columns = {"名前", "出身", "選択"};

	@Override
	public Class<?> getColumnClass(int columnIndex) {
		return data[0][columnIndex].getClass();
	}

	// データのカラム数を返す
	@Override
	public int getColumnCount() {
		// TODO 自動生成されたメソッド・スタブ
		return columns.length;
	}

	// データの行数を返す
	@Override
	public int getRowCount() {
		// TODO 自動生成されたメソッド・スタブ
		return data.length;
	}

	// カラム名をStringで返す
	@Override
	public String getColumnName(int column) {
		// TODO 自動生成されたメソッド・スタブ
		return columns[column];
	}


	@Override
	public Object getValueAt(int rowIndex, int columnIndex) {
		// TODO 自動生成されたメソッド・スタブ
		return data[rowIndex][columnIndex];
	}


	// セルの編集に関するメソッド
	@Override
	public boolean isCellEditable(int rowIndex, int columnIndex) {
		// TODO 自動生成されたメソッド・スタブ
		// 編集可能なセルの設定
		if(columnIndex == 2) {
			return true;
		} else {
			return false;
		}
	}

	@Override
	public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
		// TODO 自動生成されたメソッド・スタブ
		data[rowIndex][columnIndex] = aValue;
		fireTableCellUpdated(rowIndex, columnIndex);
	}

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
	}

}

### 補足情報（FW/ツールのバージョンなど）

参考にしたサイト
http://asistobe851.web.fc2.com/my-memo/RowManagement.html

初学者のため無駄なコードがあると思いますが、ご容赦ください。

Comments

[jimbe]

エラーメッセージやコードはファイル毎等別々のマークダウンとし、コード以外の説明等はコードのマークダウンの外に出してください。コードのマークダウンは最小化/最大化が出来、通常は最小状態で表示されますので、パっと見で説明があるのが分かりません。
コードのマークダウン内では修飾等は使えません。

また、javafx は swing とは無関係ですので、タグには入れないでください。

Answer 1

テーブルの行数と選択行がうまく「removeRow()」に送られていないのでしょうか？

いいえ。TableModel tableModelと、DefaultTableModel modelがごっちゃになっています。

tableModelにはデータがありますが、modelはカラです。
そのためmodel.removeRow(i);が範囲外となります。

tableModel.removeRow(i);としたいところですが、AbstractTableModelにはremoveRowがありませんし自前実装も面倒です。
DefaultTableModelから派生したほうが楽でしょう（既にTableModelインターフェースがあるので、TableModelクラスでなくMyTableModel等がいいです）
TableModel (Java Platform SE 8 )

参考にしたサイト
http://asistobe851.web.fc2.com/my-memo/RowManagement.html

複数行選択や並び替えに未対応です。

JTableに表示されている表から行を選択して削除したい
テーブルに表示されている行を選択して、ボタンを押下することで選択行が削除される機能

JTableの選択した行でしょうか？
選択カラムにチェックが入った行でしょうか？

チェック行ならjimbeさんの回答通り、逆順に回せばいいですね。

選択行の場合は、並び替えを考慮する必要があります。
How do you remove selected rows from a JTable? - Stack Overflow

はっきり言ってかなり混乱が見られますが、こちらの手元（Java17）では当該質問コードでおかしい状況を再現できませんでした（回答コードも本質的には同じはず）
もしかしたら何かバグがあって、どこかの時点で改善されたのかもしれません？？

import javax.swing.*;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.TableRowSorter;
import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;


public class ListPractice extends JFrame implements ActionListener {
    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new ListPractice().setVisible(true));
    }

    final JTable table;
    final DefaultTableModel model;
    final TableRowSorter<DefaultTableModel> sorter;
    final JTextField filterText = new JTextField();

    ListPractice() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(400, 300);
        setLocationRelativeTo(null);


        Object[][] data = {
            { "田中", "大阪", true, },
            { "長谷川", "東京", false, },
            { "藤原", "奈良", false, },
            { "橋本", "和歌山", false, },
            { "山田", "北海道", false, },
        };
        String[] columns = { "名前", "出身", "選択", };

        // 匿名（無名）クラス（別にMyTableModelクラスを作ってもいいのだが^^;
        model = new DefaultTableModel(data, columns) {
            @Override public Class<?> getColumnClass(int columnIndex) {
                return getValueAt(0, columnIndex).getClass();
            }

            @Override public boolean isCellEditable(int rowIndex, int columnIndex) {
                return columnIndex == 2;
            }
        };

        sorter = new TableRowSorter<>(model);

        table = new JTable(model);
        table.setRowSorter(sorter);

        var filterPanel = Box.createHorizontalBox();

        var filterLabel = new JLabel("検索");
        filterLabel.setLabelFor(filterText);
        filterPanel.add(filterLabel);

        filterText.addActionListener(e -> {
            var filter = RowFilter.regexFilter(filterText.getText(), 0);
            sorter.setRowFilter(filter);
        });
        filterPanel.add(filterText);

        add(filterPanel, BorderLayout.NORTH);


        var mainPanel = Box.createVerticalBox();

        var deleteButton = new JButton("削除");
        deleteButton.setAlignmentX(RIGHT_ALIGNMENT);
        deleteButton.setActionCommand("DELETE");
        deleteButton.addActionListener(this);
        mainPanel.add(deleteButton);

        mainPanel.add(new JScrollPane(table));

        add(mainPanel, BorderLayout.CENTER);
    }

    @Override public void actionPerformed(ActionEvent e) {
        var command = e.getActionCommand();
        if ("DELETE".equals(command)) {

// これではおかしい
//            ListSelectionModel lsModel = table.getSelectionModel();
//            for (int i = 0; i < table.getRowCount(); i++) {
//                if (lsModel.isSelectedIndex(i)) {
//                    model.removeRow(i);
//                }
//            }

            // こちらの手元ではコレで動いているが...
            while (table.getSelectedRow() != -1) {
                model.removeRow(table.convertRowIndexToModel(table.getSelectedRow()));
            }
        }
    }
}

Answer 2

テーブルの行数と選択行がうまく「removeRow()」に送られていないのでしょうか

そもそも削除対象のモデルが違います。
ご自身で作成された TableModel から削除しなければならないのに、中身のない DefaultTableModel から作った model から削除しようとしています。

また、"選択"列と SelectionModel は関係ありません。
選択列の値によって処理を行うのであれば、そのようなコードを書く必要があります。

クラス名を変更、モデルを内部に取り込み、パネルをクラス化、ActionListener は無名クラスにしています。
モデルのデータを配列から(削除し易いように)リスト化、行削除メソッド(removeRow)を作成、 ControllPanel の削除ボタンを押すとリストの最後から順に選択状態を調べてモデルの行削除メソッドを呼びます。

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.*;

import javax.swing.*;
import javax.swing.table.AbstractTableModel;
import javax.swing.table.TableRowSorter;

public class TableFrame extends JFrame {
  public static void main(String[] args) {
    SwingUtilities.invokeLater(new Runnable() {
      @Override
      public void run() {
        new TableFrame().setVisible(true);
      }
    });
  }

  TableFrame() {
    super("TableFrame");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setSize(400, 300);
    setLocationRelativeTo(null);

    LocalTableModel model = new LocalTableModel();

    JTable table = new JTable(model);
    add(new JScrollPane(table), BorderLayout.CENTER);

    TableRowSorter<?> sorter = new TableRowSorter<LocalTableModel>(model);
    table.setRowSorter(sorter);
    FilterPanel filterPanel = new FilterPanel(sorter);

    ControllPanel controllPanel = new ControllPanel(model);

    JPanel northPanel = new JPanel(null);
    northPanel.setLayout(new BoxLayout(northPanel, BoxLayout.Y_AXIS));
    northPanel.add(filterPanel);
    northPanel.add(controllPanel);
    add(northPanel, BorderLayout.NORTH);
  }

  /** モデル */
  private static class LocalTableModel extends AbstractTableModel {
    private static final String[] columns = {"名前", "出身", "選択"};

    private static class Data {
      String name, birthplace;
      boolean select;

      Data(String name, String birthplace, boolean select) {
        this.name = name;
        this.birthplace = birthplace;
        this.select = select;
      }
    }

    private List<Data> dataList = new ArrayList<>(Arrays.asList(
        new Data("田中", "大阪", true),
        new Data("長谷川", "東京", false),
        new Data("藤原", "奈良", false),
        new Data("橋本", "和歌山", false),
        new Data("山田", "北海道", false)
    ));

    @Override
    public Class<?> getColumnClass(int columnIndex) {
      return columnIndex == 2 ? Boolean.class : String.class;
    }

    @Override
    public int getColumnCount() {
      return columns.length;
    }

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

    @Override
    public String getColumnName(int column) {
      return columns[column];
    }

    @Override
    public Object getValueAt(int rowIndex, int columnIndex) {
      Data data = dataList.get(rowIndex);
      switch(columnIndex) {
      case 0: return data.name;
      case 1: return data.birthplace;
      case 2: return data.select;
      }
      return null;
    }

    @Override
    public boolean isCellEditable(int rowIndex, int columnIndex) {
      return columnIndex == 2; // 編集可能なセルの設定
    }

    @Override
    public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
      if(columnIndex != 2) return;
      dataList.get(rowIndex).select = (boolean)aValue;
      fireTableCellUpdated(rowIndex, columnIndex);
    }

    //削除
    void removeRow(int rowIndex) {
      dataList.remove(rowIndex);
      fireTableRowsDeleted(rowIndex, rowIndex);
    }
  }

  /** フィルターパネル */
  private static class FilterPanel extends JPanel {
    FilterPanel(TableRowSorter<?> sorter) {
      super(null);
      setLayout(new BoxLayout(this, BoxLayout.X_AXIS));

      JLabel filterLabel = new JLabel("検索");
      add(filterLabel);

      JTextField filterText = new JTextField();
      filterText.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
          sorter.setRowFilter(RowFilter.regexFilter(filterText.getText(), 0));
        }
      });
      add(filterText);

      filterLabel.setLabelFor(filterText);
    }
  }

  /** コントロールパネル */
  private static class ControllPanel extends JPanel {
    ControllPanel(LocalTableModel model) {
      super(null);
      setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));

      //削除ボタン
      JButton deleteButton = new JButton("削除");
      deleteButton.setAlignmentX(RIGHT_ALIGNMENT);
      deleteButton.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
          for(int i=model.getRowCount()-1; i>=0; i--) { //先頭からやると削除時に後続のインデックスがズレてくるので、最期からやる
            if((boolean)model.getValueAt(i, 2)) { //選択
              model.removeRow(i);
            }
          }
        }
      });
      add(deleteButton);
    }
  }
}