장영웅_3.28_DogKnight 과제 제출#13
Open
See-Y wants to merge 9 commits into
Open
Conversation
See-Y
commented
Mar 31, 2022
- 과제하면서 어려웠던 부분
- 처음 배워보는 delegate와 옵저버를 사용하기 위해 코드를 분석하는 것이 까다로웠다.
- 스크립트들과 그 안 메소들 간의 실행 순서를 알기 어려워 코드 진행을 예상하는 것이 까다로웠다.
- 주어진 코드 안에서 어디서 RoundNotify함수가 버튼을 누를때마다 실행이 되는지 알 수 없어서 전체적으로 이해하는데 살짝 어려움을 겪었다.
- 개선되었으면 하는 점
- 개선점이라기보다는 헷갈렸던 점: 어느 스크립트의 Awake()가 먼저 실행이 되는지, 그 Awake()들은 일렬적으로 실행되는지 아니면 병렬적으로 실행되는지 헷갈려서 싱글톤을 위해 Awake함수를 게임매니저에 작성할 때 고민이 되었습니다.
This file contains hidden or 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
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.