CLAP-213 작업 도메인 레이어의 비즈니스 규칙과 정책을 분리

[joowojr]

📄 요약(Summary)

✍🏼 상세(More)

PR Desciption

변경 사항 설명

작업(Task) 도메인 레이어의 비즈니스 규칙과 정책을 분리: Policy

• 작업 도메인 모델 클래스가 너무 많고 복잡한 비즈니스 로직을 담당하고 있어 정책을 담은 class로 분리하였습니다. (특히 작업 보드 순서를 변경하는 서비스)
• Policy는 도메인 모델 클래스에 너무 많은 책임을 주는 복잡한 도메인 규칙, 재사용 될 수 있는 로직, 요구사항에 따라 변경될 수 있는 로직 등 정책적인 부분을 캡슐화합니다.
• 핵심 비즈니스 로직과 정책을 분리하였기 때문에 각각 독립적으로 단위 테스트가 가능하며, 비즈니스 로직에 수정이 필요해도 변경사항이 크지 않습니다.
• CLAP-213 작업 도메인 비즈니스 규칙과 정책 분리 #223 커밋이 꼬여서 이 PR을 봐주시면 더 편할것 같습니다!

작업 보드 조회 post -> get mapping으로 수정

• @RequestBody 대신 @ModelAttribute 사용

댓글 첨부파일 업로드 기능
- 단일 파일 업로드로 수정: 단일 파일인데 파일 배열 업로드 메서드를 통해 업로드 되고 있었음

#105 리팩토링

• member adapter 클래스에 선언되어 있던 query dsl 메서드 분리 -> custom repository로 구현
• 잘못된 패키지 위치 수정 management -> admin

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

• request body를 보낼 때에는 조회 메서드라도 post 매핑만 사용가능하다고 알고 있었는데, @RequestBody 대신 @ModelAttribute를 사용하면 get mapping으로 request body dto를 보낼 수 있음을 알게 되었습니다.

고민 중인 사항

알림 전송 시 메세지 생성 로직 부분 캡슐화

• notification type에 따라 policy또는 util로 message 템플릿을 생성하는 클래스가 필요해보입니다.

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #215