異なるDTOから同じ名前のgetterを使って値を取得することはできますか？

Question

以下のようにgetter / setterが同じ名前のDTOが二つあります。

public class Test1Dto {
    private String userId;
    private String userName;
    private String address;
    
    // ======getter / setter は省略 ======
}

public class Test2Dto {
    private String userId;
    private String userName;
    private String address;
    
    // ======getter / setter は省略 ======
}

現在、下記のように三項演算子を使って片方のDTOがNullであれば、もう片方から取得するようにしているのですが、DTOから取得する値が増えた場合、冗長になるなぁと思っています。

Test1Dto test1Dto = null;
Test2Dto test2Dto = null;

// =======DTO値取得部分は省略========

String userId = test1Dto != null ? test1Dto .getUserId() : test2Dto .getUserId();
String userName= test1Dto != null ? test1Dto .getUserName() : test2Dto .getUserName();
String address= test1Dto != null ? test1Dto .getAddress() : test2Dto .getAddress();

三項演算子以外に、上記の冗長なコードを回避する方法はありますでしょうか？

Answer 1

interfaceのリストを渡して処理するクラスを作ればいいのではないでしょうか。
※shiketaさんのInterfaceとDTOをパクっています
getter,setter,コンストラクタ書くのダルいのでlombokを使っています。

import java.util.List;
import java.util.Objects;
import java.util.function.Predicate;
import lombok.RequiredArgsConstructor;

@RequiredArgsConstructor
public class TestDtoFacade {

  private final List<TestInterface> testDtoList;

  public String getUserId() {
    var dto = this.testDtoList.stream()
        .filter(Predicate.not(Objects::isNull))
        .findFirst();
    if (dto.isEmpty()) {
      return "";
    }
    return dto.get().getUserId();
  }
}

mainの方はlistにDTOを追加するだけです。

    Test1Dto test1Dto = null;
    Test2Dto test2Dto = null;

    List<TestInterface> list = new ArrayList<>();
    list.add(test1Dto);
    list.add(test2Dto);

    var facade = new TestDtoFacade(list);
    String userid = facade.getUserId();

Answer 2

同じような文が繰り返し書かれていたとしても、それが必要なコードなのであれば、「冗長」ではありません。
それはともかく、機械的に全プロパティーを参照してしまっていいなら、リフレクションを使えば、プロパティーが増えても、コードは変更しなくてすみます。
https://qiita.com/suke_masa/items/0df3bb92bcb69f4a95d1

Answer 3

まず、Test1DtoとTest2Dtoを同様に扱いたいのであれば、インタフェースは必要でしょう。

  interface TestInterface {
    String getUserId();
  }

  public class Test1Dto implements TestInterface {
    private String userId;
    private String userName;
    private String address;

    @Override public String getUserId() { return userId; }
  }

  public class Test2Dto implements TestInterface {
    private String userId;
    private String userName;
    private String address;

    @Override public String getUserId() { return userId; }
  }

そのうえで。

三項演算子以外に、上記の冗長なコードを回避する方法はありますでしょうか？

三項演算子がどうしても嫌いなら、こうするかな。

  private void hoge() {
    TestInterface test1Dto = null;
    TestInterface test2Dto = null;

    String userId =
        Optional.ofNullable(Optional.ofNullable(test1Dto).orElse(test2Dto)).get().getUserId();
  }

でも、このほうがまだましかも。

  private void hoge2() {
    Test1Dto test1Dto = null;
    Test2Dto test2Dto = null;

    TestInterface dto = test1Dto == null ? test2Dto : test1Dto;

    String userId = dto.getUserId();
  }

いずれにしても、NEPは発生しますけど。