JTable内のデータの編集及び更新を可能にしたい

Question

実現したいこと

以下のコードを完成させ、DBアクセスプログラムを構築させたいです。

package DBaccess;

import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.sql.Connection;
…省略
import javax.swing.table.DefaultTableModel;

public class Swing extends JFrame implements TableModelListener {

    private JTextField textField;
    private JFrame frame;
    private JPanel contentPane;
    private DefaultTableModel tableModel;
    private JTable table;


	private final Action action = new SwingAction();
	…省略

	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					Swing frame = new Swing();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public Swing() {
		setTitle("DBアクセスプログラム");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 300);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		contentPane.setLayout(null);

		JButton btnNewButton = new JButton("New button");
		btnNewButton.setAction(action);
		btnNewButton.setBounds(27, 10, 91, 40);
		contentPane.add(btnNewButton);

		JButton btnNewButton_1 = new JButton("New button");
		btnNewButton_1.setAction(action_1);
		btnNewButton_1.setBounds(123, 10, 91, 40);
		contentPane.add(btnNewButton_1);

		JButton btnNewButton_2 = new JButton("New button");
		btnNewButton_2.setAction(action_2);
		btnNewButton_2.setBounds(219, 10, 91, 40);
		contentPane.add(btnNewButton_2);

		JButton btnNewButton_3 = new JButton("New button");
		btnNewButton_3.setAction(action_3);
		btnNewButton_3.setBounds(315, 10, 91, 40);
		contentPane.add(btnNewButton_3);

		JLabel lblNewLabel = new JLabel("入力内容");
		lblNewLabel.setBounds(29, 64, 50, 13);
		contentPane.add(lblNewLabel);

		textField = new JTextField();
		textField.setBounds(27, 87, 379, 19);
		contentPane.add(textField);
		textField.setColumns(10);

		tableModel = new DefaultTableModel();
        table=new JTable(tableModel);
        tableModel.addTableModelListener(this);
        contentPane.add(table);

		JLabel lblNewLabel_1 = new JLabel("表示内容");
		lblNewLabel_1.setBounds(27, 116, 50, 13);
		contentPane.add(lblNewLabel_1);

		JScrollPane scrollPane = new JScrollPane(table);
		scrollPane.setBounds(27, 135, 379, 116);
		contentPane.add(scrollPane);
	}
	public void tableChanged(TableModelEvent e) {
  	  int row = e.getLastRow(); 
  	  int col = e.getColumn(); 
  	  Object dl =table.getValueAt(row,col);

  	  tableModel.setValueAt(dl, row, col);
  	  String message
  	  = "行" + row + "列" + col + "が変更されました。";
  	  JOptionPane.showMessageDialog(Swing.this, message);
    }


	
	private class SwingAction extends AbstractAction {
		public SwingAction() {
			putValue(NAME, "登録");
			putValue(SHORT_DESCRIPTION, "Some short description");
		}
		public void actionPerformed(ActionEvent e) {
			String text = textField.getText();
            if( text == null ) {
            System.out.println("nullである");
            } else {
            System.out.println("nullではない");
            }
            System.out.println("[" + text + "]");
            Connection con = null;
            Statement stmt = null;
            try {

            	
            	Class.forName("com.mysql.cj.jdbc.Driver");
                con=DriverManager.getConnection(STR_CONN);
              
                stmt=con.createStatement();
               
                String sql= "insert into tbl_dbaccess(NAIYOU)values('"+text+"')";

               
                stmt.executeUpdate(sql);
              
                      }
                      catch(Exception r){r.printStackTrace();}
            finally {
                
                if(stmt != null) {
                    try {
                        stmt.close();
                    }catch (Exception r) {
                        System.out.println(r.getMessage());
                  }
                }
                if(con != null)
                    try {
                        con.close();
                    }catch (Exception r) {
                        System.out.println(r.getMessage());
                  }
                }
            }

}
	
・・削除更新項目省略・・
	
	private class SwingAction_3 extends AbstractAction {
		public SwingAction_3() {
			putValue(NAME, "表示");
			putValue(SHORT_DESCRIPTION, "Some short description");
		}
		public void actionPerformed(ActionEvent e) {

			Connection con = null;
			Statement stmt = null;
			ResultSet rs = null;
			try {
							
							Class.forName("com.mysql.cj.jdbc.Driver");
							con=DriverManager.getConnection(STR_CONN);

					

							
							stmt=con.createStatement();

							
							String sql="select PID, NAIYOU as 内容 from tbl_dbaccess";

							
							rs=stmt.executeQuery(sql);

							
							table.setModel(new MyTableModel(rs));
			    }
						
							catch(Exception r){
									r.printStackTrace();}
							finally {

								
								if(rs != null) {
									try {
										rs.close();
										}catch (Exception r) {
											System.out.println(r.getMessage());
										}
												}

								if(stmt != null) {
									try {
										stmt.close();
									}catch (Exception r) {
										System.out.println(r.getMessage());
									}
												  }

								if(con != null) {
									try {
										con.close();
									}catch (Exception r) {
										System.out.println(r.getMessage());
										
									}
								}
							}



				}
}

	
	class MyTableModel extends AbstractTableModel
	{
		private ArrayList<String> columnname;
		private ArrayList<ArrayList> data;

		public boolean isCellEditable(int rowIndex, int columnIndex) {
			return true;
		}

		public MyTableModel(ResultSet rs)
		{
			try {
					
						ResultSetMetaData rm=rs.getMetaData();
						int cnum = rm.getColumnCount();
						columnname = new ArrayList<String>(cnum);

						
						for(int i=1; i<=cnum; i++) 
{columnname.add(rm.getColumnLabel(i));
						}
						
						data = new ArrayList();
						while(rs.next()) {							ArrayList<String>rowdata = new ArrayList<String>();								for(int i=1; i<=cnum; i++) {								rowdata.add(rs.getObject(i).toString());								}							data.add(rowdata);
						}
			}
			catch(Exception e) {
				e.printStackTrace();
			}
		}
		public void setValueAt(Object val, int rowIndex, int columnIndex) {
			fireTableCellUpdated(rowIndex, columnIndex);}
		public String getColumnName(int column) {
			return (String)columnname.get(column);
		}
		public int getRowCount() {
			return data.size();
		}
		public int getColumnCount() {
			return columnname.size();
		}
		public Object getValueAt(int row, int column) {
			ArrayList rowdata = (ArrayList)data.get(row);
			return rowdata.get(column);
		}
	}
	}

