[2단계 - 블랙잭 베팅] 포스티(윤강민) 미션 제출합니다

[ykmxxi]

안녕하세요 아서 😄
step2 리뷰 요청 드립니다.

체크 리스트

• 미션의 필수 요구사항을 모두 구현했나요?
• Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
• 애플리케이션이 정상적으로 실행되나요?
• 프롤로그에 셀프 체크를 작성했나요?
  • 1단계 블랙잭 게임 프롤로그(포스티)

객체지향 생활체조 요구사항을 얼마나 잘 충족했다고 생각하시나요?

1~5점 중에서 선택해주세요.

• 1 (전혀 충족하지 못함)
• 2
• 3 (보통)
• 4
• 5 (완벽하게 충족)

선택한 점수의 이유를 적어주세요.

step2를 진행하기 전 블랙잭 룰을 찾아보는 시간을 가졌습니다. 이전 step1 때 페어와 함께 진행하면서 잘못된 블랙잭에 관한 지식으로 생겨난 로직들이 다수 존재해(승패 계산), 이 부분을 찾아보고 수정하는데 많은 시간이 걸렸습니다.

일단 정상적인 기능 수행으로 돌리기 위해 책임 분리, 객체 지향 생활 체조 원칙 등을 뒤로 하고 먼저 잘못된 로직들을 수정해 step1 보다 더 지키지 못한 것 같습니다.

이러한 점들까지 지켜가면서 구현을 진행하면 너무나 오래 걸릴 것 같아 먼저 리뷰 요청 드리는 점 양해 바랍니다. 아서 😢

어떤 부분에 집중하여 리뷰해야 할까요?

step2 첫 리뷰 요청은 기능 완성에 먼저 의의를 두고 진행했습니다. 이번 요청 이후 피드백을 주시면 커멘트 남겨주신 부분에 대해 고민을 진행한 후 반영을 하고, 아래에 정리한 사항들을 리팩터링 해 다시 요청 드릴 예정입니다.

• 테스트 코드 작성: 몇몇 테스트 코드에서 가짜 객체를 사용하고 있는 상황이 있어 책임 분리가 쉽지 않은 상황이라 너무 오래 시간이 걸릴 것 같아 진행하지 못하고 있습니다
• 책임 분리: BlackJackRule, BlackJackGame의 책임에 대해 다시 생각해 볼 여지가 있는 것 같습니다. 게임에 참여하는 각각의 플레이어들에게, 카드들에게 분리 시킬 수 있는지에 대해 좀 더 생각해보고, 생각이 바뀐다면 반영해 볼 생각입니다

[ykmxxi]

• 인스턴스 변수가 세 개 이상인 클래스가 됐습니다. 너무 큰 책임을 가지고 있지 않나? 라는 생각이 들어 다음 리뷰 요청 때 좀 더 작은 책임들로 나눈 설계를 만들어볼 생각입니다