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

[2단계 - 블랙잭 베팅] 폰트(김영재) 미션 제출합니다 #893

Open
wants to merge 27 commits into
base: yeong0jae
Choose a base branch
from

Conversation

yeong0jae
Copy link

체크 리스트

  • 미션의 필수 요구사항을 모두 구현했나요?
  • Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
  • 애플리케이션이 정상적으로 실행되나요?
  • 셀프 체크를 작성했나요? 프롤로그

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

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

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

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

MVC 패턴에 묶여서 생각하고 싶지 않아 기존 컨트롤러에서 하던 일을 Application이 하도록 변경했습니다.

집중하고 싶은 부분은 "객체들이 본인의 역할을 자율적으로 수행하고 있는가" 입니다.
더 캡슐화할 수 있는 부분이나 새로운 객체가 담당할 수 있을 부분을 말씀해주시면 감사하겠습니다.
추가로 outputview에 모델이 아닌 데이터만 넘길 수 있도록 개선해보려고 합니다.

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