안녕하세요:)
bowook님께서 말씀하신 방법도 좋은 것 같아요! 필요없는 else문이 사라져서 간결한 코드가 될 것 같아요~
다만, 제가 염려한 부분은 코드의 의미 전달 명확성의 하락 입니다.
`if (matcher.find()) {
return input.substring(matcher.end());
} throw new Il...
코드의 흐름을 이해하는 데 있어,
해당 문자열이 무엇을 의미하는지 확실히 와닿지 않는 부분이 있는 것 같아요! (약간의 매직 넘버랄까,,)
기본 구분자와 커스텀 구분자를 분리하여 각각을 좀 더 명확하게 처리하도록 하는 건 어떨까요?
기본 구분자 같은 경우는 상수로 선언하여 사용할 수도 있을 것 같아요!
코드를 작은 단위로 잘 쪼개신 것 같습니다. 이렇게 세분화하면 확장성과 유지보수 측면에서 유리할 수 있겠지만, 한편으로는 코드가 다소 복잡해질 수 있다는 생각도 듭니다.
코드를 잘게 쪼갤수록 유지보수는 유리하지만, 가독성이 떨어진다는 단점 또한 생기게 됩니다. 저는 아직 이러한 균형 잡기가 어려운데 혹시 이에 대해서 어떤 생각을 가지고 계신지 궁금합니다.
코드를 작은 단위로 잘 쪼개신 것 같습니다. 이렇게 세분화하면 확장성과 유지보수 측면에서 유리할 수 있겠지만, 한편으로는 코드가 다소 복잡해질 수 있다는 생각도 듭니다.
코드를 잘게 쪼갤수록 유지보수는 유리하지만, 가독성이 떨어진다는 단점 또한 생기게 됩니다. 저는 아직 이러한 균형 잡기가 어려운데 혹시 이에 대해서 어떤 생각을 가지고 계신지 궁금합니다.
입력을 안내하는 가이드 메시지의 경우에는 InputView가 담당해도 괜찮을 것 같아 이렇게 진행했습니다!
그렇지만, 피드백 주신것처럼 OutputView가 콘솔에 출력하는 책임을 담당하도록 `printMessage()` 같은
메서드를 만들어서 사용하는 것도 괜찮을 것 같아요!
감사합니다 👍
생각을 잘 정리해주신거 같아서 밑에 남깁니다.
이와 같이 간단한 도메인 에서는 getter 를 써도 문제가 없습니다.
하지만 getter 를 사용하면, 코드의 중복이 발생하게 됩니다.
예시로, 여기서 더하기 말고, 빼기를 하는 기능 요구사항이 추가되면 어떻게 될까요?
더하기 하는 부분도 구분자를 `getDelimiters` 로 받고, 빼...