> 여기에 모아두지 않은 상수들은 위치에 있는 클래스 내에 있을 때 더 확인하기 쉽고 응집성이 높다고 생각해서 옮기지 않았습니다!
> `DelimiterConstants` 내에 있는 상수들은 여러 클래스에서 사용되기도 하고, 구분자가 변경될 가능성을 생각하면 모아두는 것이 더 좋을 것 같아서 이렇게 하였습니댜
저도 이와 같은 생각으로 여러 클래...
> 코드와는 상관없지만 개인적으로 궁금한 점은, View의 역할은 어디까지라고 생각하시나요?
>
> 실제 계산기에서는 출력을 할 때 3자리 단위로 끊어서 보여줘요 ex) 100,000,000
>
> 그렇다면, View에서 3자리씩 끊어주는 함수를 만들어도 될까요? 아니면 다른 곳에서 분리를 하고, String을 출력해야하는걸까요?
저는...
저도 이번에 전역적으로 Validator 처리를 해줬었는데요!
리뷰 중에, “해당 도메인에 맞는 에러를 전역적으로 처리하는 대신, 각 도메인에서 구체적으로 처리하는 것에 대해 어떻게 생각하세요?“라는 질문을 보았습니다.
처음에는 비즈니스 로직을 처리하면서 유효성 검증까지 맡으면 과도한 책임을 갖는 게 아닐까 하는 생각이 들었지만, 도메인별 에러...
반복문으로 numbers 리스트의 요소의 개수만 큼 validatePositivity를 호출하고 있는 것 같아요!
이 부분에 대해서 궁금한 부분이 리스트 형태로 인자로 넘기고 validatePositivity 한 번만 호출해, 그 안에서 for문을 돌려 요소를 확인하는 방법에 대해서는 어떻게 생각하시나요?
DI 객체들을 다른 테스트 클래스에서 단위 테스트를 하고 있긴한데, 지환님이 말씀하시는 단위 테스트는 어떤 의미 인가요??
순수 기능 테스트를 위해서는 모킹이 필요하다는건 백퍼 공감해요 ㅎㅎ Numbers가 간단한 getter 기능 밖에 없어서 모킹하지 않았는데, 요게 쫌 아쉬운 포인트이긴 하네요 ~ ㅎㅎ
좋은 피드백 정말 감사해요~ ☺️