🚀 2단계 - 문자열 덧셈 계산기

[hyoojuu]

No description provided.

[mskangg]

안녕하세요 효주님! :)
이번 미션 리뷰를 맡은 강민수입니다 🙇

2단계 미션 잘 구현해주셨네요! 👍 💯
소소한 피드백 남겨드렸는데 확인 부탁드립니다! 🙏

[mskangg]

NumberFormatException 은 숫자가 아닌 경우일 때 사용하는 예외입니다 :)
음수는 숫자이기 때문에 올바른 예외는 아닐것 같아요! 🤔

음수를 전달할 경우 RuntimeException 예외가 발생해야 한다. (예 : “-1,2,3”) 이 요구사항을 만족하기 위해 바로 예외를 전파하는 방법은 어떨까요?

Suggested change

	if (parsedValue < 0) {
	throw new NumberFormatException();
	}
	if (parsedValue < 0) {
	throw new RuntimeException();
	}

[hyoojuu]

넵 해당 조건에 적절한 예외인지에 대한 판단을 놓친 것 같습니다. 말씀해주신대로 수정하겠습니다. 감사합니다!

[mskangg]

별도 try/catch 문을 할 필요 없을것 같아요!

NumberFormatException 은 unchecked exception 이기도 하구요!

java.lang.Object
  └── java.lang.Throwable
        └── java.lang.Exception
              └── java.lang.RuntimeException
                    └── java.lang.IllegalArgumentException
                          └── java.lang.NumberFormatException

[mskangg]

아! @hyoojuu 님

문자열 계산기에 숫자 이외의 값 또는 음수를 전달하는 경우 RuntimeException 예외를 throw한다.
이 요구사항을 지키기 위해 catch 하고 다시 부모 예외로 전파하신걸까요? 😅

NumberFormatException 은 Runtime 을 상속하고 있는 예외이기 때문에 다시 예외를 전파할 필요가 없을것 같아보여서 피드백 남겼습니당 😄
이미 알고 계셨던 내용이라면 스윽~ 지나가시면 될것 같아요 🙃

[hyoojuu]

앗 넵,, 상속하고 있는 것은 알고 있었는데
Unchecked Exception은 명시적으로 예외 처리를 하지 않아도 된다는 점에 대한 이해가 부족했던 것 같습니다.
수정하겠습니다! 감사합니다.

[mskangg]

👍

[mskangg]

👍

[mskangg]

이번 미션에서 중요한 기능을 담당하는 split 메서드에서 두가지의 행위를 하고 있는것 같아요!

1. 구분자를 찾기
2. 구분자를 기준으로 분리

split 이라는 메서드는 2번에 해당하는 행위만 존재하게 리팩토링 하는건 어떨까요? 🤔

    private static String[] split(String text, String delimiter) {
        return text.split(delimiter);
    }

위 처럼 리팩토링이 되려면 아래와 같이 메서드로 나누는 방법이 있을것 같아요 :)

1. 구분자를 찾기
2. 구분자로 분할할 text 추출

String delimiter = findDelimiter(text);
String textToSplit = findTextToSplit(text);
sum(toInt(split(textToSplit, delimiter);

[hyoojuu]

넵 말씀해주신대로 1, 2 각각의 행위만 하는 메소드로 분리해보겠습니다!

[mskangg]

피드백반영 잘해주셨네요! 👍
다음 단계 진행시에는 요구사항 진행하면서 체크리스트를 먼저 작성한 뒤 진행해보시는것 추천드립니다!
그럼 다음 단계 진행해주세요 🔥

[mskangg]

💯

[mskangg]

👍