🚀 3단계 - 자동차 경주

[ajy9844]

안녕하세요 영직 리뷰어님!
3단계 자동차 경주 미션 리뷰 요청 드립니다. (많이 늦어져 죄송합니다😭)

코드를 작성하면서 생긴 질문은 코멘트로 달아두겠습니다!
이번에도 고치면 좋을 부분에 대해 많이 알려주시면 감사하겠습니다.
시간 내어 리뷰해 주셔서 감사합니다!

[laco-dev]

자동차 경주 미션 잘 해주셨습니다.
지난 미션까지 개선한 내용도 같이 반영을 해주셨네요. 👍🏻

코틀린을 배워가는 과정에서 필요한 몇가지 내용과 함께 고민해볼만한 점을 같이 남겨 보았습니다.

[laco-dev]

차량의 초기 위치에 대한 테스트도 작성 해 보세요.
이 경우 생성자를 통해 차량의 초기 위치를 다음과 같이 하면 테스트를 보다 효율적으로 작성할 수 있습니다.

class Car(initialPosition: Int = DEFAULT_POSITION) {
   var position: Int = initialPosition
     private set
}

[laco-dev]

제공하는 함수가 정적 메소드밖에 없는 경우에는 class 대신 object로 만들어서 사용할 수 있습니다.

[laco-dev]

view()가 무슨 값을 반환하는지 알 수 있을까요?

사용하는 곳의 코드를 줄이려고 하는 것 보다 오히려 가독성이 떨어질 수 있습니다.
명확하게 함수를 분리해서 받아보면 어떨까요?

InputView.getNumberOfCars()
InputView.getTryCount()
...

[laco-dev]

"실행 결과"는 ResultView에서 출력해야 하지 않을까요?

[laco-dev]

지연초기화를 이용하는 것이 어떤 장점이 있을까요?
RacingGame의 상태가 계속해서 변경이되면서 재활용 될 경우는 없다고 생각이 듭니다.

한번의 경주에서 한번의 결과를 만들어 내도록 하면 어떨까요?

object RacingGame {
   fun race(cars: List<Car>, times: Int, ...) { ... }
}

[laco-dev]

테스트 코드에서는 무작위라는 불확실성을 테스트 하는 것은 상당히 어렵습니다.
그렇기 때문에 우리는 의존성 주입과 같은 방식을 이용해 도메인 로직에만 테스트 하도록 집중을 할 수 있어요.

차량이 이동 가능한지에 대한 조건을 정하고, 각각 구현하도록 만들어서 자동차의 이동 조건을 테스트에서 제어가 가능하도록 만들어 보세요.

// Hint
interface CarMoveable {
  fun getMovementValue(): Int
}

class RandomCarMoveable: CarMoveable  {
   override fun getMovementValue(): Int { ... }
}

object AlwaysForwardCarMoveable: CarMoveable {
   override fun getMovementValue(): Int { return 4 }
}

...

[laco-dev]

자동차의 경우 입력받은 값이 "무작위" 라는 사실에 관심이 있을까요?