EverestAPI / EverestAPI/Everest

Deprecate EntityList.FindFirst/FindAll

Open
#1,039 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C#
Stars
516
Forks
106
Avg merge
4d 9h
Merged PRs (30d)
5

Description

### I am planning on implementing this myself

No

### Describe your request

These methods are a beginner trap that can end up severely affecting performance in large rooms, especially as part of hooks.
Since it's possible to dynamically add types to the Tracker now, that solution should be preferred.

Potential obsoletion message:
`[Obsolete("O(n) operation, use Scene.Tracker.GetEntity instead. (requires [Tracked] attribute on T. If T is not a type you own and it does not have the [Tracked] attribute, use Tracker.AddTypeToTracker on that type first.)")]`

### Additional context

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.