Use Nlog Debug Target instead of direct print

Author: taooceros

Flow.Launcher.Infrastructure/Logger/Log.cs

@@ -32,21 +32,30 @@ static Log()
             };
 
             var fileTargetASyncWrapper = new AsyncTargetWrapper(fileTarget);
+
+            var debugTarget = new DebuggerTarget
+            {
+                Layout = "${level:uppercase=true}|${message}"
+            };
+            
             configuration.AddTarget("file", fileTargetASyncWrapper);
+            configuration.AddTarget("console", debugTarget);
 #if DEBUG
-            var rule = new LoggingRule("*", LogLevel.Debug, fileTargetASyncWrapper);
+            var fileRule = new LoggingRule("*", LogLevel.Debug, fileTargetASyncWrapper);
+            var debugRule = new LoggingRule("*", LogLevel.Debug, debugTarget);
 #else
             var rule = new LoggingRule("*", LogLevel.Info, fileTargetASyncWrapper);
+            var debugRule = new LoggingRule("*", LogLevel.Info, consoleTarget);
 #endif
-            configuration.LoggingRules.Add(rule);
+            configuration.LoggingRules.Add(fileRule);
+            configuration.LoggingRules.Add(debugRule);
             LogManager.Configuration = configuration;
         }
 
         private static void LogFaultyFormat(string message)
         {
             var logger = LogManager.GetLogger("FaultyLogger");
             message = $"Wrong logger message format <{message}>";
-            System.Diagnostics.Debug.WriteLine($"FATAL|{message}");
             logger.Fatal(message);
         }
 
@@ -57,9 +66,9 @@ private static bool FormatValid(string message)
             return valid;
         }
 
-        
+
         public static void Exception(string className, string message, System.Exception exception, [CallerMemberName] string methodName = "")
-        { 
+        {
             exception = exception.Demystify();
 #if DEBUG
             ExceptionDispatchInfo.Capture(exception).Throw();
@@ -96,7 +105,6 @@ private static void ExceptionInternal(string classAndMethod, string message, Sys
         {
             var logger = LogManager.GetLogger(classAndMethod);
 
-            System.Diagnostics.Debug.WriteLine($"ERROR|{message}");
 
             logger.Error("-------------------------- Begin exception --------------------------");
             logger.Error(message);
@@ -123,8 +131,6 @@ private static void LogInternal(string message, LogLevel level)
                 var prefix = parts[1];
                 var unprefixed = parts[2];
                 var logger = LogManager.GetLogger(prefix);
-
-                System.Diagnostics.Debug.WriteLine($"{level.Name}|{message}");
                 logger.Log(level, unprefixed);
             }
             else
@@ -134,6 +140,7 @@ private static void LogInternal(string message, LogLevel level)
         }
 
         /// <param name="message">example: "|prefix|unprefixed" </param>
+        /// <param name="e">Exception</param>
         [MethodImpl(MethodImplOptions.Synchronized)]
         public static void Exception(string message, System.Exception e)
         {
@@ -172,7 +179,6 @@ private static void LogInternal(LogLevel level, string className, string message
 
             var logger = LogManager.GetLogger(classNameWithMethod);
 
-            System.Diagnostics.Debug.WriteLine($"{level.Name}|{message}");
             logger.Log(level, message);
         }
 

Plugins/Flow.Launcher.Plugin.Program/Logger/ProgramLogger.cs

@@ -48,8 +48,6 @@ static ProgramLogger()
         internal static void LogException(string classname, string callingMethodName, string loadingProgramPath,
             string interpretationMessage, Exception e)
         {
-            Debug.WriteLine($"ERROR{classname}|{callingMethodName}|{loadingProgramPath}|{interpretationMessage}");
-
             var logger = LogManager.GetLogger("");
 
             var innerExceptionNumber = 1;