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

Open
wants to merge 9 commits into
base: minwoo
Choose a base branch
from

Conversation

NEOCLID
Copy link

@NEOCLID NEOCLID commented Mar 26, 2022

  • 느낀 점
    우선, 이렇게 잘 나와 있음에도 헤매는 자신을 보며 답답함을 많이 느꼈고, 한편으로는 가이드라인이 없는 상황에서 진행되는 창조적 프로젝트의 일원이 되는 것이 두렵기도 하면서 설렜습니다. 완성하고 나니 별것없는 게임 같았으나 그 만드는 과정이 상당히 어려움을 관찰하며 앞으로의 활동에 더욱 진중히 임해 실력을 상당히 향상시켜야겠다는 다짐을 다졌습니다.

  • 어려웠던 점
    구조에 대한 이해가 잘 안 되는 부분이 많았습니다. 예컨대 튜토리얼에서 시키는대로 하면서도 이해하지 못한 부분들이 꽤 있습니다. (주로는 코드보다는 unity 자체에 대한 이야기였습니다.) 사실 저는 뭐든 자세히 알수록 잘 다루고, 그 영향을 매우 크게 받는 편이라, 앞으로 많이 나아질 부분인 것 같긴 합니다.

  • 하고 싶은 말
    리뷰어 설정이 안 됩니다.
    image
    열심히 찾아봤으나 잘 되지 않아서 방법을 찾으면 그때 수정하겠습니다.
    이제 C# 공부를 해 보겠습니다..! 플레이 영상도 첨부합니다. (인코딩해서 다시 첨부하겠습니다.)

NEOCLID added 9 commits March 25, 2022 11:17
Why is it going up..?
Step1 completed (before scripting)
He's moving Well..!
Character done
but, something is wrong i think.
no camera effects included
left: from Static.
before attaching UI
before putting musical effects
I made it..!!!
@NEOCLID NEOCLID changed the base branch from main to minwoo March 26, 2022 06:29
@brainini brainini requested review from ahnjinmo and MondaySHB March 26, 2022 08:16
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.

Update 대신, FixedUpdate를 잘 구현해 주셨습니다!!


public class WaypointPatrol : MonoBehaviour
{
public NavMeshAgent navMeshAgent;

Choose a reason for hiding this comment

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

https://docs.unity3d.com/kr/530/ScriptReference/NavMeshAgent.html 에서 보다 더 NavMeshAgent의 기능을 보다 풍성하게 사용할 수 있습니다.

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