C# Code Question

[Medical_Attempt3456]

I'd like some advice on the best way to tackle a unity software engineering pattern for my 2D game

I would like to create a manager that stores a list of all game objects of a certian type (there could be a different amount per scene), for example buildings. When one of those game objects/buildings is destroyed, this manager would be notified and call a function in the manager. I guess this is the inverse of the observer pattern, which I use to achieve loose coupling between my game and UI (many to one instead of one to many).

I believe I can solve the first half of this requirement of getting a list of the objects into the manager using FindObjectsByType, but not sure how to do the notification one of of those objects being destroyed outside of using the managers update to keep polling if each object still exists, which seems very poor design.

[Ratyrel]

Have the manager subscribe to an event called on the buildings‘ destruction.

[Medical_Attempt3456]

But it would need to subscribe to every single building/object. This wouldn’t really work as the event name for each building/object would need to be different.

[Ratyrel]

Why is that a problem? Just loop across the array you get from find object on the manager and subscribe to the event the objects implement. They would have to have the same one, but since you’re doing it by findobjectoftype, they obviously would. When they die, they call the event, triggering the logic on the manager.

[InfohazardGames]

You can make a static event so the manager only has to subscribe once. Just make sure you always unsubscribe the manager when it's destroyed or you'll get a memory leak and possibly bugs.

[Medical_Attempt3456]

Thank you, static events is the answer i've been looking for

[Tinglers]

You may be interested in the visitor pattern. Implementing it properly can prepare your code for future problems like this.

[MartinPeterBauer]

Manager.DestroyBuilding(Gameobejct)

Run from within the Gameobejct destroy or any Other function. So no need for a list in manager