🚀 2단계 - 수강신청(도메인 모델)

[Eundms]

• Session 내 인스턴스 변수를 줄이다가 더 복잡해지는 것 같아 일단 리뷰 요청 드립니다.
  • 더 줄일 수 있는 방안을 잘 모르겠습니다 ㅠ

[javajigi]

전체적인 객체 설계, 단위 테스트 코드 구현 👍
도메인 객체도 작은 단위로 분리하고, Session의 복잡도를 낮추기 위해 노력하다보니 복잡도가 낮게 잘 구현했네요. 👍
피드백할 부분이 많지는 않았지만 한번 고민해 봤으면 하는 바람으로 피드백 남겨봅니다.
고민해보고 반영할 부분 반영하고 빠르게 다음 단계 넘어가시죠.

[javajigi]

수강 신청과 관련해 이 두 객체(state, type)가 관여하고 있는데 이 둘을 Enrollment와 같은 객체로 분리해 보는 것은 어떨까?
아니면 수강 신청과 관련한 모든 값을 별도의 객체로 분리해 구현해 보는 것은 어떨까?

[javajigi]

Session 객체와 같이 인스턴스 변수가 많은 객체를 테스트하려면 객체를 생성하는데 어려움이 있다.
중복 코드 또한 많이 발생해 Session을 생성할 때 생성자의 인자가 변경되는 경우 변경할 부분이 많아진다.
https://www.arhohuttunen.com/test-data-builders/ 문서 참고해 Session과 같이 복잡한 객체의 테스트 데이터를 생성할 때 어떤 방법을 사용할 것인지 선택해 보면 좋겠다.
이번 기회에 내가 선호하는 방법을 적용해 보고 앞으로도 쭈욱 활용하는 방식이면 좋겠다.