Step3 : 리뷰 요청드립니다. #4222

Mo-bile · 2025-11-27T17:26:10Z

안녕하세요?
리뷰 요청드립니다 👍

금번 보너스 번호 기능을 추가하면서 어디 object에 2,3등 간에 등수를 결정시킬지 고민을 오래한것 같습니다.

그 결과

로또 번호에 대한 추가 검증이므로 'rankDecideByBonusNumber 메서드를 추가
Bonus 클래스를 새로 만들어 보너스 번호와 로또 번호의 한번 더 검증
Rank 클래스에 2 혹은 3등인지 구분하는 메서드

이렇게 크게 구분하여 이번 미션을 진행한 것 같습니다.
단순하게 구현하면 indent 가 2이상 늘어나버리고 복잡한 형태가 만들어지고, 보기싫어지고 읽기 싫어지더라고요

처음에 요구사항을 보면 쉽다고 생각했는데 막상 해보니 녹록치 않은 과제였던 것 같습니다

좋은 과제 감사드립니다 💯

1. 가변인자 변경 2. 일부 메서드 명 변경

1. 메서드 호출을 명확히 하기위해 호출 변경 2. Lotto 호출 생성자 방식에 맞춰 테스트 코드 픽스쳐 변경

javajigi

3단계 미션 진행하느라 수고했어요. 👍
적은 수의 객체 분리로 로또 미션 진행했는데요.

규칙 3: 모든 원시값과 문자열을 포장한다.
규칙 8: 일급 콜렉션을 쓴다.

위 두 개 원칙을 지키면서 객체 분리에 대한 리팩터링 도전해 보세요.
추가로 메서드 이름이 의도를 드러내지 못하는 경우가 많은데요.
메서드 이름은 가능하면 AI에게 피드백을 받아 개선해 보는 습관을 들여보면 좋겠습니다.

src/main/java/lotto/controller/LottoApplication.java

src/main/java/lotto/domain/model/Bonus.java

src/main/java/lotto/domain/model/LottoNumber.java

src/main/java/lotto/domain/model/WinningResult.java

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

src/test/java/lotto/domain/model/LottosTest.java

src/test/java/lotto/domain/model/LottoTicketsTest.java

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

1. Lotto 와 Bonus 간에 메시지를 주는 객체간 관계 재조정 및 테스트 제거 2. findMatchCount 접근제어자 private 로 변경하면서 테스트 제거

…동성을 좀 더 드러나게 변경

…스쳐 가독성향상

그 이유는 Bonus 의 역할과 지정된 행동이 일반적인 로또 번호와 차이가 없음

…나게 변경

javajigi

피드백 반영하느라 수고 했어요. 👍
2단계에서 3단계로 오면서 당첨 번호에 보너스 번호가 추가되었는데요.
당첨 번호를 담당하는 객체를 추가해 보면 어떨까요?
이렇게 당첨 번호를 담당하는 객체를 추가하면서 Lotto의 책임과 역할을 당첨 번호를 담당하는 객체로 이동해 볼 것을 추천합니다.

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

src/main/java/lotto/domain/model/LottoTickets.java

src/main/java/lotto/domain/Rank.java

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

src/test/java/lotto/domain/model/WinningResultTest.java

src/test/java/lotto/domain/RankTest.java

2. 포함여부에 대한 구현 대신 컬렉션 API 활용하여 간편하게 변경 3. LottoNumber 객체가 생겨서 출력이 상이해진 부분 수정 4. 테스트 코드 금액표시에 3자리수 마다 _ 추가하여 가독성 향상

1. WinningLotto 신설해서 당첨값 관리 책임에 대한 객체 신설 2. Rank 와 Lotto 간에 순환 참조 개선 (순위 뽑은 뒤 다시 2,3등 뽑는구조 개선)

1. 등수를 최종 지정하는 역할을 Rank 에서 WinningLotto 로 이관

