헥사고날/클린 아키텍처 다시 공부하고 패키지 다시 분리

[jxmen]

현재 코드 구조에서는 용어를 잘못 알거나 오남용된 경우가 있는 것 같다.

• [x] SubjectService에서 ChatUseCase 의존하는 부분 AddChatPort로 변경 (ISP 원칙 준수)
• [x] README.md 디렉토리 구조 수정
• [x] Persistence Adapater 패키지 이동
• [x] Repository 추상화 제거
• [x] JpaEntity와 도메인 분리 - Mapper

Related

• https://github.com/jxmen/cs-ai-interviewer/issues/56

[jxmen]

108 에서 너무 이른 추상화라 판단되는 몇몇 부분은 Port 인터페이스를 사용하지 않고 실제 구현체를 의존하는 방식으로 사용하고 있다. (제미니님 코드 작성 방식과 유사)

[jxmen]

만들면서 배우는 클린 아키텍처 책이 좀 모호하다. adapter/in 같은 경우도 presentation이라는 명칭이 더 직관적인 것 같다..

[jxmen]

리뷰 내용

• repository에 의존하는 클래스들은 persistence adapater이다. 예를 들어 chatAppdner
• chatAppender 네이밍이 좀 그렇다?
• repository도 도메인에 있으면 안됨
  • (db entity에 대한 repository이지, 도메인 자체에 대한 repository는 아니기 때문)
• 원래 클린 아키텍처에서는 execute 메서드 하나만 가진다. 괜히 많이 쪼개는게 아님
• ReactiveMemberChatUseCase는 꼭 빼야할까? 테스트 때문에 실제 구현체가 변경되는 것이 맞을지는 고민 필요
  • 시나리오 테스트를 원래 잘게 쪼갰다면, mocking하는 코드가 길어지더라도 해당 테스트 자체에서만 mocking하는 로직이 들어갔을 것
• Query는 비즈니스 로직과 관련된 것이라면 usecase로 만들어야 한다.

[jxmen]

138