Refactor event handling and improve code formatting

[tokKurumi]

TODO: implement all listeners and events

[tokKurumi]

I gonna create separate project with source gen to analyze all Listeners in code to remove manual resolving

[tokKurumi]

Need a source gen

[oylsister]

what source gen?

[tokKurumi]

The thing that allows to write code which generator another code. Here I want to implement custom attribute [Listener] or [Event] and mark class with it:

[Event]
public class PlayerHurtEvent : EventBase<EventPlayerHurt>
{
    public override GameEventHandler<EventPlayerHurt> Handler => OnPlayerHurt;

    private HookResult OnPlayerHurt(EventPlayerHurt @event, GameEventInfo info)
    {
        Console.WriteLine($"Player {@event.Userid} hurt with {@event.DmgHealth} damage.");
        return HookResult.Continue;
    }
}

then it should analyze code and take each class with attribute [Event]. Then we have a collection of these classes and we can use them with existing:

public static class EventsExtensions
{
-   private static readonly List<IEventBase> _events = [
-      new PlayerHurtEvent(),
-      // other events
-   ];
+   private static readonly List<IEventBase> _events = SourceGenerators.Events; // collection of events provided by source gen

    public static void RegisterAllEventHandlers(this BasePlugin plugin)
    {
        foreach (var @event in _events)
        {
            @event.Register(plugin);
        }
    }

    public static void RemoveAllEventHandlers(this BasePlugin plugin)
    {
        foreach (var eventInstance in _events)
        {
            eventInstance.Deregister(plugin);
        }
    }
}

[tokKurumi]

The output of source gen should be

public static partial class SourceGenerators {
    public static readonly List<IEventBase> = [
        // all events here which were created by source generator, but not manual resolving
    ];
}

[oylsister]

There is a bunch of code has been implemented recently, you probably have to sync them to latest one. Anyway, does your implementation work well when trying to print message from event and delegate?

[tokKurumi]

There is a bunch of code has been implemented recently, you probably have to sync them to latest one. Anyway, does your implementation work well when trying to print message from event and delegate?

I did not test it yet. But my guesses it should improve code read ability and it will remove not necessary class Events

[tokKurumi]

There is a bunch of code has been implemented recently, you probably have to sync them to latest one.

Yeah, this is the main problem, the second is source gen. I do it in this weekend