[step4] 🚀 4단계 - 자동차 경주(우승자)

[shgpwn]

최초 구현

[wheejuni]

전반적으로 큰 고민이 없이 미션 진행 및 코드 작성이 이뤄진 것 같아, 리뷰어로서 아쉬움이 진하게 남는 PR입니다.
물론 생소한 과제를 접하고 답을 찾아가는 과정이 늘 즐거울 순 없겠지만, 조금 더 치열한 고민과, 깊이있는 설계를 기대합니다. 그동안 해오셨던 방식으로 똑같이 코드를 작성하실 것이라면, 우리 과정에 참여하시는 의미를 살리기 어렵다고 생각합니다.
제가 요런 생각을 갖게된 근거들을 하나씩 살펴 드리겠습니다.

1. 테스트가 너무 빈약합니다. 과한 테스트도 문제지만, 전체적인 코드 변경에 비해 테스트 코드가 너무 적게 추가되었습니다.
2. for loop 사용 등 새로운 시도보다는 기존에 프로그래밍을 진행하셨던 방법대로 진행하신 것 같습니다.
3. 강의 자료나 라이브 강의 등을 통해서 개선 포인트라고 전달되었던 부분들이 반영되지 않았습니다.
4. 접근 제어자 누락 등, 기본적으로 지켜주시면 좋을 법한 내용들을 챙겨주지 않으셨습니다.

따라서 아래와 같은 액션 포인트 잡아 드리겠습니다.

1. for loop을 통해 객체 상태 변경하는 로직은 모두 stream API의 사용으로 개선해 보세요.
2. 접근 제어자가 누락된 부분에 대해선 package-private을 의도하시는 게 아니라면 모두 접근 제어자를 붙여 주시기 바랍니다.
3. 하나의 클래스에 대해 하나의 테스트 클래스를 무조건 두고, 테스트 케이스 최대한 도출해 주시면 좋겠습니다.

조금 더 나은 코드를 기대해 보겠습니다. 감사합니다 👍

[wheejuni]

요기에도 접근 제어자가 붙어야 할 것 같습니다. 하나 붙여주시죠!

[wheejuni]

1. 접근 제어자 누락되면 안됩니다. 꼭 챙겨주세요
2. 지금 컨트롤러에서 다 관리하고 있는 요 변수들을 챙겨줄 RacingGame 이라는 클래스를 설계하고 작성해보세요. 이 클래스의 기능은 다음과 같습니다.

• Inputview 로부터 플레이어 이름 입력받음
• 몇 번 돌릴건지 회차 입력받음
• 실제 게임 수행
• 회차별 결과 관리

[wheejuni]

stream API 로 개선할 수 있습니다. 개선 시도해보세요.

[wheejuni]

테스트의 구성에 있어서 몇 가지 클래스들에 대한 테스트가 한 테스트 클래스 파일로 합쳐져 있는데요, 요건 일단...

1. 테스트 먼저 작성하시고 구현하는 흐름을 가져가고 계시지 않다는 걸 알 수 있습니다.
2. 이 자체로 코드 스멜입니다. 테스트는 메소드 단위 실행이 아닌 클래스 단위 실행이면 좋겠고, 각 클래스가 의미단위를 가졌으면 합니다.

[wheejuni]

외부에서 주입받는 방식이면 더욱 좋을 듯합니다.

[shgpwn]

@wheejuni 리뷰어님 외부에서 주입받는 방식이 무슨 의미일까요??
InputView에서는 입력값만 받고, split 같은 입력 값에 대한 로직은 외북에서 처리하는 방식을 말씀하신 걸까요??
제가 이해한 게 맞는지 궁금하여 확인차 여쭈어 봅니다^^