Leverage C# primary constructors whenever possible

src/AppConfiguration.cs

@@ -116,17 +116,6 @@ private void WriteToFile()
 )]
 internal partial class AppConfigurationSerializerContext : JsonSerializerContext {}
 
-abstract class AppConfigurationException : Exception
-{
- protected AppConfigurationException(string message) : base(message) {}
-}
-
-class AppConfigurationLoadException : AppConfigurationException
-{
- public AppConfigurationLoadException(string message) : base(message) {}
-}
-
-class AppConfigurationWriteException : AppConfigurationException
-{
- public AppConfigurationWriteException(string message) : base(message) {}
-}
+abstract class AppConfigurationException(string message) : Exception(message) {}
+class AppConfigurationLoadException(string message) : AppConfigurationException(message) {}
+class AppConfigurationWriteException(string message) : AppConfigurationException(message) {}

src/Operations/AskForRebootOperation.cs

@@ -3,17 +3,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-class AskForRebootOperation : IOperation
+class AskForRebootOperation(IUserInterface ui, RebootRecommendedFlag rebootFlag) : IOperation
 {
- private readonly IUserInterface ui;
- private readonly RebootRecommendedFlag rebootFlag;
-
- public AskForRebootOperation(IUserInterface ui, RebootRecommendedFlag rebootFlag)
- {
- this.ui = ui;
- this.rebootFlag = rebootFlag;
- }
-
  public void Run()
  {
  if (rebootFlag.IsRebootRecommended)

src/Operations/AutoUpdatesDisabler.cs

@@ -4,11 +4,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class AutoUpdatesDisabler : IOperation
+public class AutoUpdatesDisabler(IUserInterface ui) : IOperation
 {
- private readonly IUserInterface ui;
- public AutoUpdatesDisabler(IUserInterface ui) => this.ui = ui;
-
  public void Run()
  {
  ui.PrintMessage("Writing values into the Registry...");

src/Operations/BrowserOpener.cs

@@ -2,15 +2,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class BrowserOpener : IOperation
+public class BrowserOpener(string url) : IOperation
 {
- private readonly string url;
-
- public BrowserOpener(string url)
- {
- this.url = url;
- }
-
  public void Run()
  {
  var startInfo = new ProcessStartInfo {

src/Operations/ConsumerFeaturesDisabler.cs

@@ -4,11 +4,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class ConsumerFeaturesDisabler : IOperation
+public class ConsumerFeaturesDisabler(IUserInterface ui) : IOperation
 {
- private readonly IUserInterface ui;
- public ConsumerFeaturesDisabler(IUserInterface ui) => this.ui = ui;
-
  public bool IsRebootRecommended { get; private set; }
 
  public void Run()

src/Operations/DefenderDisabler.cs

@@ -4,7 +4,7 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class DefenderDisabler : IOperation
+public class DefenderDisabler(IUserInterface ui, ServiceRemover serviceRemover) : IOperation
 {
  private static readonly string[] defenderServices = [
  "wscsvc",
@@ -20,17 +20,8 @@ public class DefenderDisabler : IOperation
  @"\Microsoft\Windows\Windows Defender\Windows Defender Verification"
  ];
 
- private readonly IUserInterface ui;
- private readonly ServiceRemover serviceRemover;
-
  public bool IsRebootRecommended { get; private set; }
 
- public DefenderDisabler(IUserInterface ui, ServiceRemover serviceRemover)
- {
- this.ui = ui;
- this.serviceRemover = serviceRemover;
- }
-
  public void Run()
  {
  CheckForTamperProtection();

src/Operations/EdgeRemover.cs

@@ -8,17 +8,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class EdgeRemover : IOperation
+public class EdgeRemover(IUserInterface ui, AppxRemover appxRemover) : IOperation
 {
- private readonly IUserInterface ui;
- private readonly AppxRemover appxRemover;
-
- public EdgeRemover(IUserInterface ui, AppxRemover appxRemover)
- {
- this.ui = ui;
- this.appxRemover = appxRemover;
- }
-
  public void Run()
  {
  UninstallEdgeChromium();

src/Operations/ErrorReportingDisabler.cs

@@ -3,24 +3,15 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class ErrorReportingDisabler : IOperation
+public class ErrorReportingDisabler(IUserInterface ui, ServiceRemover serviceRemover) : IOperation
 {
  private static readonly string[] errorReportingServices = ["WerSvc", "wercplsupport"];
  private static readonly string[] errorReportingScheduledTasks = [
  @"\Microsoft\Windows\Windows Error Reporting\QueueReporting"
  ];
 
- private readonly IUserInterface ui;
- private readonly ServiceRemover serviceRemover;
-
  public bool IsRebootRecommended => serviceRemover.IsRebootRecommended;
 
- public ErrorReportingDisabler(IUserInterface ui, ServiceRemover serviceRemover)
- {
- this.ui = ui;
- this.serviceRemover = serviceRemover;
- }
-
  public void Run()
  {
  DisableErrorReporting();

src/Operations/FeaturesRemover.cs

@@ -4,19 +4,10 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class FeaturesRemover : IOperation
+public class FeaturesRemover(string[] featuresToRemove, IUserInterface ui) : IOperation
 {
- private readonly string[] featuresToRemove;
- private readonly IUserInterface ui;
-
  public bool IsRebootRecommended { get; private set; }
 
- public FeaturesRemover(string[] featuresToRemove, IUserInterface ui)
- {
- this.featuresToRemove = featuresToRemove;
- this.ui = ui;
- }
-
  public void Run()
  {
  using var dismClient = new DismClient();

src/Operations/LicensePrinter.cs

@@ -3,11 +3,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class LicensePrinter : IOperation
+public class LicensePrinter(IUserInterface ui) : IOperation
 {
- private readonly IUserInterface ui;
- public LicensePrinter(IUserInterface ui) => this.ui = ui;
-
  public void Run()
  {
  Stream licenseFile = GetType().Assembly.GetManifestResourceStream("Win10BloatRemover.Resources.License.txt")!;

src/Operations/OneDriveRemover.cs

@@ -6,12 +6,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class OneDriveRemover : IOperation
+public class OneDriveRemover(IUserInterface ui) : IOperation
 {
- private readonly IUserInterface ui;
-
- public OneDriveRemover(IUserInterface ui) => this.ui = ui;
-
  public void Run()
  {
  DisableOneDrive();

src/Operations/PrivacySettingsTweaker.cs

@@ -4,7 +4,7 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class PrivacySettingsTweaker : IOperation
+public class PrivacySettingsTweaker(IUserInterface ui) : IOperation
 {
  private static readonly string[] appPermissionsToDeny = [
  "location",
@@ -14,9 +14,6 @@ public class PrivacySettingsTweaker : IOperation
  "userAccountInformation"
  ];
 
- private readonly IUserInterface ui;
- public PrivacySettingsTweaker(IUserInterface ui) => this.ui = ui;
-
  public void Run()
  {
  ui.PrintMessage("Writing values into the Registry...");

src/Operations/ScheduledTasksDisabler.cs

@@ -3,17 +3,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class ScheduledTasksDisabler : IOperation
+public class ScheduledTasksDisabler(string[] scheduledTasksToDisable, IUserInterface ui) : IOperation
 {
- private readonly string[] scheduledTasksToDisable;
- private readonly IUserInterface ui;
-
- public ScheduledTasksDisabler(string[] scheduledTasksToDisable, IUserInterface ui)
- {
- this.ui = ui;
- this.scheduledTasksToDisable = scheduledTasksToDisable;
- }
-
  public void Run()
  {
  foreach (string task in scheduledTasksToDisable)

src/Operations/ServiceRemover.cs

@@ -1,26 +1,14 @@
 using Microsoft.Win32;
-using System.Diagnostics;
 using System.IO;
 using Win10BloatRemover.UI;
 using Win10BloatRemover.Utils;
 
 namespace Win10BloatRemover.Operations;
 
-public class ServiceRemovalOperation : IOperation
+public class ServiceRemovalOperation(string[] servicesToRemove, IUserInterface ui, ServiceRemover serviceRemover) : IOperation
 {
- private readonly string[] servicesToRemove;
- private readonly IUserInterface ui;
- private readonly ServiceRemover serviceRemover;
-
  public bool IsRebootRecommended => serviceRemover.IsRebootRecommended;
 
- public ServiceRemovalOperation(string[] servicesToRemove, IUserInterface ui, ServiceRemover serviceRemover)
- {
- this.servicesToRemove = servicesToRemove;
- this.ui = ui;
- this.serviceRemover = serviceRemover;
- }
-
  public void Run()
  {
  ui.PrintHeading("Backing up services...");

src/Operations/SuggestionsDisabler.cs

@@ -4,11 +4,8 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class SuggestionsDisabler : IOperation
+public class SuggestionsDisabler(IUserInterface ui) : IOperation
 {
- private readonly IUserInterface ui;
- public SuggestionsDisabler(IUserInterface ui) => this.ui = ui;
-
  public void Run()
  {
  DisableSuggestions();

src/Operations/TelemetryDisabler.cs

@@ -4,7 +4,7 @@
 
 namespace Win10BloatRemover.Operations;
 
-public class TelemetryDisabler : IOperation
+public class TelemetryDisabler(IUserInterface ui, ServiceRemover serviceRemover) : IOperation
 {
  private static readonly string[] telemetryServices = [
  "DiagTrack",
@@ -34,17 +34,8 @@ public class TelemetryDisabler : IOperation
  @"\Microsoft\Windows\PI\Sqm-Tasks"
  ];
 
- private readonly IUserInterface ui;
- private readonly ServiceRemover serviceRemover;
-
  public bool IsRebootRecommended { get; private set; }
 
- public TelemetryDisabler(IUserInterface ui, ServiceRemover serviceRemover)
- {
- this.ui = ui;
- this.serviceRemover = serviceRemover;
- }
-
  public void Run()
  {
  RemoveTelemetryServices();

src/UI/ConsoleMenu.cs

@@ -4,22 +4,14 @@
 
 namespace Win10BloatRemover.UI;
 
-class ConsoleMenu
+class ConsoleMenu(MenuEntry[] entries, RebootRecommendedFlag rebootFlag)
 {
  private const int FirstMenuEntryNumber = 1;
 
  private bool exitRequested = false;
- private readonly MenuEntry[] entries;
- private readonly RebootRecommendedFlag rebootFlag;
 
  private static readonly Version programVersion = typeof(ConsoleMenu).Assembly.GetName().Version!;
 
- public ConsoleMenu(MenuEntry[] entries, RebootRecommendedFlag rebootFlag)
- {
- this.entries = entries;
- this.rebootFlag = rebootFlag;
- }
-
  public void RunLoopUntilExitRequested()
  {
  while (!exitRequested)