JDBC UPDATE 入力された値のみ更新

Question

System.out.print("更新するIDを入力:");
String empId = br.readLine();

System.out.print("A1:");
String empName = br.readLine();
System.out.print("A2:");
String gender = br.readLine();
System.out.print("A3):");
String birthday = br.readLine();
System.out.print("A4:");
String deptId = br.readLine();

con = db.getConnection();

String sqlup = "UPDATE **** SET A1 = ?,A2 = ?,A3 = ?,A4 = ? WHERE ID = ?";
ps = con.prepareStatement(sqlup);

ps.setString(5, ID);
ps.setString(1, A1);
ps.setString(2, A2);
ps.setString(3, A3);
ps.setString(4, A4);

ps.executeUpdate();
System.out.println("更新しました");
	}
else{
System.out.println("該当する項目が存在しません");

はじめまして。
私、現在JavaおよびoracleDBを用いて簡単なコンソールアプリケーションを作成しているのですが、
JavaからのSQLで

String sql = "UPDATE **** SET A1=?, A2=?, A3=?...WHERE **** =?"

といったソースコードを記述したのですが、これに対して『コンソール上から入力した値のみを更新する』といった処理にするにはどうしたらよいでしょうか？
コンソールからの入力がなかった項目、入力を飛ばした項目に関しては元の値をそのままに、その他入力された項目がある場合は当該の項目を更新する、といった状態にしたいです。
御回答のほどよろしくお願いいたします。

Comments

[m.ts10806]

「コンソール上から入力された値を受け取る｣部分はできているのでしょうか？

[m.ts10806]

プログラムコード(およびエラーメッセージ)は質問内容としては最も重要な部分であるため、見やすくしていただけると助かります。<code>ボタン押下→「コード」部分にコードを貼り付け→「ここに言語を入力」に対象言語名記入（エラーメッセージの場合は不要）の手順で「コードハイライト化」してください。（質問編集画面ではリアルタイムでプレビューが表示されるので見ながら調整してください）

[m.ts10806]

提示されたコードだと更新対象のカラムの情報も受け取ってsetされていますし、そのまま出来そうに思うのですが、何がうまくいかないのでしょうか？

[969]

コンソールからの入力でnullがあった場合は元の値を参照させたいと思うのですがうまくいきません。

[m.ts10806]

元の値とは？要件も質問にあるものと違いますので、質問を編集して要件を具体的に書いて下さい。

[m.ts10806]

細かいですが　System.out.print("A4:);　　←" 閉じられてません・・

[969]

ご指摘ありがとうございます。　先ほど編集したときに間違って消してしまったようです。

[m.ts10806]

後半のコードもだいぶおかしくなっているように思います。コード・要件　ともに整理して調整してください。でないと中々的確な回答が得られないと思います。

[969]

バインドに関しては入力した順に記述しております。まだ初心者でして、不明なところ、至らないところございますがご容赦くださいませ。

[m.ts10806]

回答してみました。動作未検証ですので、実装イメージだけ持っていただければと思います。

Answer 1

配列に更新対象のカラムと値をputしていってカラム配列のforでset部を生成、
値配列のforでパラメータセット、
最後にIDを入れる。

※未検証です。もしずれてたりnullや空入力がとれていない場合は調整してください。

        int cnt = 0;
        Map<Integer,String> columns = new HashMap<>();
        Map<Integer,String> values = new HashMap<>();

        Boolean flg = false;
    	String empId ="";

    	System.out.print("更新するIDを入力:");
    	empId = br.readLine();

    	System.out.print("A1:");
    	String empName = br.readLine();
    	if(empName != null) {
    		columns.put(cnt, "A1=?");
    		values.put(cnt, empName);
    		cnt++;
    	}
    	System.out.print("A2:");
    	String gender = br.readLine();
    	if(gender != null) {
    		columns.put(cnt, "A2=?");
    		values.put(cnt, gender);
    		cnt++;
    	}
    	System.out.print("A3:");
    	String birthday = br.readLine();
    	if(birthday != null) {
    		columns.put(cnt, "A3=?");
    		values.put(cnt, birthday);
    		cnt++;
    	}
    	System.out.print("A4:");
    	String deptId = br.readLine();
    	if(deptId != null) {
    		columns.put(cnt, "A4=?");
    		values.put(cnt, deptId);
    		cnt++;
    	}
    	if(cnt > 0) {
        	String sqlup = "UPDATE **** SET ";
            for(int i = 0;i<columns.size();i++) {
            	if(i > 0) {
            		sqlup += ",";
            	}
            	sqlup += columns.get(i);
            }

            sqlup += " WHERE ID = ?";
        	
            ps = con.prepareStatement(sqlup);
            for(int i = 0;i<values.size();i++) {
            	ps.setString((i+1), values.get(i));
            }
            ps.setString((cnt+1), empId); //最後にID
            ps.executeUpdate();
    	}

追記。
この場合、IDは入力必須としておかないと全レコード更新してしまうので、
入力されるまでwhileなどでまわすような工夫はしておいた方が良いかと思います。

Comments

[退会済みユーザー]

br.readLine() が null を返したときは EOFを検出したときなので、続けて br.readLine() を呼ぶと例外が発生すると思います。

[m.ts10806]

てことはnullじゃなく空文字判定になりますかね。
注釈に書いた通り未検証の実装イメージ提示なのでその辺りは質問者さんに調整してもらってそれも糧にしてもらいたいですね。

Answer 2

入力された値のみというのを、変化があった項目のみとするなら、変更前後の値をチェックした上で、setに記述する項目を動的に変更する必要があります。

一般的には、無駄な更新を避けるため、値に変化があったかどうかはチェックしますが、更新自体は動的SQLとはせず、画面で更新対象となっている項目をsetに列挙して、画面上の値で更新するようにします。

Answer 3

といったソースコードを記述したのですが、これに対して『コンソール上から入力した値のみを更新する』といった処理にするには

↑コンソールから入力した値に応じてSQLを組み立てるようにすればいいのでは？

Answer 4

コンソール上から入力した値のみ

をUPDATE ... SET の並びに入れて WHERE ...にします。
通常は変更される可能性のある列を前部画面に表示しておいて、変更後[更新]ボタンで1つのUPDATEでシンプルに対応します。