…o` 는 로또번호 lotto 객체와 Rank 사이에 조율할 수있게 변경

javajigi

이전 단계보다 확실히 코드가 많이 개선되었음을 느낄 수 있네요. 👍
코드 컨벤션도 잘 지키고 있네요.
다음 단계 넘어가기 전에 꼭 연습해 봤으면 하는 부분이 있어 피드백 남겨요.
LottoNumber 재사용 부분만 반영하고 다음 단계로 가시죠.

javajigi · 2025-11-29T23:23:27Z

src/main/java/lotto/domain/model/LottoNumber.java

@@ -0,0 +1,18 @@
+package lotto.domain.model;
+
+public record LottoNumber(int value) {


상황

1만명의 사용자가 동시에 당첨 여부를 확인할 수 있어야 한다.

1명의 사용자는 평균 5장의 로또를 구매한 상태이다.

위 요구사항을 서버에서 생성되는 LottoNumber의 인스턴스의 갯수는?
1 * 5장 * 6개 숫자/1장 * 1만명 = 30만개이다.

동시에 생성되는 인스턴스 갯수가 너무 많다.
인스턴스 갯수를 줄일 수 있는 방법은?
로또 숫자 값을 LottoNumber 객체로 래핑하는 경우 매번 인스턴스가 생성되기 때문에 인스턴스의 갯수가 너무 많아져 성능이 떨어질 수 있다.
LottoNumber 인스턴스를 생성한 후 재사용할 수 있도록 구현한다.

힌트 : Map과 같은 곳에 인스턴스를 생성한 후 재사용하는 방법을 찾아본다.

javajigi · 2025-11-29T23:25:13Z

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

+
+import lotto.domain.Rank;
+
+public record WinningLotto(LottoNumber bonusNumber, Lotto winNumbers) {


Suggested change

public record WinningLotto(LottoNumber bonusNumber, Lotto winNumbers) {

public record WinningLotto(Lotto winNumbers, LottoNumber bonusNumber) {

저 같으면 로또를 첫 번째 인자로 구현할 것 같음

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

javajigi · 2025-11-29T23:26:47Z

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

+        }
+    }
+
+    public Rank rankDecide(Lotto lotto) {


WinningLotto로 이 메서드를 이동한 점 👍

javajigi · 2025-11-29T23:27:19Z

src/main/java/lotto/domain/model/WinningResult.java

-
-    private void increaseCount(Match match) {
-        this.matchMap.compute(match, (k, count) -> {
+    public void recordRank(Rank rank) {


javajigi · 2025-11-29T23:27:42Z

src/main/java/lotto/domain/Rank.java

+        this.winnerReturn = winnerReturn;
+    }
+
+    public static Rank valueOf(int matchCount, boolean bonusMatch) {


…가하여 인스턴스 생성을 방지하여 성능 개선

javajigi

캐시 적용 잘 했네요. 👍
단, 캐시 적용에 아쉬운 부분이 있어 피드백했으니 4단계 진행할 때 함께 반영해 보세요.
4단계는 최대한 컴파일 에러를 내지 않으면서 안정적으로 기능 추가하는 연습해 볼 것을 추천해요.

javajigi · 2025-11-30T07:13:21Z

src/main/java/lotto/domain/model/LottoNumber.java

+
+public record LottoNumber(int value) {
+
+    public LottoNumber(String value) {


캐시 적용 👍
단 이와 같이 구현할 경우 캐시를 사용하지 않고 LottoNumber를 직접 생성 가능하다.
LottoNumber를 외부에서 직접 생성하지 못하도록 private으로 구현하고 캐시 효과를 내도록 리팩터링해본다.

힌트: 정적 팩터리 메서드

모재영 added 14 commits November 26, 2025 10:14

refactor : findMatchCount 의 인자 타입을 변경하여 테스트 쉽게 변경

88f9c28

refactor

def4fcd

1. 가변인자 변경 2. 일부 메서드 명 변경

refactor : 가변인자 개선

4ea4fcf

feat : Match 상수를 추가하여 추가된 인자 처리

27e74c1

fix : 출력문 수정

d5af92d

refactor

152584c

1. 메서드 호출을 명확히 하기위해 호출 변경 2. Lotto 호출 생성자 방식에 맞춰 테스트 코드 픽스쳐 변경

refactor : Match enum을 Rank 로 변경하여 객체의 존재(what he is) 명확화

411132d

feat : 보너스 번호 관리를 위한 Bonus 객체 추가

11edc97

refactor : Rank 내 멤버와 메서드명 등을 명확히 변경하여 그 행동을 드러냄

48857cc

feat : 로또 5개일치시 2, 3등 보너스 번호로 구분하여 등수 선택 기능 구현 완료

21e89da

doc : 체크리스트 완료

5833fbd

refactor : 최종 2,3 등 등수결정에 관한 책임을 Lotto 객체로 이관하여 Lotto 번호에 대한 관리 책임 강화

292320a

fix : 2개 이하로 일치할 경우 match되는게 없어서 발생하는 NPE 방지 에러 해결

88e1d84

refactor : 불필요한 increaseCount 메서드 제거

fe13c69

javajigi requested changes Nov 27, 2025

View reviewed changes

모재영 added 9 commits November 28, 2025 10:00

refactor : Bonus 에 필드 불변으로 변경

e06ee54

feat : match Count 로 테스트 코드 변경

9a7e064

refactor

ce87905

1. Lotto 와 Bonus 간에 메시지를 주는 객체간 관계 재조정 및 테스트 제거 2. findMatchCount 접근제어자 private 로 변경하면서 테스트 제거

refactor : rankDecideByBonusNumber 메서드 Lotto 에서 Rank 객체로 이관하여 Rank의 능…

274eff7

…동성을 좀 더 드러나게 변경

refacotr : 가변배열 생성자 List로 전환방식 반복문에서 스트림 방식으로 방식하여 리턴하는 방식으로 변경

c04fd46

refactor : 주요 필드명, 인자명, 메서드명변경하여 의도 명확히 드러냄

37af5d1

refactor : WinningResult 의 가변생성자 방식 불필요로 제거하고 후 recordRank 이용하여 테스트 픽…

2e4e0d4

…스쳐 가독성향상

refactor : 기존 Bonus 객체를 LottoNumber 로 객체 명과 책임을 변경한다

7811b25

그 이유는 Bonus 의 역할과 지정된 행동이 일반적인 로또 번호와 차이가 없음

refactor : 로또 관련 number, lotto, lottos 3가지의 객체명과 필드명 재정립하여 도메인의 역할 드러…

f870d37

…나게 변경

javajigi requested changes Nov 28, 2025

View reviewed changes

모재영 added 5 commits November 29, 2025 16:53

refactor : refactor : 컨벤션 정렬 및 안쓰는 import 정리

799930b

refactor

eb62af0

1. WinningLotto 신설해서 당첨값 관리 책임에 대한 객체 신설 2. Rank 와 Lotto 간에 순환 참조 개선 (순위 뽑은 뒤 다시 2,3등 뽑는구조 개선)

refactor

ad60b8a

1. 등수를 최종 지정하는 역할을 Rank 에서 WinningLotto 로 이관

refactor : 랭킹 결정하는 메서드 Rank 로 이관하여 규칙을 아는 자가 할수있게 변경하고 `WinningLott…

