studio/idle-survivors
2
0
Fork 0
Code Issues 14 Pull Requests 1 Packages Projects Releases Wiki Activity

WIP: enemy behavior #26

Draft
zephyr wants to merge 3 commits from feature-enemy-behavior into main
pull from: feature-enemy-behavior
merge into: studio:main
Conversation 1 Commits 3 Files Changed 2 +4 -8
2 changed files with 4 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ec2ede72ed - Show all commits

10
Assets/Scripts/Enemy.cs
View File

@@ -131,7 +131,7 @@ namespace DefaultNamespace
private void Retarget()
{
if (heroSet == null || heroSet.Count == 0)
if (heroSet.IsEmpty)
zephyr marked this conversation as resolved Outdated
madxmike commented 2023-08-16 17:17:32 -04:00
Outdated
Review
Copy Link

heroSet.IsEmpty

`heroSet.IsEmpty`
{
// No heroes to target.
// TODO(zeph): switch to a special behavior?
@@ -140,13 +140,7 @@ namespace DefaultNamespace
}
// TODO(zeph): target based on threat, once threat exists
var k = Random.Range(0, heroSet.Count);
var it = heroSet.GetEnumerator();
// This seems to be the best way to do this...?
for (var i = 0; i < k; i++)
{
it.MoveNext();
}
target = it.Current.gameObject;
target = heroSet[k].gameObject;
zephyr marked this conversation as resolved Outdated
madxmike commented 2023-08-16 17:15:43 -04:00
Outdated
Review
Copy Link

We probably just want to implement https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/indexers/using-indexers on RuntimeSet

We probably just want to implement https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/indexers/using-indexers on RuntimeSet
}
private void SetBehavior(BehaviorState behavior)

2
Assets/Scripts/RuntimeSet/RuntimeSetSO.cs
View File

@@ -37,5 +37,7 @@ namespace RuntimeSet
{
return GetEnumerator();
}
public T this[int index] { get { return items[index]; } }
}
}