Merge pull request #17 from theowiik:instanting-improvements

Add a SceneGetter util

GodotSharper/EnumerableExtensions.cs

@@ -16,7 +16,9 @@ public static void TimesDo(this int count, Action<int> action)
             throw new ArgumentException("Count must be greater than or equal to 0", nameof(count));
 
         for (var i = 0; i < count; i++)
+        {
             action(i);
+        }
     }
 
     /// <summary>
@@ -30,7 +32,9 @@ public static void TimesDo(this int count, Action action)
             throw new ArgumentException("Count must be greater than or equal to 0", nameof(count));
 
         for (var i = 0; i < count; i++)
+        {
             action();
+        }
     }
 
     /// <summary>

GodotSharper/Instancing/Instanter.cs → GodotSharper/Scenes/Instanter.cs

@@ -28,12 +28,11 @@ public static T Instantiate<T>()
         }
         else
         {
-            var attr = (InstantiableAttribute)
-                Attribute.GetCustomAttribute(type, typeof(InstantiableAttribute));
+            var attr = (SceneAttribute)Attribute.GetCustomAttribute(type, typeof(SceneAttribute));
 
             if (attr == null)
-                throw new FileNotFoundException(
-                    "Could not find a PackedSceneAttribute for " + type
+                throw new ArgumentException(
+                    $"Type {type} does not have a {nameof(SceneAttribute)}"
                 );
 
             path = attr.Path;

GodotSharper/Scenes/SceneAttribute.cs

@@ -0,0 +1,22 @@
+namespace GodotSharper.Instancing;
+
+/// <summary>
+///     Attribute used to mark a class as a scene and specify its path.
+/// </summary>
+[AttributeUsage(AttributeTargets.Class)]
+public sealed class SceneAttribute : Attribute
+{
+    /// <summary>
+    ///     Initializes a new instance of the <see cref="SceneAttribute" /> class with the specified path.
+    /// </summary>
+    /// <param name="path">The path of the scene.</param>
+    public SceneAttribute(string path)
+    {
+        Path = path;
+    }
+
+    /// <summary>
+    ///     Gets the path of the scene.
+    /// </summary>
+    public string Path { get; }
+}

GodotSharper/Scenes/SceneGetter.cs

@@ -0,0 +1,30 @@
+using System.Reflection;
+using Godot;
+
+namespace GodotSharper.Instancing;
+
+/// <summary>
+///     A static class that provides a method to get the path of a scene associated with a given type.
+/// </summary>
+public static class SceneGetter
+{
+    /// <summary>
+    ///     Gets the path of a scene associated with the specified type.
+    /// </summary>
+    /// <typeparam name="T">The type of the node associated with the scene.</typeparam>
+    /// <returns>The path of the scene associated with the specified type.</returns>
+    /// <exception cref="ArgumentException">Thrown when the specified type does not have a SceneAttribute.</exception>
+    public static string GetPath<T>()
+        where T : Node
+    {
+        var type = typeof(T);
+        var attribute = type.GetCustomAttribute<SceneAttribute>();
+
+        if (attribute == null)
+        {
+            throw new ArgumentException($"Type {type} does not have a {nameof(SceneAttribute)}");
+        }
+
+        return attribute.Path;
+    }
+}

GodotSharper/TimerFactory.cs

@@ -20,9 +20,10 @@ public static Timer StartedOneShot(double waitTime, Action onTimeout = null)
 
         return timer;
     }
-    
+
     /// <summary>
-    ///     Creates a new Timer object that starts automatically, triggers only once after the specified wait time, and destroys itself after triggering.
+    ///     Creates a new Timer object that starts automatically, triggers only once after the specified wait time, and
+    ///     destroys itself after triggering.
     /// </summary>
     /// <param name="waitTime">The time to wait before triggering the timer, in seconds.</param>
     /// <param name="onTimeout">An optional action to execute when the timer triggers.</param>