[feat] 여행 친구 전체 조회 API 스펙 및 비즈니스 로직 변경 #147

SunwoongH · 2024-03-07T08:34:08Z

Related Issue 📌

2차 스프린트 기능 변경 사항을 반영하여 여행 친구 전체 조회 API 비즈니스 로직을 변경하였습니다.
여행 성향 테스트 결과 통계를 계산하는 로직을 Actor 클래스로 분리하였습니다.
여행 성향 테스트 결과 통계를 계산하는 모든 책임은 TripTendencyTestActor 클래스가 가지며 서비스 클래스에서 해당 Actor 클래스로 계산을 위임한 후 TripTendencyTestResult 라는 결과만을 받아 사용할 수 있도록 리팩터링하였습니다.
Actor 클래스로 분리한 이유는 다음과 같습니다.
1. 기존에 서비스 계층에 모든 관심사가 섞여있어 여행 성향 테스트 결과 통계 계산 로직을 구분하기 힘든 불편함이 있었습니다. 해당 로직을 구분하기 힘들어 이번처럼 기능 수정이 필요할 때 개발 비용이 많이 든다고 느껴졌습니다.
1. 마찬가지로 서비스 계층에 모든 관심사가 섞여있어 여행 성향 테스트 결과 통계 계산 로직에 대한 테스트 코드를 작성하기 어렵다고 느껴졌고 하나의 서비스 클래스가 너무 많은 책임을 가지고 있다고 판단되어 분리하였습니다.

gardening-y

수고하셨습니다!!!

SunwoongH added 5 commits March 7, 2024 17:15

fix: 여행 성향 검사 결과 dto 수정 (#146)

123845b

fix: 여행 친구 전체 조회 dto 수정 (#146)

4658811

feat: 여행 성향 검사 통계 결과 dto 구현 (#146)

4d4292e

feat: 여행 성향 검사 통계 비즈니스 로직 담당 actor 클래스 구현 (#146)

8917cba

feat: 여행 친구 전체 조회 비즈니스 로직 변경 (#146)

37c4f20

SunwoongH requested a review from gardening-y March 7, 2024 08:34

SunwoongH self-assigned this Mar 7, 2024

SunwoongH added feat 선웅 labels Mar 7, 2024

feat: 프로필 성향 테스트 skip case 대응 (#146)

3e134f8

gardening-y approved these changes Mar 7, 2024

View reviewed changes

SunwoongH merged commit 71af0eb into develop Mar 7, 2024
1 check passed

SunwoongH deleted the feature/146 branch July 11, 2024 04:49