Get rid of event system

Zomis / Games

Many kinds of games, playable in both multiplayer or local play. Replays, spectator mode, and AIs.

https://games.zomis.net

MIT License

17 stars 5 forks source link

Open Zomis opened 4 years ago

Zomis commented 4 years ago

In retrospect, using a flexible event system was not a good idea.

Using a dynamic event-driven approach for everything...

makes code very unreadable and confusing
makes it difficult to figure out or troubleshoot "when A happens, B will happen"
is still an interesting concept, somehow specifying "business requirements" in code is still intriguing to me (No I don't mean Cucumber. In the actual application, not just in tests)

Zomis commented 2 years ago

Should be easier with The Big Refactoring™ #302

Zomis commented 2 years ago

Using Koin can be an option: https://insert-koin.io/

Zomis commented 1 year ago

Using SharedFlows would be similar to an event system but would be more Kotlin-ic. And would make supporting multiple coroutines/threads easier.