Skip to content

Deprecate EntityList.FindFirst/FindAll #1039

@JaThePlayer

Description

@JaThePlayer

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<T> 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

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions