Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Commit

Permalink
refactor: (TargetVisualizer) rename trackableTarget -> targetTracker
Browse files Browse the repository at this point in the history
  • Loading branch information
M4lph authored and ManuelRauber committed Jan 21, 2024
1 parent 6d274b3 commit 93477a8
Showing 1 changed file with 12 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ public class SelectedWeaponTargetVisualizer : MonoBehaviour
private GameObject TargetVisualizationPrefab { get; set; } = default!;

private readonly List<GameObject> _visualizations = new();
private ICanTrackTarget? _currentTrackableTarget;
private ICanTrackTarget? _currentTargetTracker;

private void OnEnable()
{
Expand All @@ -40,43 +40,43 @@ 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<ICanTrackTarget>();
var targetTracker = args.Transform.GetComponentInChildren<ICanTrackTarget>();

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);
}
}

private void TryToDetachFromTrackableTarget()
{
RemoveVisualizations();

if (_currentTrackableTarget != null)
if (_currentTargetTracker != null)
{
_currentTrackableTarget.TargetChanged -= TargetChanged;
_currentTargetTracker.TargetChanged -= TargetChanged;
}

_currentTrackableTarget = null;
_currentTargetTracker = null;
}

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);
}
}

Expand Down

0 comments on commit 93477a8

Please sign in to comment.