Step3 : 자동차 경주 #1512
Step3 : 자동차 경주 #1512
Conversation
| @@ -0,0 +1,10 @@ | |||
| package racingcar.mock | |||
There was a problem hiding this comment.
테스트 더블에서 mock은 호출에 대한 기대를 명세하고 내용에 따라 동작하도록 프로그래밍 된 객체를 의미합니다.
현재는 호출에 대한 기대를 정의하지 않고(어떤 메서드 호출 시 어떤 값이 반환될 지 정의하는 것) 정해진 값을 리턴하고 있으므로 Mock보다는 Stub에 가깝지 않나 하는 생각이 드네요.
자세한 내용은 아래 문서 한번 참고 부탁드립니다.
https://tecoble.techcourse.co.kr/post/2020-09-19-what-is-test-double/
| package racingcar | ||
|
|
||
| class Car( | ||
| var position: Int = 0, |
There was a problem hiding this comment.
지금과 같은 구조라면 move를 호출하지 않고도 외부에서 가변인 position의 값을 바꿀 수 있을 것 같네요. 이 부분에 대해 고민해보시면 좋겠습니다.
| var moveCondition: RandomNumberHolder | ||
| ) { | ||
| fun move() { | ||
| if (moveCondition.getRandomNumber() >= 4) { |
There was a problem hiding this comment.
추상화의 범위를 어떻게 잡을 지 정답은 없으나 여기서는 랜덤 넘버를 반환하는 것 보다는 움직일지 말지 여부를 결정하는 부분을 추상화하는게 어땠을까 하는 생각이 드네요. 이 부분에 대해서도 고민해보시면 좋겠습니다.
정답은 없으므로 변경하진 않으셔도 됩니다.
|
|
||
| data class InputView( | ||
| var gameCount: Int = 0, | ||
| var cars: List<Car> = listOf() |
There was a problem hiding this comment.
아래 프로퍼티로 정의할 필요가 없을 것 같다고 말씀드리긴 했는데요, 가변 프로퍼티라 추가로 코멘트 남깁니다.
가변성과 관련해서는 아래 내용 한번 참고해보시면 좋겠습니다.
| var gameCount: Int = 0, | ||
| var cars: List<Car> = listOf() |
There was a problem hiding this comment.
둘 다 프로퍼티로 정의할 필요가 있을까요? 필요한 값을 바로 반환할 순 없을까요?
| } | ||
|
|
||
| fun validate(input: Int) { | ||
| require(input > 0) { throw IllegalArgumentException("음수는 입력할 수 없습니다.") } |
There was a problem hiding this comment.
require은 조건에 따른 결과가 맞지 않을 경우 IllegalArgumentException을 반환하기 때문에 예외 메시지만 남겨주시면 될 것 같습니다.
리뷰어님 안녕하세요.
3단계 미션 PR 드립니다.
잘 부탁드립니다!
임의의 Random 숫자 테스트를 위해 인터페이스를 활용하여 Mock 객체를 사용했습니다.
설계