> CalculateController에서 메서드 분리가 조금 과한 것 같다는 생각이 듭니다 `calculateSum`과 `displayResult` 메서드를 한번 더 분리해서, run 메서드만 봤을 때는 이해하기가 좋지만 협업하는 관점에서, 두 메서드는 다른 메서드를 한번 호출하는 역할만 수행하는데 로직을 깊게 이해하려면 한번 더 메서드를 타...
입력 문자에 따라, 입력 문자의 문자셋에 따라, 한 문자를 표현할 수 있는 길이가 달라질 수 있다는 사실 알고 계셨나용? 🤗
입력 문자가 아주 제한되어 있는 것이 아니라면 문자 위치를 추측하는 것(만약 커스텀 구분자가 있으면 몇번째 자리에는 뭐가 있을거야)보다
헤더와 푸터의 위치를 찾고 그 사이의 문자를 찾아내는 것이 정확해요! 물론 길이체크를 따...
아래의 @dradnats1012 님 의견과 비슷한 코멘트인데요.
메서드 시그니처(`리턴값 메서드명(파라미터)`)가 좀 더 많은 정보를 주면 좋을 것 같아요.
반환받는 값이 `int output`인데 `int`는 원시타입이라 어떤 용도인지 추측이 어렵구(객체타입이면 의도를 줄 수 있겠죠?),
`output`은 무엇의 output인지 추측이 어려워...
이 부분은 저도 고민이 들었던 부분입니다..
validate 라는 메서드가 형변환한 값을 return 해버리면
검증과 형변환이라는 두가지의 책임을 지게 된다고 생각하여
두 가지 메서드로 나누어 parseInt() 메서드가 두 번 호출되는 로직이 생겼습니다
결국은 하나의 메서드가 하나의 책임을 지도록 하기 위해 이처럼 구현했습니다!
중복...