🚀 4단계 - 로또(수동)

[chanani]

안녕하세요. 4단계 로또(수동) 과제 제출합니다 ! 🙇‍♂️

과제 진행 내용 정리

• LottoPrice 클래스에서 수동으로 구매할 로또의 수량 관리
• 수동으로 입력받은 로또 번호와 자동으로 생성되는 로또 번호를 하나의 컬랙션에서 관리

[javajigi]

확실히 3단계까지 객체 설계를 잘한 느낌이 드네요. 💯
그렇다 보니 4단계 요구사항을 추가할 때 영향을 받는 객체의 수가 최소한으로 구현 가능했던 것 같아요.
바로 머지하려다 마지막 단계인 만큼 약간의 아쉬움이 남아 보이는 부분에 대해 피드백 남겨봤어요.

[javajigi]

BuyLotto라는 이름이 사용자가 구매한 Lotto로 지은 이름으로 보여짐
이에 AI에게 물어보니 다음과 같은 답변을 줌
앞으로 이름을 지을 때 아쉬움이 남는다면 AI를 활용해 보면 어떨까?

현재 BuyLotto라는 이름은 객체의 책임을 명확히 나타내지 못하고 있습니다. 객체 이름은 **무엇(What)**을 캡슐화하고 관리하는지를 명료하게 보여줘야 합니다.

현재 클래스는 사용자가 구매한 로또 번호 목록을 캡슐화하고, 이 목록에 대한 **일련의 비즈니스 로직(당첨 여부 확인)**을 수행하는 일급 컬렉션의 역할을 하고 있습니다.

따라서 다음과 같은 이름들을 추천합니다.

🥇 가장 추천하는 이름: PurchasedLottos
설명: 이 이름은 이 객체가 구매된(Purchased) 로또(Lottos)들의 컬렉션임을 명확하게 전달합니다. 이는 객체의 상태와 본질을 가장 잘 나타냅니다.

장점:

**행위(Buy)**가 아닌 **본질(Purchased Lottos)**을 나타냅니다.

List를 캡슐화한 일급 컬렉션임을 암시합니다.

메서드(getLottos(), match())와의 연관성도 자연스럽습니다.

🥈 좋은 대안: LottoTickets
설명: 로또 티켓들이라는 의미로, 사용자가 소유한 로또 번호 세트들의 묶음을 나타내는 직관적인 이름입니다.

장점: 도메인 용어로서 매우 흔하게 사용되며 간결합니다.

🥉 도메인 특화: CustomerLottos
설명: 이 객체가 **사용자(구매자)**가 소유한 로또 목록이라는 점을 강조하고 싶을 때 적합합니다.

장점: 시스템에서 이 목록이 누구에게 속하는지 명확히 할 때 유용합니다.

💡 왜 BuyLotto는 아쉬운가?
BuyLotto는 **구매하다(Buy)**라는 **행위(Verb)**와 **로또(Noun)**가 합쳐진 이름입니다.

객체 이름은 일반적으로 **명사(Noun)**를 사용하여 **본질(Identity)**을 나타내야 합니다.

이 객체가 구매 행위 자체를 담당하는 것이 아니라, **구매된 결과물(로또 목록)**을 관리하는 것이 핵심 책임이므로, **과거분사(Purchased)**나 **명사(Tickets)**를 사용하여 이름을 짓는 것이 설계의 응집도를 높여줍니다.

결론적으로, PurchasedLottos로 이름을 변경하시어 객체의 책임을 명확히 표현하는 것을 권장합니다.

[javajigi]

현재 코드는 **객체지향 생활체조 원칙의 "규칙 7: 3개 이상의 인스턴스 변수를 가진 클래스를 쓰지 않는다.'**를 지키지 못하고 있다.

이 클래스에 인스턴스 변수(필드)가 3개 이상 사용되고 있는데, 이 필드들을 줄이거나 분리하는 방법이 있을까? 이 2개의 TicketCount를 필드가 아닌 다른 방식으로 접근하는 것은 어떨까?
생성자에서 이 값을 구하기 위해 너무 많은 책임을 맡고 있는 것은 아닐까?

[javajigi]

Suggested change

	public static void createLotto(BuyLotto buyLotto, LottoPrice lottoPrice) {
	public static BuyLotto createLotto(LottoPrice lottoPrice) {

이 메서드는 BuyLotto를 생성해 반환하는 방식으로 구현하면 어떨까?
메서드 이름은 무엇인가 값을 반환하는 것으로 보여짐

위와 같이 구현한 후 두개의 ArrayList를 합칠 때 addAll(ArrayList values)와 같은 메서드로 추가하는데 위와 같이 생성한 후에 수동, 자동의 List를 합치는 또 다른 메서드를 추가하는 것은 어떨까?

[javajigi]

로또 미션 끝까지 잘 마무리했네요. 💯
최근 들어 상당히 미션 진행해 몰입하다보니 빠르게 마무리했네요.
바로 merge하려다 혹시라도 추가 연습해보고 싶은 부분이 있을 수 있어 request changes로 남겨봅니다.
혹시라도 추가 리팩터링해보고 싶은 부분 있으면 도전해 보시고요.
없으면 바로 리뷰 요청해도 됩니다.

아니면 로또 미션은 진행 상태로 두고 "수강신청 - 레거시 코드 리팩터링" 미션 진행하다 일정 시간이 지난 후 다시 리팩터링 도전해 보는 것도 좋은 연습 방법일 수 있습니다. AI와 함께 로또 미션의 객체 설계에 대한 피드백을 받아보는 것도 좋은 방법일 것 같아요.

[javajigi]

'int price'를 포장한 Price 객체가 있는데 이 객체가 price / 1_000 로직 구현을 담당해도 되지 않을까?
그렇다면 (price / 1_000) - passiveTicketCount 로직도 Price 객체가 담당할 수도 있을 것 같다.