更新データがない場合にエラーボックスを表示するようにしたい

Question

以下のコードを実行して、更新データがない場合にエラーをはくようにしたいです。

public class Validate {

private static final int ERROR_CODE_1030 = 1030;

private static final String ERROR_MESSAGE_1030 = "更新対象のデータが存在しません。";

//更新データ存在チック
		public static int chkExistsUpdData(Connection con, String str) {
			PreparedStatement pstmt = null;
			ResultSet rs = null;
			try {
				// SQL文作成
				String mySql = "select count(*) as naiyou from tbl.tbl_dbaccess where PID = ?;";
				// ステートメントオブジェクトを作成
				pstmt = con.prepareStatement(mySql);
				// 検索条件
				pstmt.setInt(1, Integer.parseInt(str));
				// 検索するSQL実行
				rs = pstmt.executeQuery();

				int count = 0; // 検索結果（件数）
				// 結果セットからデータを取り出す next()で次の行に移動
				while(rs.next()) {
					count = rs.getInt("naiyou");
				}

				// 検索結果件数チェック
				if(count == 0) {
					return ERROR_CODE_1030;
				}

			} catch (Exception ex) {
				System.out.println("例外発生：" + ex);
				ex.printStackTrace();
			} finally {
				try {
					// 実行結果をクローズ
					if(rs != null) {
						rs.close();
					}
					// ステートメントをクローズ
					if(pstmt != null) {
						pstmt.close();
					}
				} catch (SQLException se) {
					// 何もしない
				}
			}
			return IS_SUCCESS_CODE;
		}

//更新
	private class SwingAction_1 extends AbstractAction {
		public SwingAction_1() {
			putValue(NAME, "更新");
			putValue(SHORT_DESCRIPTION, "Some short description");
		}
		public void actionPerformed(ActionEvent e) {

			int row = table.getSelectedRow();
            int col = table.getSelectedColumn();
            Connection con = null;
            Statement stmt = null;
            ResultSet rs = null;
            try {

            	// データベースに接続
            	Class.forName("com.mysql.cj.jdbc.Driver");
                con=DriverManager.getConnection(STR_CONN);

				//入力文字内容チェック
				int chErrorCode = Validate.chkContents(textField.getText());
				if(chErrorCode != Validate.getErrCode0()){
				//エラーダイアログ表示
				JOptionPane.showMessageDialog(frame, Validate.getErrMsg(chErrorCode));
				}

	            // 更新データ存在チェック
				int CerrorCode = Validate.chkExistsUpdData(con, String.valueOf(table.getValueAt(row, 1)));
				if(CerrorCode != Validate.getErrCode0()) {
				//エラーダイアログ表示（更新対象のデータが存在しません）
				JOptionPane.showMessageDialog(frame, Validate.getErrMsg(CerrorCode));
					}

                // ステートメントオブジェクトを作成
                stmt=con.createStatement();
				// SQL文作成
	String Sql = "UPDATE tbl_dbaccess SET NAIYOU='"+table.getValueAt(0,col)+"'WHERE PID='"+table.getValueAt(row, 0)+"'";
				String sql2="select PID, NAIYOU as 内容 from tbl_dbaccess";
				stmt.executeUpdate(Sql);
                rs=stmt.executeQuery(sql2);
                //System.out.println(Sql);
                //問い合わせ
                table.setModel(new MyTableModel(rs));
                      }
                      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());
                  }
                }
            }
	}

現状以下のエラーが表示され先に進めていない状態です。
java.lang.ArrayIndexOutOfBoundsException: 1 >= 0
ご教示頂けますと幸いです。

Comments

[jimbe]

> java.lang.ArrayIndexOutOfBoundsException: 1 >= 0
これが発生している行をスタックトレースから教えてください.

Answer 1

まず, 「更新データがない」ことがあるのでしょうか.
前のご質問からすると, テーブルにはDBからのデータが並んでいるはずで, それを編集して更新しているはずです.

そして, Statement#executeUpdate() は処理した件数を返します.
キーである(と思われる) PID を条件にした UPDATE であれば, 成功すれば 1, 失敗すれば 0 となるのではないでしょうか.
ですので, UPDATE 前に SELECT で確認せずとも, UPDATE の実行結果からエラーを判定すれば良いのではと思います.

以下は(MySql 環境が無いので) DB を模して動作するモノです.
更新はボタン操作でなくテーブルの「内容」の編集後に自動で行い, 再表示も自動ですので表示ボタンもありません. (PID は編集不可としています.)
登録ボタンは入力内容に文字列がある場合のみ有効, 削除ボタンは行を選択している場合のみ有効とする等で, エラーになる状況を減らしています.
(MySql にアクセスするクラスも作ってはあるのですが, 動かしていないことと文字数制限に掛かるので入れていません.)

