> mvc 패턴을 사용하고 계신데 calculator와 StringParser는 서비스에 더 가깝지 않을까요? 정규 표현식을 잘 사용하고 계시고, 알아보기 쉽게 상수화해서 적확한 이름을 붙이는 센스에 감탄했습니다. 메서드 별로 주석을 다신 것도 좋고요. 자바독 형식을 사용해서 메서드 주석을 달아보시는 것도 괜찮지 않을까요?
제가 MVC 패턴에서 ...
mvc 패턴도 잘 적용하시고 클래스별로 역할과 기능이 적절하게 나누어져 있네요. 구분자와 관련된 상수도 잘 선언하셨고, 에러 메시지를 enum으로 관리하신 것도 앞으로 복잡한 요구사항이 생길 때 확장해서 사용하기 유용한 것 같습니다. 컨트롤러와 서비스의 역할을 명확히 한 점도 좋고 네이밍도 적절하게 하셔서 가독성이 높아지네요. 테스트만 조금 보충하시면...
mvc 패턴도 잘 적요하시고 클래스별로 역할과 기능이 적절하게 나누어져 있네요. 구분자와 관련된 상수도 잘 선언하셨고, 에러 메시지를 enum으로 관리하신 것도 앞으로 복잡한 요구사항이 생길 때 확장해서 사용하기 유용한 것 같습니다. 컨트롤러와 서비스의 역할을 명확히 한 점도 좋고 네이밍도 적절하게 하셔서 가독성이 높아지네요. 테스트만 조금 보충하시면...
mvc 패턴도 잘 적요하시고 클래스별로 역할과 기능이 적절하게 나누어져 있네요. 구분자와 관련된 상수도 잘 선언하셨고, 에러 메시지를 enum으로 관리하신 것도 앞으로 복잡한 요구사항이 생길 때 확장해서 사용하기 유용한 것 같습니다. 컨트롤러와 서비스의 역할을 명확히 한 점도 좋고 네이밍도 적절하게 하셔서 가독성이 높아지네요. 테스트만 조금 보충하시면...
잘 봤습니다. 메서드를 적절하게 작게 나누고 역할과 기능이 잘 드러나는 이름을 지으셨네요. 한 메서드가 역할과 기능이 너무 많으면 이름도 장황해지기 쉬운데 두 마리 토끼를 모두 잡으신 것 같습니다. mvc 패턴도 잘 적용하신 것 같습니다. 남은 프리코스 과제도 화이팅하세요!
잘 봤습니다. 메서드를 적절하게 작게 나누고 역할과 기능이 잘 드러나는 이름을 지으셨네요. 한 메서드가 역할과 기능이 너무 많으면 이름도 장황해지기 쉬운데 두 마리 토끼를 모두 잡으신 것 같습니다. mvc 패턴도 잘 적용하신 것 같습니다. 남은 프리코스 과제도 화이팅하세요!