🚀 4단계 - 로또(수동) #3550

gkssk4163 · 2023-12-03T15:28:30Z

3단계 피드백 반영 및 4단계 기능구현 완료하여 리뷰요청드립니다~!

- 메시지 보내는 방식으로 변경 - 예외메세지 추가

- 메서드 레퍼런스 적용 - unmodifiableSet 적용 - 불필요한 함수 호출 제거

csh0034

4단계 잘 구현해주셨네요 👍
몇가지 고민해볼사항 남겨두었는데
확인후에 다시 리뷰요청 부탁드려요!

csh0034 · 2023-12-04T07:35:34Z

src/main/java/lotto/domain/WinningLotto.java

@@ -13,10 +13,10 @@ public WinningLotto(Lotto winningLotto, LottoNumber bonus) {

    private void validate() {


numbers() 를 꺼내와서 순회를 하고 있지만 실제 winningLottoNumber 는
사용하고 있지 않습니다.

하단과 같이 처리해볼수 있지 않을까요?

if (winningLotto.hasNumber(this.bonus)) { // 예외 발생 }

csh0034 · 2023-12-04T07:37:19Z

src/main/java/lotto/domain/Lotto.java

        this.numbers = new TreeSet<>(parseLottoNumber(numbers));
        validate();
    }

    public Lotto(Integer... numbers) {
-        this(Arrays.asList(numbers));
+        this(Set.copyOf(Arrays.asList(numbers)));


Suggested change

this(Set.copyOf(Arrays.asList(numbers)));

this(Set.of(numbers));

csh0034 · 2023-12-04T07:39:10Z

src/main/java/lotto/domain/Lotto.java

@@ -1,7 +1,7 @@
 package lotto.domain;


LottoNumber 에 변경사항이 없어 여기에 남겨놓습니다.

LottoNumber 의 경우 1~45 의 숫자이기 때문에 매번 인스턴스를 생성하지않고
캐싱처리하여 같은번호에 대해선 같은 인스턴스를 사용해볼수 있을것 같아요.

로또 피드백 - 정적 팩토리 메소드, 인스턴스 캐싱 을 참고하여 적용해보면 좋을것 같습니다.

csh0034 · 2023-12-04T07:40:55Z

src/main/java/lotto/domain/LottoBundle.java

 import java.util.stream.Collectors;
 import java.util.stream.IntStream;

 public class LottoBundle {

    private static final int LOTTO_PRICE = 1000;

-    private final List<Lotto> lottoList;
+    private final List<Lotto> manualLottoList;


LottoBundle 에선 수동인지 자동인지 구분없이 하나의 컬렉션으로 관리해도되지 않을까요? 🤔
정적팩토리 메서드를 활용하여 생성해야하는 자동로또 + 수동생성된 로또를 더해서 목록을
만들도록 해볼수도 있을것 같습니다.

csh0034 · 2023-12-04T07:43:55Z

src/main/java/lotto/view/InputView.java

-import java.util.Arrays;
-import java.util.List;
-import java.util.Scanner;
+import java.util.*;
 import java.util.stream.Collectors;

 public class InputView {


요구사항

예외 처리를 통해 에러가 발생하지 않도록 한다.

힌트

사용자가 잘못된 값을 입력했을 때 java exception으로 에러 처리를 한다.

상단의 요구사항에 맞춰 입력단계에서 예외가 발생하지 않도록
try catch 구문을 활용하여 재입력을 받도록 해보면 좋을것 같습니다.

csh0034 · 2023-12-04T07:48:06Z

src/test/java/lotto/domain/LottoTest.java

-                Arguments.of(Arrays.asList(1, 2, 3, 4, 5, 7), LottoResult.BONUS),
-                Arguments.of(Arrays.asList(1, 2, 3, 4, 5, 6), LottoResult.SIX),
-                Arguments.of(Arrays.asList(6, 7, 8, 9, 10, 11), LottoResult.FAIL)
+                Arguments.of(new Integer[]{1, 3, 5, 7, 9, 11}, LottoResult.THREE),


하단과 같이 주생성자를 이용하게 해볼수도 있을것 같습니다!

Suggested change

Arguments.of(new Integer[]{1, 3, 5, 7, 9, 11}, LottoResult.THREE),

Arguments.of(Set.of(1, 3, 5, 7, 9, 11)}, LottoResult.THREE)

gkssk4163 added 6 commits December 3, 2023 22:44

refactor: WinningLotto 보완

17f39bc

- 메시지 보내는 방식으로 변경 - 예외메세지 추가

refactor: Lotto 생성자 파라미터 타입 Set으로 변경

163f2e5

refactor: Lotto 기능 보완

aacac3d

- 메서드 레퍼런스 적용 - unmodifiableSet 적용 - 불필요한 함수 호출 제거

docs: 기능목록 추가

39fc959

feat: 로또 수동구매 기능구현

ccf96e4

feat: 로또 수동번호 입출력 구현

b0bc9f8

csh0034 requested changes Dec 4, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 4단계 - 로또(수동) #3550

🚀 4단계 - 로또(수동) #3550

gkssk4163 commented Dec 3, 2023 •

edited

Loading

csh0034 left a comment

csh0034 Dec 4, 2023

csh0034 Dec 4, 2023

csh0034 Dec 4, 2023

csh0034 Dec 4, 2023

csh0034 Dec 4, 2023

csh0034 Dec 4, 2023

		@@ -13,10 +13,10 @@ public WinningLotto(Lotto winningLotto, LottoNumber bonus) {

		private void validate() {

	this(Set.copyOf(Arrays.asList(numbers)));
	this(Set.of(numbers));

	Arguments.of(new Integer[]{1, 3, 5, 7, 9, 11}, LottoResult.THREE),
	Arguments.of(Set.of(1, 3, 5, 7, 9, 11)}, LottoResult.THREE)

🚀 4단계 - 로또(수동) #3550

Are you sure you want to change the base?

🚀 4단계 - 로또(수동) #3550

Conversation

gkssk4163 commented Dec 3, 2023 • edited Loading

csh0034 left a comment

Choose a reason for hiding this comment

csh0034 Dec 4, 2023

Choose a reason for hiding this comment

csh0034 Dec 4, 2023

Choose a reason for hiding this comment

csh0034 Dec 4, 2023

Choose a reason for hiding this comment

csh0034 Dec 4, 2023

Choose a reason for hiding this comment

csh0034 Dec 4, 2023

Choose a reason for hiding this comment

csh0034 Dec 4, 2023

Choose a reason for hiding this comment

gkssk4163 commented Dec 3, 2023 •

edited

Loading