Step3 - 지뢰 찾기(게임 실행) #421
Open
Step3 - 지뢰 찾기(게임 실행) #421
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
chore : position x, y 순서 변경
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.
Cell 객체에 메시지 전달 피드백 반영하였습니다.
MinesWeeperGame.kt의 startGame()에 불필요해 보이는 건 없지만 너무 많은 일을 하는 것 같습니다. 해당 부분 리펙토링에 관한 조언 부탁드리겠습니다.
Board.kt의 init에서 보드 생성과 지뢰 설치 및 주변 지뢰 정보를 세팅하고있습니다. 보드 자체가 2차원 배열이고 한단계의 들여쓰기만 하려다 보니 너무 작게 나누었나 하는 생각이 듭니다. 해당 부분에 대해 불필요한 부분이 있는지 혹은 개선해야할 부분이 있는지 봐주시면 감사하겠습니다.
게임 진행에 관해서 테스트 코드를 어떤 식으로 구성해야하는지 감이 잡히질 않습니다.
이 외에도 제가 파악하지 못한 부분 있다면 조언 부탁드리겠습니다.
감사합니다.