Add final status counts, structure for more statistics and exit codes.

Author: EraYaN

CompatibilityChecker/Analyzer.cs

@@ -0,0 +1,51 @@
+namespace CompatibilityChecker
+{
+    using System;
+    using System.Collections.Generic;
+
+    internal class BasicListingReporter : IMessageReporter
+    {
+        IMessageLogger _logger;
+
+        List<Message> _reportedMessages;
+
+        int _severityError = 0;
+        int _severityWarning = 0;
+        int _severityInformation = 0;
+        int _severityDisabled = 0;
+
+        IEnumerable<Message> IMessageReporter.ReportedMessages => _reportedMessages;
+
+        IReportStatistics IMessageReporter.Statistics => new BasicListingStatistics(_severityError, _severityWarning, _severityInformation, _severityDisabled);
+
+        public BasicListingReporter(IMessageLogger logger)
+        {
+            _logger = logger;
+
+            _reportedMessages = new List<Message>();
+        }
+        
+        void IMessageReporter.Report(Message message)
+        {
+            switch (message.Severity)
+            {
+                case Severity.Error:
+                    _severityError++;
+                    break;
+                case Severity.Warning:
+                    _severityWarning++;
+                    break;
+                case Severity.Information:
+                    _severityInformation++;
+                    break;
+                case Severity.Disabled:
+                    _severityDisabled++;
+                    break;
+                default:
+                    throw new ArgumentException(string.Format("Severity {0} is not supported by this Message Reporter.", message.Severity));
+            }
+            _reportedMessages.Add(message);
+            _logger.Report(message);
+        }
+    }
+}

CompatibilityChecker/BasicListingReporter.cs

@@ -0,0 +1,19 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace CompatibilityChecker
+{
+    class BasicListingStatistics : IReportStatistics
+    {
+        (int error, int warning, int information, int disabled) _severityCounts;
+        (int error, int warning, int information, int disabled) IReportStatistics.SeverityCounts => _severityCounts;
+
+        public BasicListingStatistics(int SeverityError, int SeverityWarning, int SeverityInformation, int SeverityDisabled)
+        {
+            _severityCounts = (error: SeverityError, warning: SeverityWarning, information: SeverityInformation, disabled: SeverityDisabled);
+        }
+    }
+}

CompatibilityChecker/BasicListingStatistics.cs

@@ -0,0 +1,14 @@
+using System.Collections.Generic;
+
+namespace CompatibilityChecker
+{
+    public interface IMessageReporter
+    {
+        IEnumerable<Message> ReportedMessages { get; }
+
+        IReportStatistics Statistics { get; }
+
+        void Report(Message message);
+        
+    }
+}

CompatibilityChecker/IMessageReporter.cs

@@ -0,0 +1,7 @@
+namespace CompatibilityChecker
+{
+    public interface IReportStatistics
+    {
+        (int error, int warning, int information, int disabled) SeverityCounts { get; }
+    }
+}

CompatibilityChecker/IReportStatistics.cs

@@ -13,6 +13,8 @@ internal Message(CompatibilityDescriptor descriptor, params object[] arguments)
             _arguments = arguments;
         }
 
+        internal Severity Severity => _descriptor.DefaultSeverity;
+
         public override string ToString()
         {
             string message = string.Format(_descriptor.MessageFormat, _arguments);

CompatibilityChecker/Message.cs

@@ -18,6 +18,7 @@ private static void Main(string[] args)
             }
             else
             {
+                
                 FileInfo referenceFile = new FileInfo(args[0]);
                 FileInfo newFile = new FileInfo(args[1]);
                 if (referenceFile.Exists && newFile.Exists)
@@ -26,8 +27,19 @@ private static void Main(string[] args)
                     {
                         using (PEReader newAssembly = new PEReader(File.OpenRead(newFile.FullName)))
                         {
-                            Analyzer analyzer = new Analyzer(referenceAssembly, newAssembly, null);
+                            Analyzer analyzer = new Analyzer(referenceAssembly, newAssembly, null, null);
                             analyzer.Run();
+                            if (analyzer.HasRun)
+                            {
+                                Console.Error.WriteLine(string.Format("Analyzer done. {0} errors, {1} warnings, {2} informational items.",analyzer.ResultStatistics.SeverityCounts.error, analyzer.ResultStatistics.SeverityCounts.warning, analyzer.ResultStatistics.SeverityCounts.information));
+                                if(analyzer.ResultStatistics.SeverityCounts.error > 0)
+                                {
+                                    Environment.ExitCode = -2;
+                                }
+                            } else
+                            {
+                                Environment.ExitCode = -1;
+                            }
                         }
                     }
                 }

CompatibilityCheckerCLI/Program.cs

@@ -6,7 +6,7 @@ namespace CompatibilityCheckerCoreCLI
 {
     using File = System.IO.File;
     using FileInfo = System.IO.FileInfo;
-    
+
     internal class Program
     {
         private static void Main(string[] args)
@@ -17,7 +17,8 @@ private static void Main(string[] args)
                 Environment.ExitCode = 1;
             }
             else
-            {                
+            {
+
                 FileInfo referenceFile = new FileInfo(args[0]);
                 FileInfo newFile = new FileInfo(args[1]);
                 if (referenceFile.Exists && newFile.Exists)
@@ -26,12 +27,24 @@ private static void Main(string[] args)
                     {
                         using (PEReader newAssembly = new PEReader(File.OpenRead(newFile.FullName)))
                         {
-                            Analyzer analyzer = new Analyzer(referenceAssembly, newAssembly, null);
-                            analyzer.Run();                            
+                            Analyzer analyzer = new Analyzer(referenceAssembly, newAssembly, null, null);
+                            analyzer.Run();
+                            if (analyzer.HasRun)
+                            {
+                                Console.Error.WriteLine(string.Format("Analyzer done. {0} errors, {1} warnings, {2} informational items.", analyzer.ResultStatistics.SeverityCounts.error, analyzer.ResultStatistics.SeverityCounts.warning, analyzer.ResultStatistics.SeverityCounts.information));
+                                if (analyzer.ResultStatistics.SeverityCounts.error > 0)
+                                {
+                                    Environment.ExitCode = -2;
+                                }
+                            }
+                            else
+                            {
+                                Environment.ExitCode = -1;
+                            }
                         }
                     }
                 }
             }
         }
     }
-}
+}