도메인 객체안은 해당 도메인만 관련된 비즈니스 로직들만을 작성하고, 서비스 객체는 도메인들의 로직을 조합하여 처리하게 하여 비즈니스 프로세스의 흐름을 관리 하게 하여 역할을 분리 했습니다! 이렇게 하면 만약 요구사항이 변경돼 Calculator 도메인의 비즈니스 로직에 다른 도메인 객체의 비즈니스 로직을 가져와 사용해 여러 도메인이 섞이게 되면 각 도...
MVC 변화에 관심이 많으신 코드라고 느껴지네요!!
실제로 MVC 패턴은 점차 변화되면서 Controller가 많아지는 문제를 MainController에서 처리했죠!
하지만, 현재 필요한 Controller가 그렇게 많지 않아보입니다!!
CalculatorController로 모두 처리해 볼 수 있지 않았을까요?!
하지만 학습한 기술을...
상수는 어떤 역할을 가지고 있다고 생각하시는지 여쭤봐도 될까요?
저와의 의견 차이가 있는 부분이 있습니다!
저는 메세지 그 자체에서 다 표현할 수 있지 않을까?
literal pool의 특성을 잘 사용하고 있는 코드인가?
라는 생각이 듭니다!!
한 번 참고해보시고 지정님의 의견을 들려주시면 감사할 것 같아요!!
현재 3개의 validate에서 BigInteger 객체를 생성하면서 검증하고 있습니다.
그럼 실제 검증이 끝나고 BigInteger 객체를 또 생성할 것으로 보이는데요.
한 번의 숫자 변환을 위해서 4번의 객체 생성이 필요한가?
1번의 객체 생성으로 개선할 수 있을 것으로 보입니다!
단위테스트도 진행하시고 인수 테스트도 진행하신 부분이 굉장히 인상 깊습니다!!
단순히 기능 구현 뿐만 아니고 전체 로직에 대한 꼼꼼한 테스트는 좋은 습관이라고 생각합니다.
특히 좋은 기술로 소개되어지는 부분들 또한 접목하려고 하신 것 같더라구요!
다만, 아쉬운 점은 코드에서 무결성을 지키지 않는 부분이 있었다는 것입니다.
그 부분에 대해서는...
단위테스트도 진행하시고 인수 테스트도 진행하신 부분이 굉장히 인상 깊습니다!!
단순히 기능 구현 뿐만 아니고 전체 로직에 대한 꼼꼼한 테스트는 좋은 습관이라고 생각합니다.
특히 좋은 기술로 소개되어지는 부분들 또한 접목하려고 하신 것 같더라구요!
다만, 아쉬운 점은 코드에서 무결성을 지키지 않는 부분이 있었다는 것입니다.
그 부분에 대해서는...
@soeunnPark
바로 에러를 던지게 된다면, 해당 메서드는 조건을 `확인`하는 책임과 그 조건에 따라 `예외를 발생`하는 책임을 모두 가지게 될 것 같아요. `isNotPositive( )`가 내부 검증에서만 사용되긴 하지만 저는 가독성을 위한 분리는 좋은 코드에 가깝지 않을까 싶은데, 이에 대해서는 어떻게 생각하시는지 궁금합니다!🙂