EOF에 대해 알아보고 [인텔리제이 설정에서 자동 개행을 적용](https://hstory0208.tistory.com/entry/IntelliJ-%ED%8C%8C%EC%9D%BC-%EB%81%9D%EC%97%90-%EC%9E%90%EB%8F%99%EC%9C%BC%EB%A1%9C-%EA%B0%9C%ED%96%89-%EC%B6%94%EA%B0%80%ED...
자료형을 그대로 작성하는 것보다는 변수명에 의미가 담겨있으면 더 알아보기 편하겠네요! 👍
```suggestion
/* Controller가 실행 흐름 담당 */
List<Integer> numbers = stringService.parse(inputString);
Integer result = ...
Calculator 클래스가 너무 여러 기능을 하는 것 같습니다!
클래스명만 보았을 때는 어떤 것을 계산해주는 로직이 주요하게 구현되어있어야 할 것 같은데 여기서는 검증 로직과 입출력 로직의 사용 부분도 함께 있어 기능의 의미가 모호해질 수 있을 듯합니다. 각 기능을 분리하여 구현해보시면 좋을 것 같아요!
요 메서드는 `NumberInputUtil` 클래스에 넣어두신 이유가 궁금합니다!
코드는 주관이 너무 다양하지만.. 저는 개인적으로 유효하지 않은 symbol이 있는지 확인하는 메소드라면, 결국 input을 검증하는 역할이라고 생각해서 `InputValidator`에서 수행하면 더 어울리지 않을까 싶었거든요-!
맛집이라는 소개 받고 왔습니다 ㅎㅎ
전체적으로 코드 구조가 굉장히 깔끔해서 보기도 편했습니다!
그리고 저는 `Util` 클래스들에 대해서 인스턴스화를 막는건 생각 못해봤었는데, 동규님 코드 보니 제가 디테일한 부분을 꽤 놓치고 있었단걸 깨닫고 갑니다..!
코멘트들에 대해서는 서슴없이 의견 달아주세요!
맛집이라는 소개 받고 왔습니다 ㅎㅎ
전체적으로 코드 구조가 굉장히 깔끔해서 보기도 편했습니다!
그리고 저는 `Util` 클래스들에 대해서 인스턴스화를 막는건 생각 못해봤었는데, 동규님 코드 보니 제가 디테일한 부분을 꽤 놓치고 있었단걸 깨닫고 갑니다..!
코멘트들에 대해서는 서슴없이 의견 달아주세요!