Проверьте дату между диапазоном дат, которые также обработать нулевые значения Java

Question

Моего стандартный метод для проверки, если дата между двумя датами в Java выглядит следующим образом:

public static boolean isWithinRange(Date date, Date startDate, Date endDate) { 
    return !(date.before(startDate) || date.after(endDate)); 
} 

Я хочу, чтобы добавить поддержку для нулевых значений на STARTDATE или endDate (то есть пользователь не ввел дату.Если startDate имеет значение null, я хочу только проверить endDate, и если endDate имеет значение null, я хочу только проверить startDate, и если оба значения null, то это правда. Мое текущее решение выглядит так:

public static boolean isWithinRange(Date date, Date startDate, Date endDate) { 
    if (startDate == null && endDate == null) { 
     return true; 
    } 

    if (!(startDate != null || !date.after(endDate))) { 
     return true; 
    } 

    if (!(endDate != null || !date.before(startDate))) { 
     return true; 
    } 

    return !(date.before(startDate) || date.after(endDate)); 
} 

альтернативный более читаемый пример:

public static boolean isWithinRange(Date date, Date startDate, Date endDate) { 
    if (startDate == null && endDate == null) { 
     return true; 
    } 

    if (startDate == null && date.before(endDate))) { 
     return true; 
    } 

    if (endDate == null && date.after(startDate))) { 
     return true; 
    } 

    return date.after(startDate) && date.before(endDate)); 
} 

Но это похоже на действительно плохой код. Есть ли другой способ справиться с этим?

Answer 1

Как насчет:

return (startDate == null || !date.before(startDate)) 
    && (endDate == null || !date.after(endDate)); 

---

Это использует тот факт, что эти два утверждения эквивалентны:

!(date.before(startDate) || date.after(endDate)) 
!date.before(startDate) && !date.after(endDate) 

И тот факт, что || является коротким замыканием, которое предотвращает NullPointerExceptions.

Answer 2

Это должно быть эквивалентно:

Усиливает

public static boolean isWithinRange(Date date, Date startDate, Date endDate) { 
    return !(startDate != null && date.before(startDate) || endDate != null && date.after(endDate)); 
} 

Answer 3

Java 8, чтобы интерфейс Comaparator обеспечивают довольно изящный способ сделать это:

private static final Comparator<Date> NULL_SAFE_BEFORE = 
     Comparator.nullsFirst(Comparator.naturalOrder()); 

private static final Comparator<Date> NULL_SAFE_AFTER = 
     Comparator.nullsLast(Comparator.naturalOrder()); 

public static boolean isWithinRange(Date date, Date startDate, Date endDate) { 
    return NULL_SAFE_BEFORE.compare(startDate, date) < 0 && 
      NULL_SAFE_AFTER.compare(date, endDate) < 0; 
}