클래스의 역할이 너무 많습니다. 위에서 말씀드린 것처럼 view 클래스를 따로 만드시는 편이 좋을 것 같습니다. veiw -> controller -> service -> controller -> view의 흐름을 따르면 클래스의 역할도 작게 유지하고 분명해질 것입니다.
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠?
controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요.
네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠?
controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요.
네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
main메서드에 너무 많은 책임이 있는것 처럼 보여서(사용자 입력 처리, 문자열 분할, 구분자 추출, 덧셈 계산이 모두 main에 들어감) 컨트롤러 클래스를 하나 만들어서 분리해보는건 어떨까요?
아니면 간단한 계산기의 기능을 구현하는 과제였으므로 빠르고 간단하게 기능을 구현하는것이 더 중요하게 판단되었을까요?
정민님이 작성하신 계산기 메서드에서는 double 타입을 지원하는 계산기 로직으로 작성하셨는데,
해당 메서드에서 커스텀 구분자로 온점이 들어올경우 소수와 구분자를 제대로 구별하지 못할거 같아서 이를 어떤식으로 처리하는지 고민해보는것도 좋을것 같아요
저는 이러한 문제때문에 들어오는 수의 타입을 int타입으로 제한하여 구현했었던것 같아요. 만약 소수...
1주차 미션 하시느라 고생 많으셨습니다 !!! 👍
이번 코드를 보면서 구분자를 Enum으로 관리하는 아이디어는 한 수 배웟습니다 ㅎㅎ 😄
학습하시면 좋을 것 같은 제안 두가지 하고 가겠습니다 !!
1. Enum 활용법
2. 테스트 코드 API
에 대해서 학습하시면 더욱 좋은 개발자로 성장하실 것 같아요 !!
구분자가 될 수 없는 Delimiter를 enum으로 생성하는 아이디어는 가독성 면에서 엄청 좋네요 !!! 👍
InputValidator를 따로 분리하신 이유가 궁금합니다.
제 생각엔 Delimiter에서 이 데이터 값들을 꺼내오는 방식이 아닌 구분자 데이터를 Delimiter에게 넘겨줘서 이 안에서 처리하는 로직을 구현했을 것 같습니다 ...
1주차 미션 하시느라 고생 많으셨습니다 !!! 👍
이번 코드를 보면서 Basic을 Enum으로 관리하고 List에 구분자들을 저장하는 아이디어는 한 수 배웟습니다 ㅎㅎ 😄
학습하시면 좋을 것 같은 제안 두가지 하고 가겠습니다 !!
1. Enum 활용법
2. 테스트 코드 API
에 대해서 학습하시면 더욱 좋은 개발자로 성장하실 것...
1주차 미션 하시느라 고생 많으셨습니다 !!! 👍
이번 코드를 보면서 Basic을 Enum으로 관리하고 List에 구분자들을 저장하는 아이디어는 한 수 배웟습니다 ㅎㅎ 😄
학습하시면 좋을 것 같은 제안 두가지 하고 가겠습니다 !!
1. Enum 활용법
2. 테스트 코드 API
에 대해서 학습하시면 더욱 좋은 개발자로 성장하실 것...