java8のOptionalの複数オブジェクトの判定方法

Question

実現したいこと

画面から入力された２つの日付でDB検索を行うのですが、
開始日だけ入力される場合と、終了日だけ入力される場合と、両方入力される場合があり、
それぞれのwhere句文作成したい。

以下の記述で両方ある場合は実現できたのですが、片方だけある場合の記述の仕方がわかりません・・・。

試したソース

	public static void main(String[] args) {
		Optional<Date> day1 = Optional.ofNullable(new Date());
		Optional<Date> day2 = Optional.ofNullable(new Date());

		Optional<String> ret = day1.flatMap(str1 -> 
									day2.flatMap(str2 -> {
										return Optional.ofNullable("両方ある場合");
									})
								);
	}

環境

Java 8
Spring 4.6

Answer 1

StringJoiner使っちゃダメ？

StringJoiner sj = new StringJoiner(" AND ");
Optional<Date> day1 = Optional.ofNullable(new Date());
Optional<Date> day2 = Optional.ofNullable(new Date());

day1.ifPresent(d -> sj.add("DATE >= " + d));
day2.ifPresent(d -> sj.add("DATE <= " + d));
System.out.println(sj);

Comments

[KSwordOfHaste]

あっけないぐらい分かり易いですね！

[keinon]

StringJoinerなんてできたんですね！
知らかなったです。
こちらも検討してみます。

Answer 2

最初に直接関係ない話ですが、値がnullの可能性があるかないかでOptional.ofとOptional.ofNullableは使い分けた方がよいと思います。

それはそれとして、以下のような感じではいかがでしょう

import java.sql.Date;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.Optional;
import java.util.stream.Collectors;
import java.util.stream.Stream;

public class Boo {
  public static void main(String[] args) {
    Optional<Date> d1 = Optional.of(new Date(sdf.parse("2018-01-01").getTime()));
    Optional<Date> d2 = Optional.of(new Date(sdf.parse("2018-02-02").getTime()));
    Optional<Date> dn = Optional.empty();
    
    System.out.println(buildCondition(d1, d2));
    System.out.println(buildCondition(d1, dn));
    System.out.println(buildCondition(dn, d2));
  }
  
  static String buildCondition(Optional<Date> from, Optional<Date> to) {
    return Stream.of(
      from.map(d -> "DATE >= " + fmt(d)),
      to.map(d -> "DATE <= " + fmt(d))
    ).flatMap(Optional::stream).collect(Collectors.joining(" AND "));
  }

  static SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");
  
  static String fmt(Date date) {
    return sdf.format(date);
  }
}

==>
DATE >= 2018-01-01 AND DATE <= 2018-02-02
DATE >= 2018-01-01
DATE <= 2018-02-02

---

ご質問にあるコードからPreparedStatementのsetDateは使わない想定のようでしたので、日付はリテラルとしてフォーマットするようなコードにしています。フォーマットの仕方そのものはいいかげんです。

---

追記：Java9前提です。Java8にはOptional#stream()がありませんので。

---

追記2:Java8で上記の方針とするならちょっと長めのコードになりますね。
.flatMap(Optional::stream).～
=>
.flatMap(o -> o.map(Stream::of).orElseGet(Stream::empty)).～

Comments

[keinon]

回答ありがとうございます。
こちらの環境がjava8のため、そのままは使えませんが、文字列の組み立て方が良いヒントになりそうです。

自分の頭ではif-elseの中身をそれぞれの分岐ですべて記述しないと無理そうだったのですが、こんな組み立て方もできるのですね・・・。
勉強になります。

[KSwordOfHaste]

Optionalを要素が０個か1個のストリーム的なものとみなすことができ、実際他の言語ではそのままストリーム的なものと互換性があるものとして設計されてたりするようです。そうなってるとflatMapと組み合わせて便利なことが多いと思います。残念ながらJavaはそうではないので、Java9でOptional#streamがサポートされたのだと思います。Java8のコードも追記してみました。

ついでに元のコードで「flatMap(o -> o.stream())」とちょっと迂遠なことをしていたので、
「flatMap(Optional::Stream)」に直しました ;;;

[keinon]

ご親切にありがとうございます。
ご提示くださった仕組みでメンバーと相談してみます。

Answer 3

Optional<String> ret = day1
    .map(Optional::of)
    .orElse(day2)
    .map(day -> "どちらかがnullでない場合");

シンプルに書くなら、こちらでいかがでしょうか。

Comments

[keinon]

ご回答ありがとうございます。
どちらかnullではなく、day1がnullの場合とday2がnullの場合で挙動が変わってくる為、この案を元にもう少し考えてみます。