-
Notifications
You must be signed in to change notification settings - Fork 56
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
[둘리] 2단계 블랙잭 제출합니다. #35
Conversation
피드백을 반영하면서 컨트롤러를 크게 고쳤습니다! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
피드백 잘 반영해주셨네요 👍
마지막으로 재입력 받는 기능을 UI layer로 옮겨보고 끝내보면 좋겠습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지금까지 블랙잭 미션 고생하셨습니다 👏
조금 어려운 피드백도 있었는데 잘 반영해주셨어요 👍
다음 미션도 힘내세요!
안녕하세요, 리뷰어님! 1단계 피드백을 반영하고 2단계 기능을 구현하였습니다.
제가 봐도 아직 코드가 부족하다고 생각하는데, 혼자 고민해도 어려워서 질문드립니다!
다른 크루들의 의견을 들어보니 CardGame에 View의 함수를 인자로 넘겨주었다고 하는데요.
저는 도메인에서 View의 함수를 사용하는 일은 피해야한다는 입장입니다! 도메인에서 View의 함수를 사용하는 것이, 도메인에서는 View에서 오는 함수인지 모른다고 할지라도 MVC에 위배되지 않는건가요?