[자동차 경주] 정남진 미션 제출합니다. #107
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
원래는 CarTest에 자동차 이름에 관한 테스트를 같이 작성할 계획이였지만 따로 테스트를 하는 것이 더 옳다 판단하여 테스트를 분리
1 line 함수라 expression body로 변환 및 0을 BigInteger에 정의된 상수로 변경
기능 요구 사항에서는 각 자동차에 이름을 부여할 수 있다고 했지만 만약 자동차에 이름을 부여하지 않을 경우 자동차 이름 입력, 차수별 실행 결과와 최종 우승자 출력시 문제가 발생할 소지가 있으므로 자동차 이름을 반드시 부여하도록 변경
| value class CarName(private val name: String) { | ||
| init { | ||
| require(name.length <= MAX_CAR_NAME_LENGTH) | ||
| require(name.isNotEmpty()) |
There was a problem hiding this comment.
자동차 이름이 중복되는 경우도 예외 처리 하셨으면 더 좋을 거 같아요!
There was a problem hiding this comment.
요구 사항에 자동차에 이름을 부여할 수 있다고 해서 이름을 부여하지 않은 경우를 고민할때 중복 이름에 관해서도 생각해봤는데 사람도 동명이인이 있듯이 우선 중복 이름을 허용하긴 했습니다. 사실 중복 이름을 허용하면 최종 우승자 식별에 좀 어려움이 있긴하죠
|
|
||
| import java.math.BigInteger | ||
|
|
||
| fun validateTryNumber(tryNumber: BigInteger) = require(tryNumber > BigInteger.ZERO) |
There was a problem hiding this comment.
BigInteger라는 타입을 쓸 수 있군요! 배워갑니다!
There was a problem hiding this comment.
Int의 최대값이 2147483647로 생각보다 작아서 BigInteger를 사용해봤습니다. 나중에 Long 타입 이상의 큰 정수값까지 다루어야할 경우 한번 사용해보세요.
jhseo12
reviewed
Oct 29, 2024
| fun main() { | ||
| // TODO: 프로그램 구현 | ||
| val ui = Ui() | ||
|
|
There was a problem hiding this comment.
Ui 생성과 아래 코드를 나눈 이유는 위는 단순 인스턴스 생성이고 아래부터는 프로그램이 실행되는 로직이라 분리하였습니다.
|
|
||
| override fun toString(): String = name | ||
|
|
||
| companion object { |
There was a problem hiding this comment.
동반 객체는 클래스에 연관된 상수와 인스턴스를 조금 더 간편하게 생성하기 위한 helper 함수를 정의하기 위해서 사용했습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.