feature(step2): 로또(자동) 구현 #4160

LeeJin0527 · 2025-04-06T04:02:54Z

안녕하세요~! 오늘도 잘 부탁드립니다 😄

- 1000원당 1장 로또 갯수 계산 - 테스트 코드 추가 - LottoCountCalculator 추가

wooobo

2단계 잘 진행해주셨습니다~

몇가지 코멘트 남겨 놓았습니다~
확인 후 재요청 부탁드립니다!

wooobo · 2025-04-06T13:35:30Z

src/main/java/lotto/domain/LottoCountCalculator.java

+
+public class LottoCountCalculator {
+
+    private static final int PRICE_PER_TICKET = 1000;


Amount라는 객체를 도입해보는 건 어떨까요?
객체지향과 TDD 관점에서 보면, static 메서드만 존재하는 클래스는 이번 과정에서 지양하는 편이 좋을것 같습니다~ 😄

wooobo · 2025-04-06T13:37:59Z

src/main/java/lotto/domain/LottoNumberGenerator.java

+    private static final int LOTTO_MINIMUM_NUMBER = 1;
+    private static final int LOTTO_MAXIMUM_NUMBER = 45;


Suggested change

private static final int LOTTO_MINIMUM_NUMBER = 1;

private static final int LOTTO_MAXIMUM_NUMBER = 45;

private static final int LOTTO_MINIMUM_NUMBER = 1;

private static final int LOTTO_MAXIMUM_NUMBER = 45;

이 상수는 이미 Lotto 객체가 책임지고 있으니, 중복해서 책임을 갖기보다는 한 곳에서만 관리하는 편이 더 좋지 않을까요?

wooobo · 2025-04-06T13:39:22Z

src/main/java/lotto/domain/LottoResult.java

+
+public class LottoResult {
+
+    private static final int NO_MATCH = 0;


Rank가 가지는건 어떤가요?

wooobo · 2025-04-06T13:40:45Z

src/main/java/lotto/domain/Lottos.java

+    private final List<Lotto> values;
+
+    public Lottos(List<Lotto> values) {
+        this.values = List.copyOf(values);


copy와 같은 동작은 오히려 Lottos를 사용하는 쪽에서 제어하거나, 정적 팩토리 메서드인 Lottos.copyOf(...) 형태로 제공하는 편이 더 자연스럽지 않을까요?

wooobo · 2025-04-06T13:54:35Z

src/main/java/lotto/domain/LottoResult.java

+
+    private void compareAll(Lottos lottos, Lotto winningLotto) {
+        for (Lotto lotto : lottos.getValues()) {
+            int matchCount = (int) lotto.getNumbers().stream()


값을 꺼내기보단 값을 가지고 있는 객체에게 시켜보면 어떨까요?

for (Lotto lotto : lottos.getValues()) { Rank rank = lotto.match(winningLotto); ...

wooobo

2단계 잘진행주셨습니다!

몇가지 코멘트 남겨놓았어요~
다음 단계진행전에 확인 부탁드려요~

이만 merge진행하겠습니다!
그럼 다음 단계도 화이팅입니다!

wooobo · 2025-04-07T12:09:54Z

src/main/java/lotto/domain/Amount.java

+    private final int value;
+
+    public Amount(int value) {
+        InputValidator.validatePurchaseAmount(value);


InputValidator가 꼭 필요한지 의문이드는데요!

Amount 도메인 책임이 아닐지 생각이들어서요!

빈값일수없다

0이상이어야한다라는

wooobo · 2025-04-08T04:04:07Z

src/main/java/lotto/Main.java

        Lottos lottos = buyLottos(count);
        ResultView.printPurchasedLotto(lottos);

        Lotto winningLotto = InputView.inputWinningNumbers();
        LottoResult result = new LottoResult(lottos, winningLotto);

-        ResultView.printStatistics(result.getResults(), result.calculateProfitRate(amount));
+        ResultView.printStatistics(result.getResults(), result.calculateProfitRate(amount.getValue()));


객체에게 메시지를 보내기를 좀 더 활용해본다면,

public class LottoResult { .... public Amount calculateProfitRate(Amount amount) { return amount.multiply(getTotalReward()); }

이런식을 생각해볼수도 있을것 같아요!

wooobo · 2025-04-08T04:57:13Z

src/main/java/lotto/domain/LottoMachine.java

+        List<Lotto> lottoList = new ArrayList<>();
+
+        for (int i = 0; i < count; i++) {
+            lottoList.add(LottoNumberGenerator.generate());


Lotto에 대한 의존성을 깊은 레이어까지 거져가는게 맞을지 고민이 되네요.
그래서 인자로 넘겨주는 방식이 더 나을까 생각 중입니다 🤔
관점 공유 정도로 봐주시면 감사하겠습니다~

lottoList.add(LottoNumberGenerator.generate(Lotto.MIN_NUMBER, Lotto.MAX_NUMBER));

wooobo · 2025-04-08T09:01:57Z

src/main/java/lotto/domain/LottoResult.java

+            int matchCount = lotto.countMatchWith(winningLotto);
+            Rank rank = Rank.from(matchCount);
+
+            if (rank.isWin()) {


2depth를 없애기위해서 다양한 시도를 해볼수있을것 같아요.
그중에 가장 쉬운 방법이 메소드 분리일것 같아요 🤔

wooobo · 2025-04-08T09:07:12Z

src/test/java/lotto/domain/LottoNumberGeneratorTest.java

+        Lotto numbers = LottoNumberGenerator.generate();
+
+        Set<Integer> unique = new HashSet<>(numbers.getNumbers());
+
+        assertThat(unique).hasSize(6);


Suggested change

Lotto numbers = LottoNumberGenerator.generate();

Set<Integer> unique = new HashSet<>(numbers.getNumbers());

assertThat(unique).hasSize(6);

assertThatThrownBy(() -> LottoNumberGenerator.generate(100, 200))

저같은 경우 TDD를 통해서 어떤 메시지(인자 or 메소드명)을 전달할지 작은 범위에서 부터 테스트하였다면
이러한 테스트를 만들었을것 같아요 🤔

RepeatedTest테스트가 필요한 경우가 있지만, 지금 같은 경우에 운좋게 RepeatedTest를 모두 통과하는 경우의 수가 발생한다면 정말 좋은 테스트일까요?

wooobo · 2025-04-08T09:14:47Z

src/test/java/lotto/domain/LottoCountCalculatorTest.java

+    @DisplayName("금액에 따른 로또 갯수를 계산한다.")
+    @ParameterizedTest
+    @CsvSource({
+            "900, 0",


"900, 0",는 특별한 케이스로 구분지어서 별도로 만들수 있을것 같아요

wooobo · 2025-04-08T09:16:27Z

src/main/java/lotto/view/ResultView.java

+            System.out.printf(MESSAGE_COUNT, rank.getMatchCount(), rank.getReward(), count);
+        }
+
+        String profitState = profitRate >= 1.0 ? PROFIT : LOSS;


관점에 따라 다르겠지만,
1.0또한 도메인 로직으로 바라본다면,
isLoss를 물어볼수있는 객체를 고민해볼것같아요 🤔

LeeJin0527 added 13 commits April 5, 2025 12:18

refactor(step1): 에러메시지 동적으로 받을 수 있도록 리팩토링

edd7ea6

refactor(step1): 테스트당 assert 하나 적용 테스트 코드 리팩토링

1200200

refactor(step1): SRP 만족하도록 객체 분리 리팩토링

0aabd1e

refactor(step1): SRP 만족하도록 객체 분리 리팩토링

ea0df1e

docs(step2): 기능 목록 구현

8d2d898

move(step2): package 구조 분리 (lotto, calculator, common)

2e0ecf1

feat(step2): 구입 금액 입력 기능 구현 및 예외 처리 추가

64bae0e

feat(step2): 금액에 따른 로또 갯수 계산

61e105e

- 1000원당 1장 로또 갯수 계산 - 테스트 코드 추가 - LottoCountCalculator 추가

feat(step2): 로또 번호 생성 기능 구현

e7a75bf

feat(step2): 로또 출력 기능 구현

8f52a7c

feat(step2): 당첨 번호 입력 기능 구현

9dfa3ce

feat(step2): 당첨 결과 및 수익률 계산 로직 추가

f10222d

feat(step2): 결과 값 출력 로직 구현

19786ac

LeeJin0527 changed the base branch from master to leejin0527 April 6, 2025 04:03

wooobo requested changes Apr 6, 2025

View reviewed changes

LeeJin0527 added 5 commits April 7, 2025 20:42

feat(step2): 결과 값 출력 로직 구현

f473e08

refactor(step2): 정적 메서드 제거 Amount 객체 추가

8d1af1f

refactor(step2): LOTTO 안의 상수 값 사용하도록 수정

34a4491

refactor(step2): 값 꺼내지 않고 객체가 행동하도록 리팩토링 (Rank, Lotto)

76a9711

refactor(step2): 불변 객체 정적 팩터리 메서드 생성하도록 수정

fb379c5

wooobo approved these changes Apr 8, 2025

View reviewed changes

wooobo merged commit b4b51a1 into next-step:leejin0527 Apr 8, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feature(step2): 로또(자동) 구현 #4160

feature(step2): 로또(자동) 구현 #4160

LeeJin0527 commented Apr 6, 2025

wooobo left a comment

wooobo Apr 6, 2025

wooobo Apr 6, 2025

wooobo Apr 6, 2025

wooobo Apr 6, 2025

wooobo Apr 6, 2025

wooobo left a comment •

edited

Loading

wooobo Apr 7, 2025

wooobo Apr 8, 2025

wooobo Apr 8, 2025

wooobo Apr 8, 2025

wooobo Apr 8, 2025

wooobo Apr 8, 2025

wooobo Apr 8, 2025


		public class LottoCountCalculator {

		private static final int PRICE_PER_TICKET = 1000;

		private static final int LOTTO_MINIMUM_NUMBER = 1;
		private static final int LOTTO_MAXIMUM_NUMBER = 45;


		public class LottoResult {

		private static final int NO_MATCH = 0;

feature(step2): 로또(자동) 구현 #4160

feature(step2): 로또(자동) 구현 #4160

Conversation

LeeJin0527 commented Apr 6, 2025

wooobo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

wooobo left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

wooobo left a comment •

edited

Loading