From 876f7223b07f3484502cfc75d514890fa69aae64 Mon Sep 17 00:00:00 2001 From: Matt Nadareski Date: Wed, 30 Oct 2024 13:41:45 -0400 Subject: [PATCH] Separate out console write as well --- SabreTools.Reports/BaseReport.cs | 4 ++-- SabreTools.Reports/Formats/Textfile.cs | 21 +++++++++++++++++---- 2 files changed, 19 insertions(+), 6 deletions(-) diff --git a/SabreTools.Reports/BaseReport.cs b/SabreTools.Reports/BaseReport.cs index 83afb0ff..30b3590f 100644 --- a/SabreTools.Reports/BaseReport.cs +++ b/SabreTools.Reports/BaseReport.cs @@ -40,8 +40,8 @@ namespace SabreTools.Reports { return statReportFormat switch { - StatReportFormat.None => new Textfile(statsList, true), - StatReportFormat.Textfile => new Textfile(statsList, false), + StatReportFormat.None => new ConsoleOutput(statsList), + StatReportFormat.Textfile => new Textfile(statsList), StatReportFormat.CSV => new CommaSeparatedValue(statsList), StatReportFormat.HTML => new Html(statsList), StatReportFormat.SSV => new SemicolonSeparatedValue(statsList), diff --git a/SabreTools.Reports/Formats/Textfile.cs b/SabreTools.Reports/Formats/Textfile.cs index 5fa1b952..a1d348e5 100644 --- a/SabreTools.Reports/Formats/Textfile.cs +++ b/SabreTools.Reports/Formats/Textfile.cs @@ -13,17 +13,15 @@ namespace SabreTools.Reports.Formats /// internal class Textfile : BaseReport { - private readonly bool _writeToConsole; + protected bool _writeToConsole = false; /// /// Create a new report from the filename /// /// List of statistics objects to set - /// True to write to consoke output, false otherwise - public Textfile(List statsList, bool writeToConsole) + public Textfile(List statsList) : base(statsList) { - _writeToConsole = writeToConsole; } /// @@ -127,4 +125,19 @@ namespace SabreTools.Reports.Formats sw.Flush(); } } + + /// + /// Console report format + /// + internal sealed class ConsoleOutput : Textfile + { + /// + /// Create a new report from the filename + /// + /// List of statistics objects to set + public ConsoleOutput(List statsList) : base(statsList) + { + _writeToConsole = true; + } + } }