Conversation
| import java.util.regex.Matcher; | ||
| import java.util.regex.Pattern; | ||
|
|
||
| public class StringCalculator { |
There was a problem hiding this comment.
현재 문자열 계산기는 수의 유효성 검증, 값 더하기, 문자열 분리 및 변환 등 많은 책임을 가지고 있어요! 한 객체가 하나의 책임을 가질 수 있도록 책임을 분리해보는 게 어떨까요? 이와 관련해서 객체지향 원칙 중 단일 책임 원칙을 참조해보면 좋을 것 같아요!
예를들어 스스로 양수임을 보장하는 객체가 있다면? 문자열 계산기는 해당 객체에게 그 책임을 위임하고 협력한다면, 문자열 숫자 변환, 음수 검증을 직접 책임지지 않아도 될거예요! 관련해서 참고해보면 더 좋을만한 아티클이예요😊
https://tecoble.techcourse.co.kr/post/2020-05-29-wrap-primitive-type/
https://jojoldu.tistory.com/412
| public String[] initSeparator(String input) { | ||
| Matcher matcher = Pattern.compile(DELIMITER_REGEX).matcher(input); | ||
|
|
||
| String customSeperator = DEFAULT_SEPERATORS; | ||
| String inputString = input; | ||
|
|
||
| if (matcher.find()) { | ||
| customSeperator = matcher.group(1); | ||
| inputString = matcher.group(2); | ||
| } | ||
|
|
||
| return new String[]{customSeperator, inputString}; | ||
| } |
There was a problem hiding this comment.
다른 객체가 initSeparator 메소드와 협력한다면,
String[] 반환값에서 특정 인덱스가 무엇을 의미하는지 구체적으로 아는 과정이 동반되고, 이를 기억해야한다고 생각해요!
만약 String[] 반환값의 순서가 바뀐다면 이를 사용하던 클라이언트들은 어떻게 될까요?
캡슐화에 대해 더 알아보시면 좋을 것 같아요!
https://inpa.tistory.com/entry/OOP-%EC%BA%A1%EC%8A%90%ED%99%94Encapsulation-%EC%A0%95%EB%B3%B4-%EC%9D%80%EB%8B%89%EC%9D%98-%EC%99%84%EB%B2%BD-%EC%9D%B4%ED%95%B4
| } catch (Exception e) { | ||
| throw new RuntimeException(); | ||
| } |
There was a problem hiding this comment.
Exception은 checked exception과 unchecked exception 모두를 포함해요!
모든 예외를 하나로 처리한다면,특정 예외에 대해 적절하게 대응하기 어려울 거예요. 조금 더 구체적으로 예외를 작성해보면 어떨까요?
그리고 Exception e를, new RuntimeException()에 함께 전달하지 않으면 원래 예외의 정보가 사라지게 돼요. 🥲 디버깅 시 어떤 예외가 발생했는지 알 수 없어져서 원인을 찾기가 어려워지는데, 이 부분도 함께 확인해보시면 좋을 것 같아요!
관련해서, Checked Exception, Unchecked Exeption, 예외 전달과 관련하여 더 알아보시면 좋을 것 같아요!
| /** | ||
| * <요구사항> | ||
| * 1. 구분자를 가지는 문자열을 전달 받는다. | ||
| * 2. 기본 구분자([",", ":"])를 기준으로 분리한 각 숫자의 합을 반환한다. | ||
| * 3. 기본 구분자 외에 커스텀 구분자를 지정할 수 있다. 커스텀 구분자는 "//"와 "\n"사이에 위치하는 '문자'를 커스텀 구분자로 사용한다. | ||
| * 4. 문자열 계산기에 숫자 이외의 값 또는 음수를 전달하는 경우. RuntimeException 예외를 throw 한다. | ||
| * <p> | ||
| * <요구사항 - 정제> | ||
| * 1.문자열을 받는다. | ||
| * 2.문자열은 구분자와 숫자로 구분된다. | ||
| * 3.구분자는 "기본 구분자"와 "커스텀 구분자" 2가지로 구성된다. | ||
| * 3-1. "기본 구분자" [",", ":"] | ||
| * 3-2. "커스텀 구분자" "//"(문자)"/n" | ||
| * 4. 숫자는 0을 포함하는 자연수 | ||
| * 5. 4이외의 수가 전달되는 경우 RuntimeException 예외를 throw한다. | ||
| */ |
| @ParameterizedTest | ||
| @ValueSource(strings = {"1"}) | ||
| void oneNumber(final String text) { | ||
| assertThat(calculator.add(text)).isSameAs(Integer.parseInt(text)); |
There was a problem hiding this comment.
검증부를 하드코딩해보면 어떨까요?😃
| @DisplayName(value = "//와 \\n 문자 사이에 커스텀 구분자를 지정할 수 있다.") | ||
| @ParameterizedTest | ||
| @ValueSource(strings = {"//;\n1;2;3"}) | ||
| void findCustomDelimiter(final String text) { | ||
| String[] divided = calculator.initSeparator(text); | ||
| assertThat(divided[0]).isEqualTo(";"); | ||
| } |
There was a problem hiding this comment.
커스텀 구분자를 지정할 수 있는지에 대한 테스트는
66번줄 테스트에서 진행된 것 같은데 구분자에 대해 별도로 테스트하신 의도 설명해주실 수 있을까요?? DM으로도 좋아요!
2 participants
요청사항
안녕하세요! 리뷰어님. 우선 요구사항을 충족하는 선에서 구현을 마무리하고 피드백을 요청드립니다.
경험이 부족한 상태여서 잦은 요청을 드리겠습니다. 🥹
전달사항
별도로 제 기술적 배경을 설명드리면 피드백에 더 효과적이지 않을까 생각되어 전달드립니다.
앞으로 잘 부탁드립니다. 🙏