좋은 리뷰를 남겨주셔서 감사합니다.
돌아보니 해당 부분은 잘못 작성된 코드인것 같습니다. input을 불변처리 해야 사이드 이펙트가 나타나지 않을것 같네요.
코드를 작성할 당시에는 간단한 계산기 기능을 충족시키는 조건에서 매개변수를 직접 변경하는 것이 그렇게 어색하게 느껴지지 않아서 해당 코드로 작성했었던 것 같아요.
다시 코드를 작성한다면 숫자...
말씀하신대로 Extractor가 FormatValidator의 책임을 갖는게 어색한 것 같네요!
저는 문자열 계산기를 숫자를 추출하는 전처리 과정과 합계를 구하는 계산 과정, 이렇게 2개의 큰 기능으로 나누어 생각해서 Preprocessor와 Calculator 두 객체를 구현했습니다.
이 기준에서 Preprocessor가 Extractor, Fo...
좋은 의견과 리뷰 감사합니다!! 🙇 🙇♂️ 🙇♀️
이 부분은 구현 당시 깊게 생각하지 않고 위처럼 작성을 했다가 미션 끝나고 디스코드에서 의견 나누는걸 보면서 "아 맞다 수학적으로 0은 양수에 포함이 안되는구나"하고 생각을 한 케이스인데요ㅎㅎ
그래서 "왜 내가 큰 고민 없이 0을 양수에 포함 시켰을까?"하고 나중에 추측해보고 다음과 같...