d3713e8

…o` 는 로또번호 lotto 객체와 Rank 사이에 조율할 수있게 변경

모재영 added 6 commits November 29, 2025 22:54

refactor : 생성자 리팩터링

0e319fa

refactor : 각종 컨벤션 준수하게 리팩터링 및 정리

592d8f7

refactor : 반복문을 stream API 로 변경

49aba30

doc : 체크리스트 업데이트

46b91f3

chore : 안쓰는 패키지 제거

aea1cb1

feat : 타당성 검증 코드 추가, 방어적 복사 추가

06ef6d3

javajigi requested changes Nov 29, 2025

View reviewed changes

모재영 added 4 commits November 30, 2025 10:26

refacotor : 불필요한 생성자 제거

9358df3

feat : lottoNumber의 무한한 인스턴스 생성을 방지하기 위해 lottoNumber 의 캐싱하는 기능을 추…

bc5c566

…가하여 인스턴스 생성을 방지하여 성능 개선

docs : 리팩터링 목표 기록

deaf238

refactor : 상수값 추가

516c88f

javajigi approved these changes Nov 30, 2025

View reviewed changes

javajigi merged commit f730021 into next-step:mo-bile Nov 30, 2025

		@@ -0,0 +1,18 @@
		package lotto.domain.model;

		public record LottoNumber(int value) {


		import lotto.domain.Rank;

		public record WinningLotto(LottoNumber bonusNumber, Lotto winNumbers) {

	public record WinningLotto(LottoNumber bonusNumber, Lotto winNumbers) {
	public record WinningLotto(Lotto winNumbers, LottoNumber bonusNumber) {


		public record LottoNumber(int value) {

		public LottoNumber(String value) {

Step3 : 리뷰 요청드립니다. #4222

Step3 : 리뷰 요청드립니다. #4222

Uh oh!

Conversation

Mo-bile commented Nov 27, 2025

Uh oh!

javajigi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javajigi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javajigi left a comment

Choose a reason for hiding this comment

Uh oh!

javajigi Nov 29, 2025

Choose a reason for hiding this comment

상황

Uh oh!

javajigi Nov 29, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

javajigi Nov 29, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Nov 29, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Nov 29, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi left a comment

Choose a reason for hiding this comment

Uh oh!

javajigi Nov 30, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants