Skip to content

[SCG] 권승원 로또 1-2단계 미션 제출합니다.#162

Open
zest-sw wants to merge 7 commits intonext-step:mainfrom
zest-sw:main
Open

[SCG] 권승원 로또 1-2단계 미션 제출합니다.#162
zest-sw wants to merge 7 commits intonext-step:mainfrom
zest-sw:main

Conversation

@zest-sw
Copy link
Copy Markdown

@zest-sw zest-sw commented Apr 2, 2026

인사

안녕하세요! 제용님. 이번 미션 완료하여 pr 리뷰 부탁드립니다.
저는 자바는 잘 안 써봐서 미숙하긴 하나 이번 기회를 통해서 많이 알아가는 것 같습니다.
이번에 페어 프로그래밍하면서 색다른 경험을 한 것 같네요!

고민했던 부분

  • 로또 티켓 객체가 특별한 메서드를 제공하지 않아서 그냥 컬렉션을 사용하는 것과 비슷한 것 같은 느낌이 들었습니다.
  • 코드에서 tickets를 그냥 리스트로 받아서 tickets.get(0).getTicketNumbers()와 같이 사용되어서 가독성이 좀 떨어진다고 느꼈습니다.
  • Statisitc 객체를 테스트 할 때, util/sequenceGenerator 를 사용했는데 티켓이 2장 이상이 되면 사용이 직관적이지 않다고 느꼈습니다.
  • 코드 중간중간에 List로 그냥 쓸 때가 있고, 이번 미션의 일급 컬렉션 조건을 충족을 위해 ticket은 일급 컬렉션으로 구성했는데, 모두 다 일급 컬렉션으로 만드는 것이 나을까요? 오히려 복잡성만 증가하는 걸까요?

잘 부탁드립니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant