Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Step2: 블랙잭 #567

Open
wants to merge 22 commits into
base: ggam-nyang
Choose a base branch
from
Open

Step2: 블랙잭 #567

wants to merge 22 commits into from

Conversation

ggam-nyang
Copy link

@ggam-nyang ggam-nyang commented Jul 14, 2023

많이 늦어져버렸네요 ㅠㅠ

  • Input 하나가 클라이언트의 API 요청이라 생각했습니다. 때문에 결과값을 반환하고 클라이언트(main)에서 출력을 하고자 했습니다.
    그런데 blackjackGame.start 메서드에서 Output, Input 로직을 호출하는 것이 어색한데, 분리가 잘 안되더라구요.. 😢
    설계의 문제인지, 혹은 좋은 방법이 있는지 궁금합니다!

ps. 공식 리뷰 요청 기간이 지난것으로 알고 있습니다! 남은 미션도 완료하고자 하는데, PR을 요청드려도 되는건지 잘 모르겠습니다..!
리뷰 요청이 불가능한 것이라면 말씀해주세요!

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.

None yet

1 participant