Skip to content

Conversation

@X1n9fU
Copy link
Contributor

@X1n9fU X1n9fU commented Jun 27, 2025

#️⃣ 연관된 이슈

ex) #이슈번호, #이슈번호
#210

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • Chat 패키지 구조 변경
    chatroom, chatting으로 나눠진 패키지를 controller, domain, repository, service 내부에서 동일한 깊이에 위치하도록 설정

변경 전

chat
- chatroom
  - controller
  - domain
  ...
- chatting
  - controller
  - domain
  ...

변경 후

chat
- controller
  - ChatRoomController
  - ChattingController
- domain
- service
  • 전반적인 모듈화 진행
  • ChatRoomService, ChattingService에서 ParticipantsInfo에 직접 접근하는 로직을 모두 Participants까지 접근하도록 설정
    -> Participants 에서 ParticipantsInfo로 접근하도록 캡슐화 진행
  • 불필요한 필드 삭제 및 필드명을 직관적으로 변경
unreadMessage -> unreadCount
MessageType 삭제
등등
  • ChatRoomValidator을 통해 채팅방 생성 시 확인해야하는 로직 분리
    ( 이미 존재하는 채팅방이 있을 시에 해당 채팅방으로 리다이렉트, 303 에러로 변경)

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • blockedUser를 다루는 chatting 로직 변경 X (getAllChatRoomByUser())
  • 아직 테스트를 진행하지 않았기 때문에 테스트까지 작성 후 다시 리뷰 요청을 할 예정. 그 전에 수정한 내용이 많기 때문에 먼저 리뷰를 요청드린 것.

@X1n9fU X1n9fU requested review from doma17 and gisu1102 June 27, 2025 15:28
@X1n9fU X1n9fU self-assigned this Jun 27, 2025
@X1n9fU X1n9fU added 🔨 Refactor 코드 리팩토링 Priority: Medium 우선순위 중 Status: Reviewing 처리하고 리뷰 중인 이슈 labels Jun 27, 2025
Copy link
Contributor

@doma17 doma17 left a comment

Choose a reason for hiding this comment

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

추후 검토를 추가하겠습니다.

@X1n9fU X1n9fU removed the Status: Reviewing 처리하고 리뷰 중인 이슈 label Jul 11, 2025
@X1n9fU X1n9fU added the Status: Pending 보류 중인 이슈 label Jul 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Medium 우선순위 중 🔨 Refactor 코드 리팩토링 Status: Pending 보류 중인 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants