-
Notifications
You must be signed in to change notification settings - Fork 38
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
[충남대 Android_전영주] 미션 제출합니다. #33
Open
aengzu
wants to merge
27
commits into
next-step:main
Choose a base branch
from
aengzu:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
aengzu
commented
Jun 9, 2024
•
edited
Loading
edited
- MVC로 분리하고 싶어서 Model 클래스를 만들었는데 코드를 짜다보니 GameModel 을 모델로 분류하는 게 맞는가 하는 생각이 들었습니다. 분리한다면 모델 클래스로 어디까지 가져가야할 지 어려웠습니다. 저는 아직 MVC 패턴에 미숙하여 게임 모델이 현재 플레이어, 승리자, 보드판 상태 정보를 가지고 있고 이에 대한 기능들을 가지게 되었는데 MVC 패턴에서 벗어나게 된 것 같습니다.. 모델 클래스가 기능을 가질 수도 있는건 지 모르겠습니다. 이름을 GameManager 같은 이름으로 바꿔야하나 고민했으나 일단 따로 변경하지는 않았습니다.
- 싱글톤 패턴이라는 걸 알게되어 object 로 하나의 게임의 정보(현재 플레이어와 같은) 를 글로벌하게 사용했는데 object 가 아닌 일반 클래스를 사용해서 구현할 수도 있는 건지 궁금했습니다.
- 강의에서 배운 개념들을 활용하고 싶어서 drawable, fragment, intent 를 써먹고자 화면을 나누는 등의 부가적인 요소를 추가하였는데, 이렇게 하다 보니 코드가 더 복잡해진 것 같습니다 🥲
- 이번 실습은 Git 특강 이후에 진행되었는데 저번 실습보다 확실히 깃허브에 익숙해진 것 같습니다! (아직 많이 부족하지만)
- UI 관련하여 테스트 코드를 작성하고자 했으나 기존 외부 라이브러리를 가져와야하는 것 같아 진행하지 못하였습니다. UI 관련 테스트는 어떻게 진행하는 지 궁금합니다.
- 게임 리셋 기능 추가 - BoardActivity에서 게임 리셋 시 보드 초기화 및 턴 이미지 업데이트 - GameBoardFragment에서 보드의 이미지 초기화 기능
- 애니메이션 및 소리를 추가하려고 있으나 따로 리소스 파일을 업로드 해야하는 것 같아 제외시켰습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.