試したこと

Javaに関して、ほとんど知識がないため
模索中ですが、あまり有効な情報が得られていない状態です。

補足情報

TextFieldに文字を入力した後「登録」を押すことでSQLにデータが登録される。
そのデータは「表示」を押すことでJTableとして、表示がされる。
表示されたデータに関しては項目をダブルクリックすると、入力文字を修正し、Enter押下後に編集ができるようにしたいのだが、Enterを押しても元の文字に戻ってしまう。

どのようにしたら、解消できますか？

表示ボタン押下後

編集

Enter押下後

Comments

[m.ts10806]

タイトルと内容(起きている問題)が合致してないと思います。
DB接続およびデータの取得ができてる前提ならこのタイトルはおかしいです。
解決したい問題に寄せたタイトルにしてください。

[退会済みユーザー]

失礼いたしました。
編集いたします。

[shiketa]

```
public void setValueAt(Object val, int rowIndex, int columnIndex) {
fireTableCellUpdated(rowIndex, columnIndex);}
```
の部分のよーな気がするんだけどねぇ。「…省略」が多くて、コンパイルできない。だから試しに動かしてみることもできない。
問題は「JTable内のデータの編集及び更新を可能にしたい」なのだから、DBアクセスをおこなっているあたりを一層して、問題のJTableの更新部分だけを切り出してもっと単純な形にしてみてはどうでしょうか。ざっくりと鑑賞した感想ですが、Swingで画面を扱う部分と、DBにアクセスする部分が複雑に絡み合い過ぎていると感じます。問題を単純な形にする過程で実装の整理もできると思います。また、SwingActionやSwingAction_3といったクラス名ももっと適切な名前があるはずですので、名前の見直し(リファクタリング)も併せて実施してみてください。
いずれ、肝心の部分が提示されていないと思われるので、本人以外にはこれ以上は手を出せそうにありませんよ。

[shiketa]

コメント訂正します。fireTableCellUpdated()は、AbstractTableModelクラスのメソッドでした。失礼しました。https://docs.oracle.com/javase/jp/8/docs/api/javax/swing/table/AbstractTableModel.html#fireTableCellUpdated-int-int-

Answer 1

コメントを書いてみて気づきました。実行はできないので、おそらく、ですが。

    class MyTableModel extends AbstractTableModel
    {
...
        public void setValueAt(Object val, int rowIndex, int columnIndex) {
            fireTableCellUpdated(rowIndex, columnIndex);}

fireTableCellUpdatedは、イベントを発火させるだけです。dataを更新していませんよね。だから、fireTableCellUpdated()のあとで、getValueAt()が呼び出されて、もとの値に戻る。

        public void setValueAt(Object val, int rowIndex, int columnIndex) {
　　　　　　　 //こんな感じで、dataを更新する必要がある。気がする。
    　　　　　data.get(rowIndex).set(columnIndex, val);
            fireTableCellUpdated(rowIndex, columnIndex);}

実際、AbstractTableModelを拡張しても手間なだけでしょうね。もっと手軽にするのであれば、
DefaultTableModelを拡張して、余計な部分を取り去って、だいたいのことをDefaultTableModelに任せれはいいはずです。

  class MyTableModel extends DefaultTableModel {...

現状の実装で、DefaultTableModelを使えないのは、DBへのアクセス処理をMyTableModelのコンストラクタでおこなっているからです。コメントで書いたように画面の処理とDBアクセス処理を分離することで、このあたりはもっとすっきり書けるようになるはずです。

Comments

[退会済みユーザー]

ありがとうございます！そして、お返事が遅くなり、申し訳ありません。
上記を参考に試してみたところ、文字の編集は可能になりましたが、新しいデータを「登録」後に「更新」を押下した際に、過去のものに戻ってしまうという現象が起きてしまいました、、
こちらもご教示頂くことは可能でしょうか？

[shiketa]

それは、ムリ、です。なぜなら、ボタン押下時のaction,action_1,action_2,action_3がいったいナンなのかいったいナニをしているのかが提示されていないので、それを知っているのは、あなただけだからです。

[退会済みユーザー]

なるほどです。。
色々ありがとうございました！
とても助かりました！