[1단계 - 로또 구현] 벡터(백승주) 미션 제출합니다. #580
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
Byesol
changed the title
dusdn1702
requested changes
Feb 13, 2025
|
|
||
| ### 로또 구매 | ||
|
|
||
| - [x] 구입금액 안내멘트를 출력한다. |
| @@ -0,0 +1,10 @@ | |||
| package constant; | |||
|
|
|||
| public class ErrorMessage { | |||
Comment on lines
+16
to
+18
| private final Integer matchNumberCount; | ||
| private final Boolean matchBonusNumber; | ||
| private final Integer price; |
| public static WinLottoInfo result(LottoNumbers purchasedLotto, WinLotto winLotto) { | ||
| Integer matchNumberCount = winLotto.countMatchNumber(purchasedLotto); | ||
| Boolean bonusMatch = winLotto.bonusMatch(purchasedLotto); | ||
| if (matchNumberCount <= 2) { |
Comment on lines
+12
to
+18
| public LottoNumbers() { | ||
| this.numbers = generateLottoNumbers(); | ||
| } | ||
|
|
||
| public LottoNumbers(List<Integer> numbers) { | ||
| this.numbers = numbers; | ||
| } |
| } | ||
|
|
||
| public Integer readPurchaseAmount() { | ||
| Scanner sc = new Scanner(System.in); |
| for (Entry<WinLottoInfo, Integer> winLottoInfoIntegerEntry : result.entrySet()) { | ||
| WinLottoInfo winLottoInfo = winLottoInfoIntegerEntry.getKey(); | ||
| Integer count = winLottoInfoIntegerEntry.getValue(); | ||
| sum += winLottoInfo.getPrice() * count; |
There was a problem hiding this comment.
이렇게 사칙연산으로 구현하게 되면 계산의 우선순위에 대해 검증이 어렵지 않을까요?
| nullToZero(result.getCount(FIRST))); | ||
| } | ||
|
|
||
| public void printTotalReturn(Result result, Purchase purchase) { |
There was a problem hiding this comment.
우리가 코드에서 많이 사용하는 return이란 단어가 함수명에 들어가있으니 어색하게 느껴지는데 벡터는 어떠신가요?
| import org.junit.jupiter.api.DisplayName; | ||
| import org.junit.jupiter.api.Test; | ||
|
|
||
| class WinLottoInfoTest { |
|
남겨주신 질문도 답변드립니다 ~ 같이 고민해봐요 !
|
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.
리뷰 요청 및 질문 정리
1. MVC 구조 및 Service 역할
이번 미션에서 MVC 구조를 사용하여 구현했는데, 올바르게 작성된 구조인지 궁금합니다. 또한, 다른 크루들의 코드를 보니 서비스를 추가해서 구현하는 경우를 봤습니다.
View코드 내부에서 유효성 검사(Validation)를 진행하고 있는데, 이를 별도의 Validator 클래스로 분리하는 것이 좋을까요?2. Enum을 활용한 당첨 결과 타입 정리와 그 개수 관리
현재 로또 당첨 결과를
Enum타입으로 정의하고, 참가자의 1등, 2등 등의 당첨 결과에 대한 것들을 enummap으로 result란 클래스에서 해당 당첨지의 개수를 저장했는데 이렇게 result란 클래스를 만들어 저장하는것보다 아예 고객에 대한 구입 금액, 결과 등을 저장하는 방식이 더 괜찮을까요?https://prolog.techcourse.co.kr/studylogs/3872