[자동차 경주] 천지윤 미션 제출합니다. #100
Conversation
| if (!(cars.all { it -> it.count() <= 5 })) { | ||
| throw IllegalArgumentException("자동차 이름은 5자 이하만 가능합니다") | ||
| } |
There was a problem hiding this comment.
람다식에서 사용되는 기본 변수 네임을 it 그대로 사용하실 거라면
it ->은 생략해도 될 것 같네요 !
There was a problem hiding this comment.
오 생략해도 되는지 몰랐네요. 알려주셔서 감사합니다!
|
|
||
| import java.lang.NumberFormatException | ||
|
|
||
| class Validation { |
There was a problem hiding this comment.
저는 아래 케이스에 대해서도 고민을 해봤는데요 한번 생각해보셔도 좋을거 같습니다 !
게임 플레이를 한명만 입력되도 허용할 것인지?
이름에 특수문자를 허용하실 것인지?
,가 아닌 구분자가 들어왔을 때 : 3;4;5
There was a problem hiding this comment.
감사합니다. 오류는 아니라고 생각해서 하지 않았는데, 사용자가 실수하는 것도 고려해볼 수가 있겠네요!
|
|
||
| return count | ||
| } | ||
|
|
||
| fun main() { |
There was a problem hiding this comment.
보통 메인 함수가 가장 최상단에 위치하도록 하는 것 같습니다 :)
| } | ||
|
|
||
| fun isCanGo(): Boolean { | ||
| return Randoms.pickNumberInRange(0, 9) >= 4 |
There was a problem hiding this comment.
0과 9, 4 의 의미를 명확히 할 수 있도록 상수화를 하면 어떨까요 ?
There was a problem hiding this comment.
아하 좋네요! 앞으로 의미가 있는 숫자는 상수화를 해야겠습니다
| fun winnerPrint(winners: List<String>) { | ||
| println("최종 우승자 : ${winners.joinToString(", ")}") | ||
|
|
||
| } |
There was a problem hiding this comment.
코틀린 함수 네이밍 컨벤션에 따르면 동사 + 명사 구조로 네이밍을 하도록 가이드하고 있습니다 !
println처럼 printWinner로 네이밍 하는 것이 맞을 것 같습니다 🤔
| fun gameResult(carName: String, movingCount: Int) { | ||
| println("$carName : ${"-".repeat(movingCount)}") | ||
| } |
There was a problem hiding this comment.
함수의 이름이 "게임의 결과"를 출력한다는 의미일까요 ? 그렇다면 printGameResult 는 어떨까요 !
| } | ||
|
|
||
| fun inputCount() { | ||
| println("시도할 횟수는 몇 회인가요?") |
There was a problem hiding this comment.
"시도할 횟수는 몇 회인가요?" 와 같이 변할 수 있는 값을 변수로 빼서 관리하면 유지보수가 수월해질꺼 같습니다!
| } | ||
|
|
||
| fun checkInputCarIsNotEmpty(inputCar: String) { | ||
| if (inputCar == "") { |
There was a problem hiding this comment.
There was a problem hiding this comment.
와 감사합니다. 코틀린 문법 하나 또 배웠습니다!!
| @@ -3,6 +3,25 @@ package racingcar | |||
| import camp.nextstep.edu.missionutils.Randoms | |||
| import camp.nextstep.edu.missionutils.Console | |||
|
|
|||
|
|
|||
| fun getWinnerIndex(carsMovings : Array<Int>): MutableList<Int> { | |||
There was a problem hiding this comment.
저희 1주차 안드로이드 피드백에 배열 대신 컬렉션을 사용한다!고 있습니다. 다음에는 List를 사용해서 구현해보는 건 어떨까요?
There was a problem hiding this comment.
아 맞네요 잊어버렸던 피드백인데 알려주셔서 감사합니다! ㅠㅠㅠ
구현할 기능 목록
자동차 이름 분류
이동할 횟수
랜덤 이동값
최종 우승자
depth 2이하
테스트