Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 2, 2025

📄 요약(Summary)

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 작업(Task) 도메인 레이어의 비즈니스 규칙과 정책을 분리: Policy
    • 작업 도메인 모델 클래스가 너무 많고 복잡한 비즈니스 로직을 담당하고 있어 정책을 담은 class로 분리하였습니다. (특히 작업 보드 순서를 변경하는 서비스)
    • Policy는 복잡한 도메인 규칙, 재사용 될 수 있는 로직, 요구사항에 따라 변경될 수 있는 로직 등을 캡슐화합니다.
    • 핵심 비즈니스 로직과 정책을 분리하였기 때문에 각각 독립적으로 단위 테스트가 가능하며, 비즈니스 로직에 수정이 필요해도 변경사항이 크지 않습니다.

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

  • 알림 전송 시 메세지 생성 로직 부분 캡슐화
    • notification type에 따라 policy또는 util로 message 템플릿을 생성하는 클래스가 필요해보입니다.

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #215

<footer>
- 관련: #215
# Conflicts:
#	src/main/java/clap/server/application/service/comment/PostCommentService.java
#	src/main/java/clap/server/application/service/task/ApprovalTaskService.java
#	src/main/java/clap/server/application/service/task/CreateTaskService.java
#	src/main/java/clap/server/application/service/task/UpdateTaskService.java
@joowojr joowojr changed the title CLAP-213 CLAP-213 작업 도메인 비즈니스 규칙과 정책 분리 Feb 2, 2025
@joowojr joowojr self-assigned this Feb 2, 2025
@joowojr joowojr added 🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. and removed 🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. labels Feb 2, 2025
@joowojr joowojr requested a review from parkjaehak February 2, 2025 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-213 작업 도메인 비즈니스 규칙을 Policy로 캡슐화

2 participants