fix(di): registered GraphicsDeviceManager on the same level with `G…

…ame`

- Fixed double-disposing behaviour issue, when the game is quit (left a comment)

src/Apps/GameDesktop/CompositionRoots/GameCompositionRoot.cs

@@ -2,8 +2,6 @@
 using GameDesktop.Resources.Internal;
 using LightInject;
 using Microsoft.Xna.Framework;
-using MonoGame.ImGuiNet;
-using Myra;
 using Serilog;
 
 namespace GameDesktop.CompositionRoots;
@@ -16,21 +14,24 @@ internal class GameCompositionRoot : ICompositionRoot
 
     public void Compose(IServiceRegistry serviceRegistry)
     {
-        serviceRegistry.RegisterSingleton(factory =>
+        ServiceRegistration[] services = serviceRegistry.AvailableServices.ToArray();
+
+        var logger =
+            (ILogger)services.First(r => r.ServiceType == typeof(ILogger)).Value;
+        var container =
+            (IServiceContainer)services.First(r => r.ServiceType == typeof(IServiceContainer)).Value;
+
+        Game game = new(logger, container)
         {
-            Game game = new(factory.GetInstance<ILogger>(),
-                factory.GetInstance<IServiceContainer>())
-            {
-                IsMouseVisible = IsMouseVisible,
-                IsFixedTimeStep = IsFixedTimeStep,
-                Content = { RootDirectory = AppVariable.ContentRootDirectory, },
-            };
+            IsMouseVisible = IsMouseVisible,
+            IsFixedTimeStep = IsFixedTimeStep,
+            Content = { RootDirectory = AppVariable.ContentRootDirectory, },
+        };
+        serviceRegistry.RegisterInstance(game);
 
-            // Hack. Resolving cycle dependency issue (fundamental architecture)
-            // Implicitly adds itself in the game services container.
-            new GraphicsDeviceManager(game);
-
-            return game;
-        });
+        // Hack. Resolving cycle dependency issue (fundamental architecture)
+        // Implicitly adds itself in the game services container.
+        GraphicsDeviceManager graphicsDeviceManager = new(game);
+        serviceRegistry.RegisterInstance(graphicsDeviceManager);
     }
 }

src/Apps/GameDesktop/Program.cs

@@ -19,7 +19,9 @@
 
 try
 {
-    using ServiceContainer container = new(
+    // "Using" keyword should be used either with the container, or with the game instance.
+    // Otherwise, you'll get the double-disposing behaviour.
+    ServiceContainer container = new(
         new ContainerOptions
         {
             EnablePropertyInjection = false,