From 93477a8549ac5138a738dce28584f8ca71f83cfc Mon Sep 17 00:00:00 2001 From: M4lph Date: Sun, 21 Jan 2024 17:32:47 +0100 Subject: [PATCH] refactor: (TargetVisualizer) rename trackableTarget -> targetTracker --- .../SelectedWeaponTargetVisualizer.cs | 24 +++++++++---------- 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/FairyTaleDefender/Assets/_Game/Scripts/Runtime/Entities/Weapons/Targeting/SelectedWeaponTargetVisualizer.cs b/FairyTaleDefender/Assets/_Game/Scripts/Runtime/Entities/Weapons/Targeting/SelectedWeaponTargetVisualizer.cs index 3bd276fed..c8c998750 100644 --- a/FairyTaleDefender/Assets/_Game/Scripts/Runtime/Entities/Weapons/Targeting/SelectedWeaponTargetVisualizer.cs +++ b/FairyTaleDefender/Assets/_Game/Scripts/Runtime/Entities/Weapons/Targeting/SelectedWeaponTargetVisualizer.cs @@ -20,7 +20,7 @@ public class SelectedWeaponTargetVisualizer : MonoBehaviour private GameObject TargetVisualizationPrefab { get; set; } = default!; private readonly List _visualizations = new(); - private ICanTrackTarget? _currentTrackableTarget; + private ICanTrackTarget? _currentTargetTracker; private void OnEnable() { @@ -40,20 +40,20 @@ private void TryToAttachToTrackableTarget(WeaponSelectedEventChannelSO.EventArgs // If we select one tower and then another tower, there won't be a deselected event, but only a selected one. RemoveVisualizations(); - var trackableTarget = args.Transform.GetComponentInChildren(); + var targetTracker = args.Transform.GetComponentInChildren(); - if (trackableTarget is null) + if (targetTracker is null) { return; } - _currentTrackableTarget = trackableTarget; - _currentTrackableTarget.TargetChanged += TargetChanged; + _currentTargetTracker = targetTracker; + _currentTargetTracker.TargetChanged += TargetChanged; - if (trackableTarget.Target.Exists()) + if (targetTracker.Target.Exists()) { - AddVisualizations(trackableTarget.Target); + AddVisualizations(targetTracker.Target); } } @@ -61,12 +61,12 @@ private void TryToDetachFromTrackableTarget() { RemoveVisualizations(); - if (_currentTrackableTarget != null) + if (_currentTargetTracker != null) { - _currentTrackableTarget.TargetChanged -= TargetChanged; + _currentTargetTracker.TargetChanged -= TargetChanged; } - _currentTrackableTarget = null; + _currentTargetTracker = null; } private void TargetChanged() @@ -74,9 +74,9 @@ private void TargetChanged() RemoveVisualizations(); // It could be that we're changing the target, but it was just destroyed. - if (_currentTrackableTarget != null && _currentTrackableTarget.Target.Exists()) + if (_currentTargetTracker != null && _currentTargetTracker.Target.Exists()) { - AddVisualizations(_currentTrackableTarget.Target); + AddVisualizations(_currentTargetTracker.Target); } }