Open packdev937 opened 10 months ago
패키지 전반적으로 리팩토링을 진행하였습니다.
패키지 구조의 일부를 변경하였습니다. 기존에 domain 패키지에 모여있던 domain 클래스들을 조금 더 세분화 하여 분리하였습니다. 예를 들어, review와 관련된 domain 클래스는 모두 domain - review 패키지로 리팩토링하였습니다.
Enum 클래스 내 정적 팩토리 메소드가 찾고자 하는 값이 없을 시 예외를 던지도록 수정하였습니다.
@JsonCreator public static ReportStatus from(final String description) { return Arrays.stream(ReportStatus.values()) .filter(v -> v.getDescription().equals(description)) .findAny() .orElseThrow(IllegalArgumentException::new); }
public static ReviewRateCountResponse toResponse() { return new ReviewRateCountResponse( RATE_1.getCount(), RATE_2.getCount(), RATE_3.getCount(), RATE_4.getCount(), RATE_5.getCount() ); }
ReviewCalculator 중 일부
public ReviewRateCountResponse menuRateCount(Menu menu) { ReviewRate.resetAll(); menu.getReviews().calculateReviewRates(); return ReviewRate.toResponse(); }
Summary
패키지 전반적으로 리팩토링을 진행하였습니다.
패키지 구조
패키지 구조의 일부를 변경하였습니다. 기존에 domain 패키지에 모여있던 domain 클래스들을 조금 더 세분화 하여 분리하였습니다. 예를 들어, review와 관련된 domain 클래스는 모두 domain - review 패키지로 리팩토링하였습니다.
변수 명 변경
정적 팩토리 메소드
Enum 클래스 내 정적 팩토리 메소드가 찾고자 하는 값이 없을 시 예외를 던지도록 수정하였습니다.
책임에 맞게 분배
Map 대신 DTO
ReviewCalculator 중 일부