get을 통한 객체 내부의 컬렉션을 외부에 넘기는 것이 좋지 않다고 이해하고 있어서 그렇게 말한 것 같습니다. 저도 안좋다 이정도만 알아 공부를 더 하고 이번 주차에 최대한 적용시킬려 노력하고 있습니다. 첫번 째 링크의 규칙 8, 9와 두번 째 링크의 일급컬렉션을 참고하면 좋을 것 같습니다!
https://velog.io/@inwoo920/%EA...
해당 부분은 if - else를 이용하지 않아도 될 것 같다는 생각을 했습니다.
왜냐하면, if 조건을 통해 return으로 반환을 하고,
if 조건이 거짓이어도 else 문으로 들어가 return 반환하기 때문입니다.
아래처럼 수정해 보시면 더 직관적일 것이라고 생각합니다!
`if (doubleNum % 1 != 0){
...
Input에서는 사용자의 입력만을 받아 반환하는 것이 아닌 비교 후 반환하시는데 이유가 있으신가요?
제 생각에는 Input에서는 사용자의 입력만을 받아서 반환하는 역할을 수행하는게 더 좋다고 판단했습니다.
이유는, Input에서 검증까지 하는 역할을 수행하면 많은 책임이 있는 느낌입니다!
해당 함수의 로직을 보면 음수와 0이 하나라도 **있다면** 예외를 던지고 있습니다.
굳이 들어온 음수와 0을 List로 가지고 있을 필요가 없을 것 같은데, 어떻게 생각하시나요?
또한, "0과 음수를 저장할 리스트"와 같이 더 명확히 주석 처리를 해주신다면 좋을 것 같습니다:)