適用期間の開始日と終了日を含みたい

Question

###　重複する期間が登録された際にエラーメッセージを表示する

Javaを勉強中です。
登録される期間が既に登録されている期間と重複している場合エラーメッセージを表示するバリデーションを
作成したいのですが現在のソースだと開始日と終了日が含まれず悩んでいます。
開始日と終了日を含むにはどのように改良すると良いのでしょうか。

現在
9/1~12/31

8/19/1　メッセージ表示されない
8/19/2　メッセージ表示される

該当のソースコード

private ArrayList<Tax> taxList = new ArrayList<>();
	public void putTax(Tax tax) {
		if (taxList.stream()
				.anyMatch(
						e -> e.getApplyStartDate().isBefore(tax.getApplyEndDate())
								&& e.getApplyEndDate().isAfter(tax.getApplyStartDate()))) {
			System.out.println("税率適用期間に重複があります。登録値を確認してください。");
		}

		taxList.add(tax);
	}

public class Tax {
	private LocalDate applyStartDate;
	private LocalDate applyEndDate;
	private BigDecimal taxRate;

    // setter
    public void setApplyStartDate(LocalDate start) {
        this.applyStartDate = start;
    }

    public void setApplyEndDate(LocalDate end) {
        this.applyEndDate = end;
    }

    // Getter

    public LocalDate getApplyStartDate() {
        return this.applyStartDate;
    }

    public LocalDate getApplyEndDate() {
        return this.applyEndDate;
    }
}

Comments

[jhashimoto]

TaxにgetApplyStartDateやgetApplyEndDateがないので、コンパイルが通りません。コンパイルに必要なコードは全部含めて提示してください。

[dodox86]

isBeforeとisAfterは、「より前」と「より後」だからダメなのではないですか？
https://docs.oracle.com/javase/jp/8/docs/api/java/time/LocalDate.html

[xebme]

isBeforeの否定は該当日付を含みかつisAfterなのかも。isAfterの否定は該当日付を含み...。

[dodox86]

> アノテーションの記載漏れがありましたので

Lombokは別にJava標準なものではないし、提示のコード中にも示されている訳でもないので、@Dataアノテーションを付けているからといってsetter、getterが自動生成、というのは読んだ者に伝わらないと思います。

[__student]

ご指摘ありがとうございます。
>Lombokは別にJava標準なものではない
その通りだと思いましたのでsetterとGetter
を記載しました。

Answer 1

isAfterやisBeforeはその名の通り「より前」や「より後」かどうかを判定するので、「同じ日」はfalseになります。
同じ日を含むなら、compareToを使うのがいいかと。

        if (taxList.stream()
                .anyMatch(
                        e -> e.getApplyStartDate().compareTo(tax.getApplyEndDate()) <= 0
                             && e.getApplyEndDate().compareTo(tax.getApplyStartDate()) >= 0))

条件を反転させれば、isAfterやisBeforeを使って書けます。
反転のさせ方はいろいろありますが、個人的にはこう書きますかね。

        if (taxList.stream().anyMatch(
               e -> !(e.getApplyStartDate().isAfter(tax.getApplyEndDate()) 
                       || e.getApplyEndDate().isBefore(tax.getApplyStartDate()))
            ))

(tax開始～)tax終了→e開始(e終了)　または　(e開始)e終了→tax開始(~tax終了)
となるのが「正常」なわけですから、「そうではない」ものを探す、という意味で分かりやすいかと、
ただし、Taxクラスの開始日終了日が、「終了日は開始日以後」という当たり前の前提が満たされていれば、の話ですが。

Comments

[__student]

ご教授ありがとうございます。
2種類の書き方と考え方を回答いただきとても勉強になりました。