Conjure/gather-and-defend
9
0
Fork 0
Code Issues Pull Requests 1 Projects Wiki Activity

projectile revamp + scorpion #14

Merged
Garutako merged 3 commits from dev/projectile-revamp into main 2025-07-29 21:39:19 +00:00
Conversation 1 Commits 3 Files Changed 21 +690 -277
Garutako commented 2025-07-29 01:55:30 +00:00
Owner
Copy Link
No description provided.
Garutako added 2 commits 2025-07-29 01:55:30 +00:00
EliaGingras1 approved these changes 2025-07-29 13:35:07 +00:00
EliaGingras1 left a comment
Owner
Copy Link

Good job ! 👏👏

Good job ! 👏👏
Assets/Scripts/Root.cs Outdated
@ -43,2 +40,2 @@
_newArrow.GetComponent<Projectile>().VectorEnd = _rigidbodyOpponent.position;
_newArrow.GetComponent<Projectile>().Target = _entity.Enemy;
_newArrow.GetComponent<Projectile>().Damage = _entity.AttackDamage;
_newArrow.GetComponent<Projectile>().Origin = _entity;
EliaGingras1 commented 2025-07-29 13:34:38 +00:00
Owner
Copy Link

Oh my, je viens de voir que le code avait genre 5 call à GetComponent() avant. L'idéal serait de cache le GetComponent() dans une variable temporaire dès qu'il est appelé plus qu'une fois puisque c'est lourd, mais c'est déjà moins pire avec juste 2 appel. Tu peux changer ça si tu veux.

var projectile = GetComponent()l
projectile.Damage = ...

Oh my, je viens de voir que le code avait genre 5 call à GetComponent() avant. L'idéal serait de cache le GetComponent() dans une variable temporaire dès qu'il est appelé plus qu'une fois puisque c'est lourd, mais c'est déjà moins pire avec juste 2 appel. Tu peux changer ça si tu veux. var projectile = GetComponent<Projectile>()l projectile.Damage = ...
Garutako commented 2025-07-29 21:37:20 +00:00
Author
Owner
Copy Link

good point, donzo 🫡

good point, donzo 🫡
Garutako marked this conversation as resolved
Garutako added 1 commit 2025-07-29 21:36:08 +00:00
Garutako merged commit 39bffa8c59 into main 2025-07-29 21:39:19 +00:00
Garutako deleted branch dev/projectile-revamp 2025-07-29 21:39:19 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
EliaGingras1
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Conjure/gather-and-defend#14
No description provided.
Delete Branch "dev/projectile-revamp"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?