package teratail.q250457;
//package DBaccess;

import java.awt.*;
import java.awt.event.ActionEvent;
import java.sql.SQLException;
import java.util.*;
import java.util.List;

import javax.swing.*;
import javax.swing.border.EmptyBorder;
import javax.swing.event.*;
import javax.swing.table.AbstractTableModel;

public class DBAccess extends JFrame implements DatabaseExceptionProcessor {

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

  private DatabaseAccessor dba;
  private JTable table;

  public DBAccess() throws ClassNotFoundException, SQLException {
    super("DBアクセスプログラム");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(100, 100, 450, 300);
    setLocationRelativeTo(null);

    //dba = new MysqlDatabaseAccessor(this);
    dba = new DummyDatabaseAccessor();
    table = new JTable(dba.selectAll());

    JTextField textField = new JTextField();
    textField.setColumns(10);
    textField.setAlignmentX(JLabel.LEFT_ALIGNMENT);

    InsertAction insertAction = new InsertAction(textField);
    DeleteAction deleteAction = new DeleteAction();

    JPanel contentPane = new JPanel();
    contentPane.setLayout(new BorderLayout());
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    setContentPane(contentPane);

    Box northPanel = Box.createVerticalBox(); {
      JPanel buttonsPanel = new JPanel(new FlowLayout());
      buttonsPanel.setAlignmentX(JLabel.LEFT_ALIGNMENT); {
        buttonsPanel.add(new JButton(insertAction));
        buttonsPanel.add(new JButton(deleteAction));
      }
      northPanel.add(buttonsPanel);
      northPanel.add(createLabel("入力内容"));
      northPanel.add(textField);
      northPanel.add(createLabel("表示内容"));
    }
    contentPane.add(northPanel, BorderLayout.NORTH);
    contentPane.add(new JScrollPane(table), BorderLayout.CENTER);

    table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
    table.getSelectionModel().addListSelectionListener(new ListSelectionListener() {
      @Override
      public void valueChanged(ListSelectionEvent e) {
        int pid = -1;
        int index = table.getSelectedRow();
        if(index >= 0) {
          index = table.convertRowIndexToModel(index);
          pid = ((MyTableModel)table.getModel()).getPid(index);
        }
        deleteAction.setTarget(pid);
      }
    });
    table.clearSelection();
  }
  private JLabel createLabel(String text) {
    JLabel label = new JLabel(text);
    label.setAlignmentX(JLabel.LEFT_ALIGNMENT);
    return label;
  }

  private class InsertAction extends AbstractAction {
    private JTextField textField;
    public InsertAction(JTextField textField) {
      putValue(NAME, "登録");
      setEnabled(false);
      this.textField = textField;
      textField.getDocument().addDocumentListener(new DocumentListener() {
        @Override
        public void removeUpdate(DocumentEvent e) { checkEnable(); }
        @Override
        public void insertUpdate(DocumentEvent e) { checkEnable(); }
        @Override
        public void changedUpdate(DocumentEvent e) { checkEnable(); }
        void checkEnable() {
          InsertAction.this.setEnabled(textField.getText().length() > 0);
        }
      });
    }
    @Override
    public void actionPerformed(ActionEvent ae) {
      String text = textField.getText();
      if(dba.insert(text) == 1) {
        table.setModel(dba.selectAll());
        textField.setText(""); //クリア
      }
    }
  }

  private class DeleteAction extends AbstractAction {
    private int pid;
    DeleteAction() {
      putValue(NAME, "削除");
      setEnabled(false);
    }
    void setTarget(int pid) {
      this.pid = pid;
      setEnabled(pid >= 0);
    }
    @Override
    public void actionPerformed(ActionEvent ae) {
      if(dba.delete(pid) == 1) {
        table.setModel(dba.selectAll());
      }
    }
  }

  @Override
  public void receiveException(SQLException e) {
    JOptionPane.showMessageDialog(this, e.getMessage());
    e.printStackTrace();
  }
}

class MyTableModel extends AbstractTableModel {
  private String[] columnNames = new String[] { "PID", "内容"};
  private static class Row {
    final Integer pid;
    final String naiyou;
    Row(int pid, String naiyou) {
      this.pid = pid;
      this.naiyou = naiyou;
    }
  }
  private List<Row> rows = new ArrayList<>();
  private DatabaseAccessor dba;

  MyTableModel(DatabaseAccessor dba) {
    this.dba = dba;
  }

  void addRow(int pid, String naiyou) {
    rows.add(new Row(pid, naiyou));
    fireTableRowsInserted(rows.size()-1, rows.size()-1);
  }
  int getPid(int rowIndex) {
    return rows.get(rowIndex).pid;
  }

