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

김보령_3.28_DogKnight 과제 제출 #15

Open
wants to merge 5 commits into
base: boryoung
Choose a base branch
from

Conversation

BBO-cmd
Copy link

@BBO-cmd BBO-cmd commented Mar 31, 2022

어려웠던 부분 :
빈칸을 다 채우긴 했지만 역시나 Play가 잘 되지 않네요ㅜ_ㅜ
GameManager에서 TurnNotify()가 처음 실행될 때 Object reference not set to an instance of an object라고 에러가 떠서 _turnHandler(_gameRound, _whoseTurn)가 Character Script의 TurnUpdate와 연결되는 것을 로그 찍어서 확인했는데 뭐가 문제인지 잘 모르겠습니다. 여기서 이 instance of an object가 _turnHandler를 가리키는게 맞나요..?? 그리고 TurnNotify()라는건 _turnHandler를 통해서 Character의 TurnUpdate를 실행시키는 역할을 한다고 생각했는데 여기에서 _whoseTurn을 사전에 업데이트 하는 이유를 잘 모르겠습니다.

태호님이 상세하게 써 주신 코드 워크 플로우를 하나하나 따라가 보았는데 제가 그 의도를 완전히 이해하지 못한 것 같아요ㅠㅅㅠ 그러다 보니 어떤 것들을 맞춰 확인해야 하는지 잘 감이 오지 않습니다 흑흑
그리고 혹시 Player와 Enemy Script이름이 서로 바뀐건가요.?? my_Damage는 내가 상대방에게 입히는 데미지가 맞나요..?

이번주에 더 고민해보고 보충하겠습니다!!

개선되었으면 하는 점: 너무 예시까지 잘 정리해 주셔서 잘 봤습니다ㅎㅎ그런데 과제를 아마 쉽게 따라해 보라고 스켈레톤 코드를 주신 것 같은데 제가 태호님의 의도를 정확하게 파악하는 수준이 안되다 보니 오히려 그냥 구현하는 것보다 더 어렵게 느껴지는 것 같기도 해요..ㅜㅜ 그래도 과제 하면서 여러가지로 많이 생각을 해 볼 수 있었습니다. 감사합니다!

@BBO-cmd
Copy link
Author

BBO-cmd commented Apr 1, 2022

Final 로 애니메이션 제외하고 완성해서 커밋했습니다!
혼자서 최대한 구현해보려 했지만 결국 예시코드를 참고하였습니다ㅜㅜ
처음에는 예시코드를 최소한으로 보고 제 코드를 최대한 살려보려 했지만 분명 똑같은 의미의 코드인데 제 코드를 유지하면 정상적으로 play 되지 않다가 예시코드대로 쓰면 play되는 슬픈 경험을 많이 하였습니다.
완전히 바꾼 부분에는 원래 제 코드를 주석으로 달아놓았습니다!

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