[🚀 사이클2 - 미션 (기물 확장 + DB 적용)] 타스 미션 제출합니다.

[ghkdtjfxkd]

안녕하세요 두둠, 먼저 사과의 말씀 드리겠습니다.
사이클 2에서 궁성과 DB를 도입하는 과정에서 공부를 병행하며 진행하느라 PR이 많이 늦어졌습니다. 죄송합니다.🥲

---

궁성

궁성 내부 움직임 규칙을 추가하는 과정에서 최소한의 변경을 위해
Position, MoveStrategy 구현체 그리고 moveRule들의 내부를 수정하는 방식으로 추가적인 이동을 추가했습니다.

DB

커넥션 관리를 위해서 HikariCP를 도입했습니다.
또한, 트랜잭션 내 커넥션 전파를 위해 ScopedValue를 도입했습니다.
파라미터 전달 방식은 계층 간 결합도가 높아지고, ThreadLocal은 누수 위험과 가변성 문제가 있어
불변성과 자동 소멸이 보장되는 ScopedValue를 선택했습니다.

체크 리스트

• 미션의 필수 요구사항을 모두 구현했나요?
• Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
• 애플리케이션이 정상적으로 실행되나요?

어떤 부분에 집중하여 리뷰해야 할까요?

계층의 책임

오래 전 기억을 더듬어가며 영속성 계층을 도입하는데 많은 어려움을 겪었습니다.
특히, Repository와 DAO, Service의 책임의 경계를 구분하기 어려웠는데요.

DB에 실제 저장하는 로직, SQL을 만들어내는 책임과 DB 데이터를 기반으로 객체를 만들어내는 책임은 각각 어디가 맡아야할지 헷갈리더라고요...

1. 저는 DB와 실제로 소통하는 건 DAO, DAO와 소통하여 객체를 만들어내는 역할은 Repository, 만들어진 객체로 비즈니스 로직을 수행하는건 Service의 책임이라 생각하고 만들었는데요, 계층의 책임이 적절한지 여쭙고 싶습니다.

2. DB를 도입하면서 DAO에서 예외 처리에 대한 고민이 있습니다.
   이어할 게임 ID를 입력받아 게임을 이어할 수 있었는데요, 만약 존재하지 않는 게임id를 입력하는 경우 DAO에서 예외가 발생했습니다.
   도메인 객체가 아닌 곳에서 예외가 터져서 어떻게 처리할지 난감했습니다.

   • DAO에서 예외를 발생시킨다?
   • Optional로 받아서 Repository에서 예외를 발생시킨다?
   • Optional로 받아서 Service까지 전달 후 예외를 발생시킨다?

저는 DAO에서 예외를 발생시키는 방식을 택했는데요, JanggiDataException이라는 커스텀 예외를 발생시키는데
도메인의 내용을 아는 예외를 발생시키는게 괜찮을지도 많은 고민이 되더라고요. 🤔

도메인 설계

비대해진 Position

궁성이 도입되면서 이동 규칙 추가를 위해 Position 내부에 enum을 두어 궁성과 관련된 정보를 정의해 두었는데요.
여러군데서 VO로 사용되는 Position의 코드 라인이 너무 길어지고, 과한 책임을 가지게 되는 것은 아닐까 고민되었습니다.

그렇다고 별도로 궁성을 빼자니 결합도가 증가하고 응집도가 낮아질 수 있겠다는 생각이 들었습니다.
궁성의 위치에 대한 책임을 Position이 가지는 방식에 대한 의견을 여쭙고 싶습니다.