Skip to content

Conversation

@sycuuui
Copy link
Member

@sycuuui sycuuui commented Jul 27, 2025

🚩 관련 이슈

📋 구현 기능 명세

  • [x]

📌 PR Point

PlanService에 모든 로직이 구현되어있는 것을 기능과 로직에 맞게 패키지 및 클래스 분리하여 SRP를 최대한 지키도록 리펙토링하였습니다.

  • 도메인 service 클래스
  • 생성, 삭제, 수정 등 비지니스 흐름 조율의 메소드 구현
  • PlanService에서 PlanPlaceService, PlanReviewService, PlanReviewImageService 클래스들을 생성하여 엔티티 기준으로 기능 구현하도록 분리
  • deleter/modifier/factory 패키지
  • 삭제/수정/생성 로직 구현
  • validator 패키지
  • 엔티티 기준으로 클래스 구분하여 검증 로직 구현
  • query 패키지
  • 조회 로직 구현
  • PlanQueryService,PlanDetailQueryService,PlanReviewQueryService 클래스들을 생성하여 조회 타켓을 기준으로 조회하도록 분리
  • util 패키지
  • DateUtil 클래스 생성하여 반복적으로 호출되는 날짜 계산 메소드 분리
  • S3UrlUtil 클래스 생성하여, S3 폴더명 생성 메소드 분리

  • 어떤 부분에 리뷰어가 집중해야 하는지
    최대한 책임 분리 하도록 리펙토링 하였는데, 부족한 부분이 있는지 확인해주시면 감사하겠습니당

sycuuui added 21 commits July 27, 2025 17:22
@sycuuui sycuuui requested a review from mmihye July 27, 2025 11:42
@sycuuui sycuuui self-assigned this Jul 27, 2025
Copy link
Member

@mmihye mmihye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

책임에 따리 분리되어 더 코드가 보기 편해진거같네요!!
수고하셨습니다 👍🏻

@sycuuui sycuuui merged commit 7b866a0 into main Aug 4, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: PlanService 책임 분리

3 participants