Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[자동차 경주] 이서현 미션 제출합니다. #231

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

nuyhhyun
Copy link

@nuyhhyun nuyhhyun commented Nov 1, 2023

No description provided.

Copy link

@junjange junjange left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전체적으로 패키지 분리와 클래스 분리를 하면 코드를 이해하기 더 좋을 거 같습니다!

println(START_MESSAGE)
var CarNames: List<String> = getCarNames()
println(TRIAL_MESSAGE)
val trialCount: Int? = Console.readLine().toIntOrNull()
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trialCount가 nulll일 때 처리를 해주는 게 좋을 거 같아요

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

71번 줄에 따로 해둔다고 해놨었는데 생각해보니 null로 가지고 있는 것보다는 받은 걸 나중에 확인하는 게 훨씬 나을 수도 있겠네요!! 감사합니다!!😃

fun startGame() {

println(START_MESSAGE)
var CarNames: List<String> = getCarNames()
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

대부분 변수마다 타입을 정의 하셨는데 코틀린은 타입 캐스팅이 되기 때문에 따로 안 적으셔도 됩니다!
그냥 보기 편하려고 의도하신거라면 그냥 지나가셔도 됩니다!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

어디에 쓰고 어디에 뺄지 모르겠어서 안쓰다가도 채우게 되더라구요ㅜㅜ
어느새 까먹고 있던 부분이었는데 기준을 세워봐야겠습니다 감사합니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants