-
Notifications
You must be signed in to change notification settings - Fork 2
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
Dahyeonryu #20
base: RyuDaHun
Are you sure you want to change the base?
Dahyeonryu #20
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드들이 전체적으로 간단하지만 깔끔하게 잘 짜여진거 같네요.
접근 지정자 지정과 더불어 몇가지들만 고치면 더욱 완성도가 높아질거 같습니다.
2주차 과제 고생하셨습니다.
Assets/EnemySpawner.cs
Outdated
} | ||
|
||
// �ڷ�ƾ�� ����Ͽ� while��ó�� �������� ���� ���� | ||
IEnumerator SpawnEnemies() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코루틴 이용해서 깔끔하게 잘 구현하셨네요
Assets/Missile.cs
Outdated
public float speed = 10f; | ||
public int num = 0; | ||
// Start is called before the first frame update | ||
void Start() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
안쓰는 함수는 지워주시면 좋을거 같습니다.
Assets/PlayerController.cs
Outdated
float moveHorizontal = Input.GetAxis("Horizontal"); | ||
float moveVertical = Input.GetAxis("Vertical"); | ||
Vector2 movement = new Vector2(moveHorizontal, moveVertical); | ||
transform.Translate(movement * moveSpeed * Time.deltaTime); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
플레이어가 움직이는 코드들을 하나의 함수로 묶어서, Update함수에서는 PlayerMove와 같은 하나의 함수만 작성하는게 더 깔끔한 코드가 될거 같습니다.
Assets/PlayerController.cs
Outdated
} | ||
|
||
|
||
void OnTriggerEnter2D(Collider2D other) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
현재 플레이어가 경험치 구슬을 먹으면 사라지는 코드가 없는거 같은데 의도된 것인지 궁금하네요.
그리고 스크립트들 한 폴더에 정리해주시면 감사하겠습니다.. |
코멘트 확인 수정 하고 코멘트 달아주시면 merge하겠습니다~ |
변경점 👍
새로 구현한 기능 및 주요 변경점
버그 해결 💊
해결한 버그
리팩토링 🛠
리팩토링 내역
스크린샷 🖼
변경된 부분에 대한 스크린샷
비고 ✏
리뷰어에게 전하는 말 등
사용하지 않은 항목은 모두 지워주세요.