Javaのifの記述とDTOへの追加について

Question

前提

①batch_idを引数に、下記DBのテーブルからコード値（code_value）を取得
尚、code_valueは他のバッチ処理で更新されることがあり、batch_idによってcode_valueの上限は異なります。

batch_id	code_value
JKL-095	0
HJK-012	1
BBD-242	1
KSG-67	3

②List型のDTOを返却（List<batchCodeDto>）

③取得したコード値を元に、設計書で記載された文字列を、返却されたListに追加して返却
見づらくて申し訳ないですが、0：新規登録の場合、取得したcode_valueが0だったら、"新規登録"という文字列を追加で返却したい意図になります。

batch_id	code_value
JKL-095	0：新規登録, 1：更新, ２：削除
HJK-012	0：個人, 1：法人
BBD-242	0：直接契約, 1：請負契約, ２：業務委託, 3：準委任契約, 4：派遣契約
KSG-67	0：月初締め・清算が15日/振り込みが25日, 1：月初締め・清算が月末/振り込みが1日, ２：月末締め・清算が翌月15日/振り込みが翌月25日, 3：月末締め・清算が翌々月末/振り込みが翌々月1日, 4：10日締め・清算が25日/振り込みが翌月1日

質問

上記③にもありますが、取得したcode_valueの値から、紐づく文字列をcontent変数としてListに追加したいのですが、
・DTOにcontentを用意しておいて、そこに格納するのが定石でしょうか？それとも後からロジックで返却されたオブジェクトに追加するものでしょうか？
・if文は以下コードのような記載だと冗長な気がするのですが、スマートな記述がありましたらご教示ください。

該当のソースコード

// テーブルからデータ取得後のロジックのみ記述
if (batch_id.equals("JKL-095")) {
  if (code_value == 0) {
    content = "新規登録";
  } else if (code_value == 1) {
    content = "更新";
  } else if (code_value == 2) {
    content = "削除";
  }
} else if (batch_id.equals("HJK-012")) {
  if (code_value == 0) {
    content = "個人";
  } else if (code_value == 1) {
    content = "法人";
  }
} else if (batch_id.equals("BBD-242")) {
  if (code_value == 0) {
    content = "直接契約";
  } else if (code_value == 1) {
    content = "請負契約";
  } else if (code_value == 2) {
    content = "業務委託";
  } else if (code_value == 3) {
    content = "準委任契約";
  } else if (code_value == 4) {
    content = "派遣契約";
  }
} else if (batch_id.equals("KSG-67")) {
  if (code_value == 0) {
    content = "月初締め・清算が15日/振り込みが25日";
  } else if (code_value == 1) {
    content = "月初締め・清算が月末/振り込みが1日";
  } else if (code_value == 2) {
    content = "月末締め・清算が翌月15日/振り込みが翌月25日";
  } else if (code_value == 3) {
    content = "月末締め・清算が翌々月末/振り込みが翌々月1日";
  } else if (code_value == 4) {
    content = "10日締め・清算が25日/振り込みが翌月1日";
  }
}

// 別のリストに詰め替えて返却すべき？
batchCodeDto.add(content);

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

java17

Comments

[jimbe]

回答にありますが、前提として表にしているものを何故テーブルにしないのでしょうか。
また、 コードにするなら表から map や配列を使えることは明白と思いますが、何故 if の塊にしているのでしょうか。

[退会済みユーザー]

テーブルについてはおっしゃる通りなのですが、今回は諸々の要因でこのような形になってしまっています。
mapや配列については全件取得してから必要な値を取り出すということでしょうか？

[jimbe]

>今回は諸々の要因でこのような形

つまりデータベースに文字列を持たせるのは(今回は)無しでということですかね。
理由は不要ですが、そのような条件的なこと質問に書いておいて頂いたほうが良いです。

map 等のことは、 shiketa さんの回答と同様なコードをイメージしています。(code_value → content 文字列の変換は String 配列のつもりです。)

Answer 1

システムの要件・事情次第にもよりますが、個人的には以下のようなコードマスタのようなものを用意することが好みです。

batch_id(PK)	code_value(PK)	content
JKL-095	0	新規登録
JKL-095	1	更新
JKL-095	2	削除
HJK-012	0	個人
HJK-012	1	法人
BBD-242	0	直接契約
BBD-242	1	請負契約
BBD-242	2	業務委託
BBD-242	3	準委任契約
BBD-242	4	派遣契約
KSG-67	0	月初締め・清算が15日/振り込みが25日
KSG-67	1	月初締め・清算が月末/振り込みが1日
KSG-67	2	月末締め・清算が翌月15日/振り込みが翌月25日
KSG-67	3	月末締め・清算が翌々月末/振り込みが翌々月1日
KSG-67	4	10日締め・清算が25日/振り込みが翌月1日

contentが必要な場合は、batch_idとcode_valueをキーにこのテーブルをSELECTします。あるいは①に記載されているテーブルとこのコードマスタをJOINしてもよいです。

Comments

[退会済みユーザー]

何故か諸々要因でこのような形になってます。
本来であればDBに持たせるべきですよね。
ご回答ありがとうございます。

Answer 2

定石は知りませんが、たとえばなら、こうするかな。

public class batchCodeDto {
  
  private String batch_id;
  private int code_value;

  public String getContent() {
    return HOGEGE.get(batch_id).get(code_value);
  }

  private static final Map<String, Map<Integer, String>> HOGEGE =
      Map.of(
          "JKL-095", Map.of(
              0 , "新規登録",
              1, "更新",
              2, "削除"
          ),
          "HJK-012" , Map.of(
              0, "個人",
              1, "法人"
          ),
          "BBD-242", Map.of(/* ... */)
      );
}

Comments

[退会済みユーザー]

ありがとうございます。
キーバリューでセットしておいて取り出すということですね。