First working version after refactor

Have fixed the mistake I made with the
Parallel.Invoke actions where I accidentally
kept adding the SteamLibrary 3 times!

This now works well, and as a bonus the
game libraries are now thread safe!

DisplayMagician/GameLibraries/OriginGame.cs

@@ -18,6 +18,7 @@ namespace DisplayMagician.GameLibraries
         private string _originGameProcessName;
         private string _originGameIconPath;
         //private string _originURI;
+        private static readonly OriginLibrary _originGameLibrary = OriginLibrary.GetLibrary();
         private static readonly NLog.Logger logger = NLog.LogManager.GetCurrentClassLogger();
 
         static OriginGame()

DisplayMagician/GameLibraries/OriginLibrary.cs

@@ -42,6 +42,8 @@ namespace DisplayMagician.GameLibraries
         #endregion
 
         #region Class Constructors
+        static OriginLibrary() { }
+
         private OriginLibrary()
         {
             try

DisplayMagician/GameLibraries/SteamGame.cs

@@ -20,6 +20,7 @@ namespace DisplayMagician.GameLibraries
         private string _steamGameExe;
         private string _steamGameProcessName;
         private string _steamGameIconPath;
+        private static readonly SteamLibrary _steamGameLibrary = SteamLibrary.GetLibrary();
         private static readonly NLog.Logger logger = NLog.LogManager.GetCurrentClassLogger();
 
         static SteamGame()
@@ -32,7 +33,7 @@ namespace DisplayMagician.GameLibraries
         public SteamGame(string steamGameId, string steamGameName, string steamGameExePath, string steamGameIconPath)
         {
 
-            _gameRegistryKey = $@"{SteamLibrary.GetLibrary().SteamAppsRegistryKey}\\{steamGameId}";
+            _gameRegistryKey = $@"{_steamGameLibrary.SteamAppsRegistryKey}\\{steamGameId}";
             _steamGameId = steamGameId;
             _steamGameName = steamGameName;
             _steamGameExePath = steamGameExePath;

DisplayMagician/GameLibraries/SteamLibrary.cs

@@ -27,7 +27,7 @@ namespace DisplayMagician.GameLibraries
         // Static members are 'eagerly initialized', that is, 
         // immediately when class is loaded for the first time.
         // .NET guarantees thread safety for static initialization
-        private static readonly SteamLibrary _instance = new SteamLibrary();
+        private static SteamLibrary _instance = new SteamLibrary();
 
         // Common items to the class
         private List<Game> _allSteamGames = new List<Game>();
@@ -44,7 +44,9 @@ namespace DisplayMagician.GameLibraries
         #endregion
 
         #region Class Constructors
-        SteamLibrary()
+        static SteamLibrary() { }
+
+        private SteamLibrary()
         {
             try
             {
@@ -573,7 +575,7 @@ namespace DisplayMagician.GameLibraries
                             if (steamAppType.Equals("Game",StringComparison.OrdinalIgnoreCase))
                             {
                                 steamAppInfo.Add(detectedAppID, steamGameAppInfo);
-                                logger.Trace($"SteamLibrary/LoadInstalledGames: Adding Game with ID {detectedAppID} to the list of games");
+                                logger.Trace($"SteamLibrary/LoadInstalledGames: Adding Game with ID {detectedAppID} '{steamGameAppInfo.GameName}' to the list of games");
                             }
                                 
                         }

DisplayMagician/GameLibraries/UplayGame.cs

@@ -18,6 +18,7 @@ namespace DisplayMagician.GameLibraries
         private string _uplayGameExe;
         private string _uplayGameProcessName;
         private string _uplayGameIconPath;
+        private static readonly UplayLibrary _uplayGameLibrary = UplayLibrary.GetLibrary();
         private static readonly NLog.Logger logger = NLog.LogManager.GetCurrentClassLogger();
 
         static UplayGame()

DisplayMagician/GameLibraries/UplayLibrary.cs

@@ -11,6 +11,8 @@ namespace DisplayMagician.GameLibraries
 {
     public class UplayLibrary : GameLibrary
     {
+
+
         #region Class Variables
         // Static members are 'eagerly initialized', that is, 
         // immediately when class is loaded for the first time.
@@ -36,7 +38,9 @@ namespace DisplayMagician.GameLibraries
 
 
         #region Class Constructors
-        UplayLibrary()
+        static UplayLibrary() { }
+
+        private UplayLibrary()
         {
             try
             {

DisplayMagician/Program.cs

@@ -16,6 +16,7 @@ using System.Drawing;
 using DesktopNotifications;
 using System.Runtime.Serialization;
 using NLog.Config;
+using System.Collections.Generic;
 
 namespace DisplayMagician {
 
@@ -778,47 +779,106 @@ namespace DisplayMagician {
 
             });
 
+            Action loadSteamGamesAction = new Action(() =>
+            {
+                // Check if Steam is installed
+                GameLibrary steamLibrary = SteamLibrary.GetLibrary();
+                if (steamLibrary.IsGameLibraryInstalled)
+                {
+                    // Load Steam library games
+                    logger.Info($"Program/LoadGamesInBackground: Loading Installed Steam Games");
+                    Console.Write("Loading Installed Steam Games...");
+                    if (!steamLibrary.LoadInstalledGames())
+                    {
+                        logger.Info($"Program/LoadGamesInBackground: Cannot load installed Steam Games!");
+                    }
+                    Console.WriteLine("Done.");
+                    logger.Info($"Program/LoadGamesInBackground: Loaded all Installed Steam Games (found {steamLibrary.InstalledGameCount})");
+                }
+                else
+                {
+                    logger.Info($"Program/LoadGamesInBackground: Steam not installed.");
+                    Console.WriteLine("Steam not installed.");
+                }
+            });
 
-            // Store all the tasks in an array so we can wait on them later
+            // Now lets prepare loading all the Uplay games we have installed
-            Task[] loadGamesTasks = new Task[3];
+            Action loadUplayGamesAction = new Action(() =>
-            loadGamesTasks[0] = loadSteamGamesTask;
+            {
-            loadGamesTasks[1] = loadUplayGamesTask;
+                // Check if Uplay is installed
-            loadGamesTasks[2] = loadOriginGamesTask;
+                GameLibrary uplayLibrary = UplayLibrary.GetLibrary();
+                if (uplayLibrary.IsGameLibraryInstalled)
+                {
+                    // Load Uplay library games
+                    logger.Info($"Program/LoadGamesInBackground: Loading Installed Uplay Games");
+                    Console.Write("Loading Installed Uplay Games...");
+                    if (!uplayLibrary.LoadInstalledGames())
+                    {
+                        logger.Info($"Program/LoadGamesInBackground: Cannot load installed Uplay Games!");
+                    }
+                    Console.WriteLine("Done.");
+                    logger.Info($"Program/LoadGamesInBackground: Loaded all Installed Uplay Games (found {uplayLibrary.InstalledGameCount})");
+                }
+                else
+                {
+                    logger.Info($"Program/LoadGamesInBackground: Uplay not installed.");
+                    Console.WriteLine("Uplay not installed.");
+                }
 
-            logger.Debug($"Program/LoadGamesInBackground: Running game loading tasks.");
+            });
-            // Go through and start all the tasks
+
-            foreach (Task loadGameTask in loadGamesTasks)
+            // Now lets prepare loading all the Origin games we have installed
-                loadGameTask.Start();
+            Action loadOriginGamesAction = new Action(() =>
+            {
+                // Check if Origin is installed
+                GameLibrary originLibrary = OriginLibrary.GetLibrary();
+                if (originLibrary.IsGameLibraryInstalled)
+                {
+                    // Load Origin library games
+                    logger.Info($"Program/LoadGamesInBackground: Loading Installed Origin Games");
+                    Console.Write("Loading Installed Origin Games...");
+                    if (!originLibrary.LoadInstalledGames())
+                    {
+                        logger.Info($"Program/LoadGamesInBackground: Cannot load installed Origin Games!");
+                    }
+                    Console.WriteLine("Done.");
+                    logger.Info($"Program/LoadGamesInBackground: Loaded all Installed Origin Games (found {originLibrary.InstalledGameCount})");
+                }
+                else
+                {
+                    logger.Info($"Program/LoadGamesInBackground: Origin not installed.");
+                    Console.WriteLine("Origin not installed.");
+                }
+
+            });
+
+
+            // Store all the actions in a array so we can wait on them later
+            List<Action> loadGamesActions = new List<Action>();
+            loadGamesActions.Add(loadSteamGamesAction);
+            loadGamesActions.Add(loadUplayGamesAction);
+            loadGamesActions.Add(loadOriginGamesAction);
 
             try
             {
-                logger.Debug($"Program/LoadGamesInBackground: Waiting for all game loading tasks to finish");
+                logger.Debug($"Program/LoadGamesInBackground: Running game loading actions.");
-                Task.WaitAll(loadGamesTasks);
+                // Go through and start all the actions, making sure we only have one threat per action to avoid thread issues
+                int threads = loadGamesActions.Count;
+                ParallelOptions options = new ParallelOptions { MaxDegreeOfParallelism = threads };
+                Parallel.Invoke(options, loadGamesActions.ToArray());
                 logger.Debug($"Program/LoadGamesInBackground: All game loading tasks finished");
             }
             catch (AggregateException ae)
             {
-                logger.Error(ae, $"Program/LoadGamesInBackground exception during loadGamesTasks");
+                logger.Error(ae, $"Program/LoadGamesInBackground exception during loadGamesActions");
-                Console.WriteLine("Program/LoadGamesInBackground : Task exception!");
-                foreach (var e in ae.InnerExceptions)
-                {
-                    // Handle the custom exception.
-                    if (e is LoadingInstalledGamesException)
-                    {
-                        Console.WriteLine(e.Message);
-                    }
-                    // Rethrow any other exception.
-                    else
-                    {
-                        throw;
-                    }
-                }
             }
 
-            bool failedTask = false;
+
-            foreach (var loadGameTask in loadGamesTasks)
+            // TODO replicate this failed Task handling in Actions
+            /*bool failedAction = false;
+            foreach (var loadGameAction in loadGamesActions)
             {
-                if (loadGameTask.Exception != null)
+                if (loadGameAction.  .Exception != null)
                 {
                     failedTask = true;
                     foreach (var ex in loadGameTask.Exception.InnerExceptions)
@@ -827,7 +887,7 @@ namespace DisplayMagician {
             }
 
             if (failedTask)
-                return false;
+                return false;*/
 
             return true;