Skip to content

Conversation

ChaeAg
Copy link
Member

@ChaeAg ChaeAg commented Mar 21, 2025

Related Issue

Key Changes

Feed 도메인에 실험적으로 Facade 패턴을 도입하여, Controller가 직접 여러 Service에 의존하지 않고, 비즈니스 흐름을 단일 진입점(FeedFacade)으로 위임하도록 리팩터링했습니다.

주요 변경사항

  • FeedControllerFeedFacade로 호출 책임 이전
  • Controller는 단순한 요청 분기와 인증 처리만 담당하도록 구조 단순화
  • FeedService, CommentService, PopularFeedService, LikeService 등은 도메인 중심으로 책임을 재정리
  • Feed, Comment 엔티티의 내부 메서드 정리 및 일부 이름 변경 (hideFeedhide 등)

테스트 사항

  • 피드 조회, 생성, 수정, 삭제 정상 작동

  • 댓글 조회, 생성, 수정, 삭제 정상 작동

  • 좋아요, 신고 정상 작동

  • ❗푸시 알림 관련 기능은 notification 레코드가 쌓이는 것은 확인했으나, 로컬 환경에서 테스트하지 못했습니다. (해당 기능 테스트 필요)

To Reviewers

원래는 우선 피드 관련 로직에만 퍼사드 패턴을 적용 해보고 괜찮으면 다른 도메인에도 확장하고자 했는데 한 번 봐보시고 괜찮다 싶으면 적용해주세요 ..ㅎㅎ

@ChaeAg ChaeAg requested a review from Kim-TaeUk March 21, 2025 16:24
@ChaeAg ChaeAg self-assigned this Mar 21, 2025
@ChaeAg ChaeAg requested review from rinarina0429 and removed request for Kim-TaeUk March 21, 2025 16:24
@github-actions github-actions bot requested a review from Kim-TaeUk March 21, 2025 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] Feed 관련 코드 리팩토링
1 participant