🚀 4단계 - 로또(수동)

[hhhhhwi]

안녕하세요! PR이 점점 더 늦어지네요 ㅠ

그래도 최선을 다 해보겠습니다 ㅠ

이번 미션은.. 쉬울 거라고 생각했는데 생각보다 고민해야 되는 지점이 많았습니다 ㅠ

제일 고민되었던 지점은

1. ManualLottoNumberStrategy 적용 시 사용자의 입력값 대로 로또번호가 생성되기 떄문에 InputView의 역할이 반드시 필요한데,
   이 때 입력의 책임을 도메인 로직에 넣을 것인가?

2. LottoNumbers와 WinningNumbers의 검증 로직 측면에서는 성격이 같지만, 역할이 다른 데 WinningNumbers에 LottoNumbers를 조합해도 될까?

위 지점들이였습니다. 피드백을 받는 게 더 좋을 것 같아 일단 조악한 코드 😓 이지만 PR 먼저 올립니다! 잘 부탁드립니다!

[neojjc2]

#tag @neojjc2

[neojjc2]

안녕하세요! PR이 점점 더 늦어지네요 ㅠ

그래도 최선을 다 해보겠습니다 ㅠ

이번 미션은.. 쉬울 거라고 생각했는데 생각보다 고민해야 되는 지점이 많았습니다 ㅠ

제일 고민되었던 지점은

1. ManualLottoNumberStrategy 적용 시 사용자의 입력값 대로 로또번호가 생성되기 떄문에 InputView의 역할이 반드시 필요한데,
   이 때 입력의 책임을 도메인 로직에 넣을 것인가?
2. LottoNumbers와 WinningNumbers의 검증 로직 측면에서는 성격이 같지만, 역할이 다른 데 WinningNumbers에 LottoNumbers를 조합해도 될까?

위 지점들이였습니다. 피드백을 받는 게 더 좋을 것 같아 일단 조악한 코드 😓 이지만 PR 먼저 올립니다! 잘 부탁드립니다!

조악하다뇨 무슨 그런 말씀을 😅
이미 큰 코드 수정 없이 마지막 단계 요구사항이 반영되었다는것만 봐도
이전 구조나 개선을 잘 해주셔서라고 생각합니다

InputView나 혹은 Output을 담당하는 역할은 언제든 변경이 되어야할 부분이기 때문에
최대한 도메인로직과 거리를 두거나, 레이어를 통해 서로간의 변경되는 부분에 대한 영향을 줄여야 한다고 생각합니다.

리뷰 코멘트로 의견 드렸는데요 😄
마지막 단계인만큼 조금만 더 정리하고 가시면 좋을 것 같습니다 🙇

[neojjc2]

안녕하세요 휘연님 🙇

마지막 단계 잘 진행해주셨습니다 😄
고민하고 계신 부분에 대해 소소한 의견드렸는데요,
한번 개선검토 해주시면 좋을 것 같습니다 🙇

그럼 재 리뷰 요청 기다리고 있겠습니다 🙏

[neojjc2]

요구사항 정리 💯

[neojjc2]

검증하신 부분 좋습니다 👍
다만 Integer인 로또번호 자체가 원시객체포장 된다면 어떨까요??
왜냐하면 WinningNumbers에서 사용중인 bonusNumber도 결국 로또 번호중에 하나입니다 😄
로또 번호 범위에 대한 검증하고 있는 코드가 중복으로 있는데요, 이 부분이 좀 정리될 것 같습니다 🙇

// AS IS
List<Integer> lottoNumbers;

// TO BE
List<LottoNumber> lottoNumbers;

이 부분 한번 개선검토 해주시면 좋을 것 같습니다 🙇

[neojjc2]

LottoPrice를 통해서 자동, 수동 수량의 역할과 책임을 부여하신 부분 너무 좋습니다 👍

다만 LottoMarket에 lottoPrice를 전달해서 LottoMarket에서 수동로또를 사는 역할을 하면 어떨까요??
지금 구조에서는 main이 없는 상황이라면 수동로또를 구매할 수가 없습니다 😄

차라리 사용자로부터 수동로또번호를 다 받은 값과 , LottoPrice를 LottoMarket에게 주고
lottoMarket.buy같은 메소드를 호출하고 나면 로또번호가 담긴 티켓을 반환하는 그런 구조로 간다면
아마 고민하시는 부분이 어느정도 개선이 될 것 같습니다 😄