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

오혜원_JohnLemon 과제 제출 #25

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

Conversation

hyewon0h
Copy link

이런 고퀄의 자료 만들어 주시느라 시니어들, 운영팀 수고 많으셨습니다! 다음부턴 좀 더 미리미리해서 깔끔하게 커밋하겠습니다...!

유니티랑 c#이 낯설었는데 의외로 유니티 같은 경우 튜토리얼 시나리오를 따라가다보니 조금은 익히게 된 것 같았고, c#은 더 친해져야 할 것 같습니다.!! 틈틈히 노력하겠습니다

운영팀 화이팅...!

@hyewon0h hyewon0h changed the title Hyewon 오혜원_JohnLemon 과제 제출 Mar 28, 2022
@ahnjinmo
Copy link

ahnjinmo commented Mar 31, 2022

혜원님 과제 잘 봤습니다! 음 과제가 제대로 완성되지 않은 부분들이 많은 것 같은데, 혹시 완성된 버전을 커밋하지 못하신 건가요? 일단 제일 최신 커밋을 기준으로 파악 후 완성되지 않은 부분들을 정리했습니다. 혹시 실제로 끝까지 완성하지 못하신 것이라면 아래의 부족한 부분들 확인하신 후 마무리해주시면 좋을 것 같습니다 :)

  • Caught, Escape 효과음이 플레이 시 바로 재생됩니다. 원래는 각각 플레이어가 가고일에게 잡혔을 때, 그리고 게임을 끝냈을 때 플레이 되어야 합니다. 이는 각각 게임오브젝트에 붙은 AudioSource 컴포넌트의 'Play On Awake' 옵션이 활성화되어 있어 플레이 시 곧바로 재생되기 때문입니다.

  • Audio Listener 컴포넌트가 씬 내에 하나만 존재해야 하지만, 두 개가 존재하고 있습니다. 그래서 플레이 시 콘솔 창에 해당 경고가 뜨고 있습니다. Main Camera와 John Lemon 각각에 하나씩 붙어있는데, 둘 중 하나를 지워야합니다. 왜 하나만 있어야 하는지는 직접 찾아보시면 도움 되실 것 같습니다!

  • 캐릭터 컨트롤러가 제대로 설정되어 있지 않아, 플레이어가 키보드를 입력해도 존 레몬 캐릭터가 움직이지 않습니다. PlayerMovement 스크립트는 작성되어 있는 건 확인했는데, 정작 씬에 있는 존 레몬 오브젝트에는 Rigidbody, Collider, PlayerMovement 컴포넌트가 없어서 캐릭터가 움직이지 않습니다.

  • 존 레몬 캐릭터 컨트롤러를 제대로 설정한 후 캐릭터를 이동시켜도, 생성한 Virtual Camera가 캐릭터를 따라 이동하지 않습니다. 이는 씬 내에서 Virtual Camera를 실제로 사용할 Cinemachine Brain 컴포넌트가 붙은 유니티 카메라 오브젝트가 없기 때문입니다. MainCamera에 Cinemachine Brain 컴포넌트를 추가하면 정상적으로 캐릭터를 따라갑니다.

  • 플레이 중 가고일에게 들켰을 때, 그리고 게임이 끝났을 때 그림이 보이지 않습니다. 이는 각각의 BackGround Canvas의 하위에 있는 ExitImage, CaughtiImage UI의 RectTransform의 크기가 제대로 설정이 되어있지 않아서 그런 것으로 파악됩니다. 그림이 제대로 보일 수 있도록 RectTransform을 수정해야 합니다.

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.

2 participants