projectile revamp + scorpion #14

Merged
Garutako merged 3 commits from dev/projectile-revamp into main 2025-07-29 21:39:19 +00:00
Showing only changes of commit d58496a08b - Show all commits

View File

@ -35,10 +35,10 @@ public class Root : MonoBehaviour
Vector3 spawnPos = (_projectileSpawn == null) ? _rigidbodyAlly.position : _projectileSpawn.position;
GameObject _newArrow = Instantiate(_projectile, spawnPos, Quaternion.identity);
//Warning : the Speed of the arrow is equal to the speed of this unit, if this unit need to move, use an other variable !
_newArrow.GetComponent<Projectile>().Damage = _entity.AttackDamage;
_newArrow.GetComponent<Projectile>().Origin = _entity;
var projectile = _newArrow.GetComponent<Projectile>();
projectile.Damage = _entity.AttackDamage;
projectile.Origin = _entity;
Garutako marked this conversation as resolved Outdated

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 = ...

good point, donzo 🫡

good point, donzo 🫡
}
public void PlaySound(string soundName)