[Java]기간 내 데이터 조회로 직 사용하지 못할 정도인가요?

Question

public BooleanExpression searchDateQuery(String startDate, String endDate) { LocalDate searchStartDate &#61; (!Objects.equals(startDate, &#34;&#34;)) ? LocalDate.parse(startDate) : null; LocalDate searchEndDate &#61; (!Objects.equals(endDate, &#34;&#34;)) ? LocalDate.parse(endDate) : null; if (searchStartDate &#61;&#61; null && searchEndDate &#61;&#61; null) { LocalDateTime currentStartDate &#61; LocalDateTime.of(LocalDate.now(), LocalTime.of(0, 0, 0)); LocalDateTime currentEndDate &#61; LocalDateTime.of(LocalDate.now(), LocalTime.of(23, 59, 59)); //날짜데이터 없으면 하루 시작과 끝 return productApplication.regidate.between(currentStartDate, currentEndDate); } else if (searchStartDate !&#61; null && searchEndDate &#61;&#61; null) { LocalDateTime startDateTime &#61; LocalDateTime.of(LocalDate.from(searchStartDate), LocalTime.of(0, 0, 0)); //시작 날짜만 있으면 있으면 시작날부터 쭉 뒤로 return productApplication.regidate.goe(startDateTime); } else if (searchStartDate &#61;&#61; null) { //시작날짜 없으면 해당하는 달 1일 부터 조회일 까지 LocalDate firstMonthDate &#61; LocalDate.from(searchEndDate.with(TemporalAdjusters.firstDayOfMonth())); LocalDateTime startDateTime &#61; LocalDateTime.of(firstMonthDate, LocalTime.of(0, 0, 0)); LocalDateTime endDateTime &#61; LocalDateTime.of(LocalDate.from(searchEndDate), LocalTime.of(0, 0, 0)); return productApplication.regidate.between(startDateTime, endDateTime); } else { LocalDateTime startDateTime &#61; LocalDateTime.of(LocalDate.from(searchStartDate), LocalTime.of(0, 0, 0)); LocalDateTime endDateTime &#61; LocalDateTime.of(LocalDate.from(searchEndDate), LocalTime.of(23, 59, 59)); return productApplication.regidate.between(startDateTime, endDateTime); } } 시작일과 종료일을 받아와 그기간의 데이터를 뽑는 조건문입니다.실무에서 사용한다고 가정하였을 때 이 정도 로직은 사용할 수 없을 정도인 건가요?????좋은 방법이 떠오르질 않습니다!!..ㅜ선배님들

quedevel · Answer

로직에 오류가 있지 않다면 사용하지 못하는건 아니겠지만 좀 더 명시적인게 좋겠죠. 예를 들면 searchStartDate !&#61; null && searchEndDate &#61;&#61; null 이 조건이 아니고선 모두 between을 사용하시는것 같은데 더 줄일 수 있지 않을까요? 공통으로 묶을 수 있는것부터 시작해보시면 좋을것 같습니다.

yeori · Answer

중복 코드가 넘 많네요.보통 변수값을 정규화하는 과정인데, 날짜를 정규화하는 함수 만들어서 코드를 읽기 편하게 수정하면 좋습니다.

[Java]기간 내 데이터 조회로 직 사용하지 못할 정도인가요?

2개의 답변

#date 목록