Listで空の要素のみ更新したい

Question

javaでSQLの結果をListに詰めるのですが
複数SQLをUNIONにて繋ぎ、取得できず空またはnullとなっている箇所に対して
補わせるSQLを流すという処理を行いたいと思っています。

作ってみたソースが以下のようになります。

for (aaaEntity m : resultList1) {
// List更新用のListを作成。
String bId = null;
bId = m.getBId();
if (bId == null) {
List<aaaEntity> resultList3 = this.selectBId(resultList1);
if (resultList3 != null && resultList3.size() > 0) {
for (int i = 0; i < resultList3.size(); i++) {
for (aaaEntity x : resultList3) {
resultList1.set(i, x);
}
}
}
}
}

これだと、全上書きになってしまいました。
nullまたは空となっている要素のみ更新したいときはどのように描くべきなのでしょうか？

※resultList1でUNIONを使用し全リストを取得してきています。
※resultList3がresultList1で取得できなかったbIdの要素を取得するためのSQLの結果を格納しているListになります。

Answer 1

resultList1 のレコードと resultList3 のレコードを紐付けるキーは存在しているのでしょうか？
紐づくキー情報が無ければ組み合わせることは不可能ですので実現できません。

キーが有ることを前提にするならば以下の様なコードは参考になるでしょう。

{
    // 補完情報を列挙 ただし 補完情報が無ければ、その時点で意味がなくなるので処理を抜ける
    List<aaaEntity> detailList = this.selectBId(resultList1);
    if (null == detailList || detailList.size() < 0) return;

    for (aaaEntity m : resultList1) {
        // 詳細を持つ場合は次のレコードへスキップする。インデントを深くしない目的の continue
        if (null != m.getBId()) continue;

        for (aaaEntity detail : detailList) {

            // [TODO] m と detail のキーの合致判定をここで行う
            boolean judge = false;

            if (judge) {
                // [TODO] m に detail の持つ詳細情報の転記を行う
            }
        }
    }
}

老婆心ながら、UNION を使うという選択肢が出る時点でクエリやらテーブル設計に問題が有ると疑ったほうが良いです。業務上や立場上、手が出したくても出せないのであれば察しますが、まともな構造であれば JOIN を使って全て SQL 側で片付けるべき/片付けられる処理でしょう。UNION は重複排除のソートを行う分、レコード数に比例してパフォーマンスが大変残念な事になりやすいため、規模によっては大した問題では無いかもしれませんが、極力使用は避けるべきと考えます。

Answer 2

とりあえず上書きされる原因だけは明確ですね。
コード内コメントで対象の箇所を※で示しておきます。

for (aaaEntity m : resultList1) { // ←resultList1の要素数分繰り返す
     // List更新用のListを作成。
     String bId = null;
     bId = m.getBId();
     if (bId == null) { // ←BIDがnullの場合
         List<aaaEntity> resultList3 = this.selectBId(resultList1); // ←ここって何してるんですか？
         if (resultList3 != null && resultList3.size() > 0) {
             for (int i = 0; i < resultList3.size(); i++) {
                 for (aaaEntity x : resultList3) { // ←どうしてここでresultList3の要素数分の繰り返し処理をするの？
                     resultList1.set(i, x); // ←※ここでresultList1のi番目をresultList3の最後の要素に置き換えてる
                 }
             }
         }
     }
 }

やりたい処理に対する期待値を言葉ではなく表なので記載してもらえたら答えやすいと思います。

Comments

[退会済みユーザー]

6行目のresultList3を取得している部分は別メソッドにて、resultList1で取得できなかった項目を取得するSQLを実行しています。

[退会済みユーザー]

６行目で取得したいものってBIDなんですよね？
だとしたらそもそもメソッドのIFがおかしい気がします。
aaaEntityにAID（キー項目）,BID,CIDみたいなフィールドがある想定で回答させていただきます。
aaaEntity.BIDがNULLの場合に別SQLでBIDを取得するために必要な情報はキー項目であるAIDだけのはずです。
それなのにselectBIdメソッドにはList<aaaEntity>を渡しています。
これではresultList1の何番目のaaaEntityに設定したいのかが判断できません。
それとselectBIdメソッドの戻り値がList<aaaEntity>となっているのも気になります。
"selectBId"メソッドはBID（String値）をSELECTしてくるメソッドという意味で名前をつけたんですよね？

とりあえずコードを書くよりも、一度フローチャートをしっかりと考えてからコーディングした方が理解が深まると思います。

（ER次第だけど、SQL一回で事が済みそうな気がしてならない･･･）

Answer 3

resultList1に
{"A", "B", null, "D"}がある時

1. resultList3はどういう内容ですか
2. 最終的に得たいresultList1はどういう内容ですか

Comments

[退会済みユーザー]

resultListが上記の場合
主キーとなる
AはresultList3にも存在し、その他はresultList1でnullとなった個所を含みます。

例として
resultList1 = {A,B,null,D}
resultList3 = {A,null,C,D}などです。

最終的には
resultList1に値の更新・追加を行い
resultList1 = {A,B,C,D}となるようにしたいです。

[liguofeng29]

List<aaaEntity> resultList3 = this.selectBId(resultList1);
と書いてますが、主キーとなるってことは
aaaEntity entity = this.selectBId(resultList1);
になりませんか？

Answer 4

とりあえず、改行しました。
すべて上書きだったということは、すべてnullだったってことですか？

for (aaaEntity m : resultList1) {
     // List更新用のListを作成。
     String bId = null;
     bId = m.getBId();
     if (bId == null) {
         List<aaaEntity> resultList3 = this.selectBId(resultList1);
         if (resultList3 != null && resultList3.size() > 0) {
             for (int i = 0; i < resultList3.size(); i++) {
                 for (aaaEntity x : resultList3) {
                     resultList1.set(i, x);
                 }
             }
         }
     }
 }

Comments

[退会済みユーザー]

nullではなく、本来はresultList1にresultList3をnull項目のみ更新の形で追加したいと思っています。
現在はresultList3の結果で更新されてしまっています。