First iteration refactor events with generics

Author: danigfedev

Runtime/SOBaseEvents/Refactor.meta

@@ -0,0 +1,29 @@
+using System;
+using SOBaseEvents.Refactor.Impl;
+using UnityEngine;
+
+namespace SOBaseEvents.Refactor
+{
+    public abstract class EventListener<TEvent, TArg> : EventListenerBase where TEvent : EventSOBase<TArg>
+    {
+        public event Action<TArg> OnEventRaised;
+        
+        protected TEvent _typedEvent => _baseEvent as TEvent;
+
+        private void OnEnable()
+        {
+            if (_typedEvent != null)
+            {
+                _typedEvent.AddListener(OnEventRaised);
+            }
+        }
+
+        private void OnDisable()
+        {
+            if (_typedEvent != null)
+            {
+                _typedEvent.RemoveListener(OnEventRaised);
+            }
+        }
+    }
+}

Runtime/SOBaseEvents/Refactor/EventListener.cs

@@ -0,0 +1,60 @@
+using System;
+using System.Collections.Generic;
+using UnityEngine;
+
+namespace SOBaseEvents.Refactor
+{
+    public abstract class EventSOBase<TArg> : ScriptableObject, ISOEventBase, ISOEventRegistry<TArg>, ISOEventRaiser<TArg>
+    {
+        private readonly List<Action<TArg>> _listeners = new();
+        
+        public void AddListener(Action<TArg> listener)
+        {
+            if (_listeners.Contains(listener))
+            {
+                var (objectName, methodName) = GetListenerInfo(listener);
+                Debug.LogError($"[ScriptableObjectEvents] Listener {methodName} of GameObject {objectName} already registered. Aborting registration.");
+                return;
+            }
+            
+            _listeners.Add(listener);
+        }
+
+        public void RemoveListener(Action<TArg> listener)
+        {
+            if (!_listeners.Contains(listener))
+            {
+                var (objectName, methodName) = GetListenerInfo(listener);
+                Debug.LogError($"[ScriptableObjectEvents] Listener {methodName} of GameObject {objectName} is not registered. Aborting removal.");
+                return;
+            }
+
+            _listeners.Remove(listener);
+        }
+
+        public void RaiseEvent(TArg arg)
+        {
+            for (int i = _listeners.Count - 1; i >= 0; i--)
+            {
+                _listeners[i]?.Invoke(arg);
+            }
+        }
+        
+        private (string objectName, string methodName) GetListenerInfo(Action<TArg> listener)
+        {
+            var objectName = string.Empty;
+            var methodName = listener.Method.Name;
+                
+            if (listener.Target is MonoBehaviour mb)
+            {
+                objectName = mb.gameObject.name;
+            }
+            else if (listener.Target != null)
+            {
+                objectName = listener.Target.ToString();
+            }
+
+            return (objectName, methodName);
+        }
+    }
+}

Runtime/SOBaseEvents/Refactor/EventListener.cs.meta

@@ -0,0 +1,7 @@
+namespace SOBaseEvents.Refactor
+{
+    public interface ISOEventBase
+    {
+        
+    }
+}

Runtime/SOBaseEvents/Refactor/EventSOBase.cs

@@ -0,0 +1,9 @@
+namespace SOBaseEvents.Refactor
+{
+    //TODO This has to be made with a template if I want to support more Args dynamically
+    
+    public interface ISOEventRaiser<TEventArg>
+    {
+        void RaiseEvent(TEventArg arg);
+    }
+}

Runtime/SOBaseEvents/Refactor/EventSOBase.cs.meta

@@ -0,0 +1,13 @@
+using System;
+
+namespace SOBaseEvents.Refactor
+{
+    //TODO This has to be made with a template if I want to support more Args dynamically
+    //public interface IEventRegistry<TEventArg1, TEventArg2, TEventArg3>
+    
+    public interface ISOEventRegistry<TEventArg>
+    {
+        void AddListener(Action<TEventArg> listener);
+        void RemoveListener(Action<TEventArg> listener);
+    }
+}