건우님 코드 잘 읽었습니다. view를 통해 입출력을 나누신 부분은 좋아보여요. 여러 기능을 하나의 클래스에서 처리하는 부분은 PlusCalculator의 책임이 과중해 보입니다. SRP를 검색해보는걸 추천드려요. 또한 명확한 코드에 대해선 주석을 굳이 달지 않아도 괜찮아 보입니다. 1주차 수고하셨습니다!
건우님 코드 잘 읽었습니다. view를 통해 입출력을 나누신 부분은 좋아보여요. 여러 기능을 하나의 클래스에서 처리하는 부분은 PlusCalculator의 책임이 과중해 보입니다. SRP를 검색해보는걸 추천드려요. 또한 명확한 코드에 대해선 주석을 굳이 달지 않아도 괜찮아 보입니다. 1주차 수고하셨습니다!
이 부분에 대해 저도 코멘트를 남깁니다!
아마도 MVC v1~v2 정도의 설계를 의도하신 것 같은데 맞으실까요?!
하지만, 제가 아는 MVC 변화에 따르면 조금 더 괜찮은 구조로 개선할 수 있다고 봅니다!
참고가 되었으면 좋겠습니다. :)
트렌드에 맞는 기술력을 적용하신 부분 좋은 것 같습니다!
저는 위에 작성하신 유정님과 조금 다른 의견을 가지고 있습니다!
제가 바라보는 포인트는 아래와 같습니다.
1. 현재 Service에서 의존성을 필요로 할 때, DIP를 지킬 수 있는가?
-> 현재와 같은 구조면 문제 없다고 봅니다.
2. 그럼 의존성 주입을 위한 클래스는 왜 필요한가?
-> 저는 대개, 확장성을 고려하고 도입한다고 생각합니...