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 과제제출 #6

Open
wants to merge 11 commits into
base: jaeyeon
Choose a base branch
from

Conversation

jaethewiederholen
Copy link

  • 지금까지 구글링 하며 찔끔씩 익혔던 것들 + 몰랐던 기능들을 전체적으로 다 훑어보고 정리할 수 있었던 과제였던 것 같습니다. 😀 알면 알수록 공부할게 더 많아지는 것..같지만..ㅠㅠㅠ 뭘 더 공부해야 할지 정리된 느낌입니다😊
  • 지금껏 그냥 코드 복붙했던 기본적인 rotation과 transform 의 코드를 자세히 이해하려 하니 조금 어려웠는데 확실히 이해될 수 있어서 좋았습니다.
  • 화이띵..!ㅇvㅇ

- add john lemmon character
- add walking animation
- apply rigid body and collider
add playerMovement script
- set directional light
- add environment
- add navigation mesh
- add virtual camera
- add post processing effects
- add ui background image and exit image
- fading ending image
- add gargoyle character
- add caughtEnding Event
- add moving ghosts
- duplicate enemies
- add audio effect
- adjust gargoyle position
- build
Copy link

@MondaySHB MondaySHB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

복붙하셨던 code를 이해하셨다고 하기에 너무 잘 작성해주신 거 같습니다!!
(git commit message에서 연륜이 묻어 나시네요 ㅎㅎㅎ)

{
m_AudioSource.Stop();
}
Vector3 desiredForward = Vector3.RotateTowards(transform.forward, m_Movement, turnSpeed * Time.deltaTime, 0f);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rotation, transform 코드 이해하시느라 고생 많으셨습니다!!

m_AudioSource = GetComponent<AudioSource>();
}

void FixedUpdate()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

transform, rotate와 같은 물리적 동작에 time value가 들어가면 frame 수를 맞춰주기 위해 fixedUpdate로 구현합니다.

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