[클린코드 7기 원건우] 자동차 경주 미션 STEP 4

[rjsdnql123]

안녕하세요 리뷰어님!!
늦은 시간 알람 갈까 걱정 되네요..
리뷰 해주신 부분 고민하여 수정 하였습니다!

PR 변경 사항

요구사항에 맞춰 아래 기능을 추가 하고 수정 하였습니다.

• 전진하는 조건은 0에서 9 사이에서 무작위 값을 구한 후 무작위 값이 4 이상일 경우이다.
• 자동차 경주 게임을 완료한 후 누가 우승했는지를 알려준다. 우승자는 한 명 이상일 수 있다.
• 우승자가 여러 명일 경우 쉼표(,)를 이용하여 구분한다.
• 유저가 직접 라운드 수를 입력할 수 있다.

고민한 부분

이전 PR 과 같이 하나의 메서드가 하나의 책임을 지는 단일 책임 원칙을 고려하여 작성 하였습니다.

전 PR에서 리뷰 남겨주신거 저도 코멘트 남겼는데 궁금한거 있습니다!

• 에러 메세지 관련해서 리뷰 해주셨는데, Car Class 에서 사용되는 에러 메세지가 View 로 빠지는게 코드 구조상 더 맞는지 궁금합니다!
  • 전 Car Class 에서만 사용하는 에러 메세지니까 Car Class 에 static 으로 있는 구조가 괜찮다고 생각 했었습니다!
• 광열님께서는 Class 를 작성하실 때 static 값은 어떤 때 사용하시나요??
  • 코멘트에 남겼는데 전 래스의 상태를 변경하지 않으며, 해당 클래스의 고유의 기능, 값인가? 이러한 조건을 갖고 static 을 사용하고 있습니다!
• 광열님 께서는 로직을 작성하기 전에 테스트 코드를 먼저 작성하고 다음 코드를 작성 하시는 편이신가요??
• 실제 개발하실 때 테스트 커버리지를 어느정도 갖고 가시는지도 궁금합니다!

추가적으로..

좀 이상하다고 생각 드는 부분이나 개선 해야 할 부분 다 말씀 주시면 Step5에 모두 수정 하겠습니다.
감사합니다 : )