이 부분에 대해 저도 코멘트를 남깁니다!
아마도 MVC v1~v2 정도의 설계를 의도하신 것 같은데 맞으실까요?!
하지만, 제가 아는 MVC 변화에 따르면 조금 더 괜찮은 구조로 개선할 수 있다고 봅니다!
참고가 되었으면 좋겠습니다. :)
트렌드에 맞는 기술력을 적용하신 부분 좋은 것 같습니다!
저는 위에 작성하신 유정님과 조금 다른 의견을 가지고 있습니다!
제가 바라보는 포인트는 아래와 같습니다.
1. 현재 Service에서 의존성을 필요로 할 때, DIP를 지킬 수 있는가?
-> 현재와 같은 구조면 문제 없다고 봅니다.
2. 그럼 의존성 주입을 위한 클래스는 왜 필요한가?
-> 저는 대개, 확장성을 고려하고 도입한다고 생각합니...
전체적으로 고급진 내용이네요.
설계에 대해 스스로 많은 피드백을 거친 흔적이 보입니다.
다만 늦게 리뷰에 참여하는 만큼 개인적으로 아쉬운 부분이 있습니다.
몇 몇 분께서 인터페이스 도입에 대해 의문을 가지고 계십니다.
저도 해당 부분에 대해서 의문점이 들게 되었는데요..
개인적으로 만약, 테크 리더나 경험이 많은 개발자가 도입한게 아니...
전체적으로 고급진 내용이네요.
설계에 대해 스스로 많은 피드백을 거친 흔적이 보입니다.
다만 늦게 리뷰에 참여하는 만큼 개인적으로 아쉬운 부분이 있습니다.
몇 몇 분께서 인터페이스 도입에 대해 의문을 가지고 계십니다.
저도 해당 부분에 대해서 의문점이 들게 되었는데요..
개인적으로 만약, 테크 리더나 경험이 많은 개발자가 도입한게 아니...
customSeparatorHandler로 custom separator를 얻고 그것을 Separator 객체에 추가하는 로직입니다.
메소드명 때문에 혼돈될 수 도 있겠네요
그리고 정환님 덕분에 Separator가 VO인지 엔티티인지 일급컬렉션인지 생각해볼 수 있었어요. 이번에는 구분하지 않고 객체를 만들었지만, 2주차 과제에서는 설계전 한...
정적 팩토리 메서드를 통해 객체를 생성하는 목적을 명확하게 드러낼 수 있다고 생각합니다!
기본 구분자를 만드는 것과, 커스텀 구분자를 포함해 만드는 것이 각각 객체를 생성하는 목적과 의도가 다르기 때문에 이를 정적 팩토리 메서드를 통해 분리하는 것이 좋다고 생각합니다.
다만, 말씀해주신대로 커스텀 구분자가 기본 구분자를 포함하기 때문에 현재 로직...