  @Override
  public String getColumnName(int columnIndex) {
    return columnNames[columnIndex];
  }
  @Override
  public int getRowCount() {
    return rows.size();
  }
  @Override
  public int getColumnCount() {
    return columnNames.length;
  }
  @Override
  public Object getValueAt(int rowIndex, int columnIndex) {
    if(columnIndex == 0) return rows.get(rowIndex).pid;
    if(columnIndex == 1) return rows.get(rowIndex).naiyou;
    return null;
  }
  @Override
  public boolean isCellEditable(int rowIndex, int columnIndex) {
    return columnIndex == 1;
  }
  @Override
  public void setValueAt(Object value, int rowIndex, int columnIndex) {
    rows.set(rowIndex, new Row(rows.get(rowIndex).pid, (String)value));
    fireTableCellUpdated(rowIndex, columnIndex);
    if(dba != null) dba.update(rows.get(rowIndex).pid, (String)value);
  }
}

interface DatabaseExceptionProcessor {
  void receiveException(SQLException e);
}
interface DatabaseAccessor {
  MyTableModel selectAll();
  int insert(String text);
  int update(int pid, String text);
  int delete(int pid);
}
class DummyDatabaseAccessor implements DatabaseAccessor {
  private static String TEST[] = {"ABC","あいう","１２３"};
  private int nextPid = 0;
  private Map<Integer,String> rows = new TreeMap<Integer, String>();
  DummyDatabaseAccessor() {
    for(String s : TEST) rows.put(nextPid++, s);
  }
  @Override
  public MyTableModel selectAll() {
    MyTableModel tableModel = new MyTableModel(this);
    for(Map.Entry<Integer,String> entry : rows.entrySet()) {
      tableModel.addRow(entry.getKey(), entry.getValue());
    }
    return tableModel;
  }
  @Override
  public int insert(String text) {
    System.out.println("insert text="+text);
    rows.put(nextPid++, text);
    return 1;
  }
  @Override
  public int update(int pid, String text) {
    System.out.println("update pid="+pid+", text="+text);
    if(!rows.containsKey(pid)) return 0;
    rows.put(pid, text);
    return 1;
  }
  @Override
  public int delete(int pid) {
    System.out.println("delete pid="+pid);
    if(!rows.containsKey(pid)) return 0;
    rows.remove(pid);
    return 1;
  }
}

Answer 2

表題更新データがない場合にエラーボックスを表示するようにしたいと、訊きたいこと現状以下のエラーが表示され先に進めていない状態ですに齟齬がありすぎて、まるで理解できません。なので、表題に関することだけ。

エラーボックスとは、ダイアログボックスのことと想像しました。ダイアログボックスを表示するには、JOptionPaneが使えます。そのサンプルです。

package xx;

import javax.swing.*;
import java.awt.*;
import java.io.ByteArrayOutputStream;
import java.io.PrintStream;

public class Hogege {
  public static final void main(final String[] args) throws Exception {
    SwingUtilities.invokeLater(() -> {
      final MainWindow frame = new MainWindow();
      frame.setVisible(true);
    });
  }

  static class MainWindow extends JFrame {
    MainWindow() {
      super();
      setLayout(new FlowLayout());

      final Button doExceptionButton = new Button("do exception");
      add(doExceptionButton);
      doExceptionButton.addActionListener(ev -> {
        try {
          doSometing();
        } catch (Exception e) {
          JOptionPane.showMessageDialog(
              doExceptionButton,
              dumpException(e),
              e.getMessage(),
              JOptionPane.ERROR_MESSAGE);
        }
      });

      setTitle(this.getClass().getName());
      setSize(400, 400);
    }

    void doSometing() throws Exception {
      throw new Exception("うだうだ、ぐだぐだ、うだうだ、ぐだぐだ");
    }

    private String dumpException(Exception e) {
      final ByteArrayOutputStream baos = new ByteArrayOutputStream();
      final PrintStream ps = new PrintStream(baos);
      e.printStackTrace(ps);
      ps.close();
      return baos.toString();
    }
  }
}

Comments

[jimbe]

データが無い場合にその旨を(エラーとして)表示したいが, コードに問題があって例外(エラー?)が出てしまうということではないでしょうか.

[shiketa]

きっとそうなのだろうともおもいましたが、あえて表題側を摂りました。
ArrayIndexOutOfBoundsExceptionが発生しているのだから、見えている範囲では`getValueAt()`のどれかがおかしいのだろうと思えますが、実は見せてもらえていない部分かもしれませんし。

kakei2794さん。
実行はできているようなので、例外をキャッチする場所すべてでエラーボックスを表示すれば、StackTraceも表示されるでしょうから、具体的にどこでArrayIndexOutOfBoundsExceptionが発生しているのかヒントがつかめるかもしれませんよ。