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

YuJeongMin, 21.12.09 #12

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

YuJeongMin, 21.12.09 #12

wants to merge 1 commit into from

Conversation

tyrosine1153
Copy link
Contributor

@tyrosine1153 tyrosine1153 commented Dec 9, 2021

코드 리뷰 요청 211209

외부 팀 프로젝트에서 개발하며 작성 중인 코드입니다.
미처 다 구현되지 않아 혹시 읽는게 난해할 수 있는 점 양해 부탁 드립니다. 구현되지 않은 부분은 // Todo : 로 명시해두었습니다.
캐릭터 동작 관리를 위해서 FSM을 조사해 적용해 봤는데요. 전체적인 개념은 알고 있지만 찾았던 예제에서 동작상의 오류...개선할 점이 있다고 판단해 일단 수정을 하면서 적용을 했습니다. 그런데 수정한 부분이 사실 구현 의도에서 벗어난 게 아닌가 하는 걱정도 생기고 공부를 하면서 배운 내용과 크게 다르지 않지만 놓친 부분이 있지는 않을까 생각이 들어서 다른 사람들에게 보이는게 좋겠다 생각했습니다.
첨부한 스크립트 에서 고쳐야 할 점이 있다면 지적과 올바른 경우에 대한 지시를 주시면 감사하겠습니다.

클래스 설명

  • CharacterControl : partial 클래스로 클래스 동명의 파일과 CharacterState라는 이름으로 나누었습니다.
    캐릭터 상태와 이동, 충돌같이 캐릭터의 정보와 동작을 관리하는 클래스입니다.
    CharacterState에서는 사용할 State들을 정의만 하고 CharacterControl에서 State들을 사용해
    매 프레임마다 state를 갱신하고 클래스에서 필요한 동작을 합니다.

감사합니다.

@tyrosine1153 tyrosine1153 changed the title YuJeongMin, 21.10.18 YuJeongMin, 21.12.09 Dec 9, 2021
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