make use of Godot.Timer and _Ready()

Author: valkyrienyanko

Scenes/Main.tscn

@@ -1,11 +1,12 @@
-[gd_scene load_steps=7 format=2]
+[gd_scene load_steps=8 format=2]
 
 [ext_resource path="res://Scripts/Scenes/Main/GameManager.cs" type="Script" id=1]
 [ext_resource path="res://Scripts/Scenes/Main/MusicManager.cs" type="Script" id=2]
 [ext_resource path="res://Scripts/Scenes/Main/Debug.cs" type="Script" id=3]
 [ext_resource path="res://Scripts/Scenes/Main/NetworkManager.cs" type="Script" id=4]
 [ext_resource path="res://Scripts/Scenes/Main/SceneManager.cs" type="Script" id=5]
 [ext_resource path="res://Scenes/Prefabs/Debugger.tscn" type="PackedScene" id=6]
+[ext_resource path="res://Scripts/Scenes/Main/ErrorNotifier.cs" type="Script" id=7]
 
 [node name="Main" type="Node"]
 script = ExtResource( 1 )
@@ -23,6 +24,9 @@ script = ExtResource( 4 )
 [node name="SceneManager" type="Node" parent="."]
 script = ExtResource( 5 )
 
+[node name="ErrorNotifier" type="Node" parent="."]
+script = ExtResource( 7 )
+
 [node name="CanvasLayer" type="CanvasLayer" parent="."]
 
 [node name="Debugger" parent="CanvasLayer" instance=ExtResource( 6 )]

Scripts/Msc/UINotifyError.cs

@@ -11,10 +11,12 @@ public class UINotifyError : Control
     private Label ErrorCount { get; set; }
     private AnimationPlayer AnimationPlayer { get; set; }
 
-    public async void Init(int count)
+    public int Count { get; set; }
+
+    public override async void _Ready()
     {
         ErrorCount = GetNode<Label>(NodePathErrorCount);
-        ErrorCount.Text = $"{count}";
+        ErrorCount.Text = $"{Count}";
         AnimationPlayer = GetNode<AnimationPlayer>(NodePathAnimationPlayer);
         AnimationPlayer.Play("Appear");
         await Task.Delay(2000);

Scripts/Scenes/Main/ErrorNotifier.cs

@@ -1,32 +1,30 @@
-using System.Timers;
+using Godot;
 
 namespace GodotModules
 {
-    public static class ErrorNotifier
+    public class ErrorNotifier : Node
     {
-        private static Timer TimerErrorNotifier { get; set; }
         private static int ErrorCount { get; set; }
-
         public static void IncrementErrorCount() => ErrorCount++;
 
-        public static void StartInterval(int seconds)
+        public override void _Ready()
         {
-            TimerErrorNotifier = new(seconds * 1000);
-            TimerErrorNotifier.Elapsed += SpawnErrorNotification;
-            TimerErrorNotifier.AutoReset = true;
-            TimerErrorNotifier.Enabled = true;
+            var timer = new Timer();
+            timer.Connect("timeout", this, nameof(SpawnErrorNotification));
+            timer.WaitTime = 1.5f;
+            timer.OneShot = false;
+            timer.Autostart = true;
+            AddChild(timer);
         }
 
-        public static void Dispose() => TimerErrorNotifier.Dispose();
-
-        private static void SpawnErrorNotification(System.Object source, ElapsedEventArgs args)
+        private static void SpawnErrorNotification()
         {
             if (ErrorCount == 0)
                 return;
 
             var notifyError = Prefabs.NotifyError.Instance<UINotifyError>();
-            notifyError.Init(ErrorCount);
-            GameManager.Instance.CallDeferred("add_child", notifyError);
+            notifyError.Count = ErrorCount;
+            GameManager.Instance.AddChild(notifyError);
 
             ErrorCount = 0;
         }

Scripts/Scenes/Main/GameManager.cs

@@ -22,7 +22,6 @@ public class GameManager : Node
         public override void _Ready()
         {
             Instance = this;
-            ErrorNotifier.StartInterval(2);
         }
 
         public override void _Input(InputEvent @event)

Scripts/Scenes/Main/NetworkManager.cs

@@ -114,7 +114,6 @@ private static async Task ExitCleanup()
 
             UtilOptions.SaveOptions();
             WebClient.Client.Dispose();
-            ErrorNotifier.Dispose();
 
             ClientConnectingTokenSource?.Dispose();