From f437c6198331ce7faf311859d95bfe4ff4b8061d Mon Sep 17 00:00:00 2001
From: Stefan Roelofs <stefan@roelofs.nl>
Date: Thu, 18 Mar 2021 21:56:47 +0100
Subject: [PATCH] Some refactoring, no functional changes.

---
 .../Views/BackgroundServiceSettings.axaml.cs  | 174 +++++++++---------
 1 file changed, 88 insertions(+), 86 deletions(-)

diff --git a/UserInterface/Views/BackgroundServiceSettings.axaml.cs b/UserInterface/Views/BackgroundServiceSettings.axaml.cs
index 046f362..1789bae 100644
--- a/UserInterface/Views/BackgroundServiceSettings.axaml.cs
+++ b/UserInterface/Views/BackgroundServiceSettings.axaml.cs
@@ -1,86 +1,88 @@
-using Avalonia;
-using Avalonia.Controls;
-using Avalonia.Markup.Xaml;
-using Microsoft.Extensions.DependencyInjection;
-using hass_workstation_service.Communication.NamedPipe;
-using JKang.IpcServiceFramework.Client;
-using System.Threading.Tasks;
-using Avalonia.Interactivity;
-using System.Reactive.Linq;
-using UserInterface.ViewModels;
-using System.Security;
-using hass_workstation_service.Communication.InterProcesCommunication.Models;
-
-namespace UserInterface.Views
-{
-    public class BackgroundServiceSettings : UserControl
-    {
-        private readonly IIpcClient<ServiceContractInterfaces> client;
-
-        public BackgroundServiceSettings()
-        {
-            this.InitializeComponent();
-            // register IPC clients
-            ServiceProvider serviceProvider = new ServiceCollection()
-                .AddNamedPipeIpcClient<ServiceContractInterfaces>("broker", pipeName: "pipeinternal")
-                .BuildServiceProvider();
-
-            // resolve IPC client factory
-            IIpcClientFactory<ServiceContractInterfaces> clientFactory = serviceProvider
-                .GetRequiredService<IIpcClientFactory<ServiceContractInterfaces>>();
-
-            // create client
-            this.client = clientFactory.CreateClient("broker");
-
-
-            DataContext = new BackgroundServiceSettingsViewModel();
-            Ping();
-        }
-        public async void Ping() {
-            while (true)
-            {
-                try
-                {
-                    var result = await this.client.InvokeAsync(x => x.Ping("ping"));
-                    if (result == "pong")
-                    {
-                        ((BackgroundServiceSettingsViewModel)this.DataContext).UpdateStatus(true, "All good");
-                    }
-                    else
-                    {
-                        ((BackgroundServiceSettingsViewModel)this.DataContext).UpdateStatus(false, "Not running");
-                    }
-                }
-                catch (System.Exception)
-                {
-                    ((BackgroundServiceSettingsViewModel)this.DataContext).UpdateStatus(false, "Not running");
-                }
-
-                var autostartresult = await this.client.InvokeAsync(x => x.IsAutoStartEnabled());
-                ((BackgroundServiceSettingsViewModel)this.DataContext).UpdateAutostartStatus(autostartresult);
-
-                await Task.Delay(1000);
-            }
-        }
-
-        public void Start(object sender, RoutedEventArgs args)
-        {
-            //TODO: fix the path. This will depend on the deployment structure.
-            System.Diagnostics.Process.Start("hass-worstation-service.exe");
-        }
-
-        public void EnableAutostart(object sender, RoutedEventArgs args)
-        {
-            this.client.InvokeAsync(x => x.EnableAutostart(true));
-        }
-        public void DisableAutostart(object sender, RoutedEventArgs args)
-        {
-            this.client.InvokeAsync(x => x.EnableAutostart(false));
-        }
-
-        private void InitializeComponent()
-        {
-            AvaloniaXamlLoader.Load(this);
-        }
-    }
-}
+using Avalonia;
+using Avalonia.Controls;
+using Avalonia.Markup.Xaml;
+using Microsoft.Extensions.DependencyInjection;
+using hass_workstation_service.Communication.NamedPipe;
+using JKang.IpcServiceFramework.Client;
+using System.Threading.Tasks;
+using Avalonia.Interactivity;
+using System.Reactive.Linq;
+using UserInterface.ViewModels;
+using System.Security;
+using hass_workstation_service.Communication.InterProcesCommunication.Models;
+
+namespace UserInterface.Views
+{
+    public class BackgroundServiceSettings : UserControl
+    {
+        private readonly IIpcClient<ServiceContractInterfaces> _client;
+
+        public BackgroundServiceSettings()
+        {
+            this.InitializeComponent();
+            // register IPC clients
+            ServiceProvider serviceProvider = new ServiceCollection()
+                .AddNamedPipeIpcClient<ServiceContractInterfaces>("broker", pipeName: "pipeinternal")
+                .BuildServiceProvider();
+
+            // resolve IPC client factory
+            IIpcClientFactory<ServiceContractInterfaces> clientFactory = serviceProvider
+                .GetRequiredService<IIpcClientFactory<ServiceContractInterfaces>>();
+
+            // create client
+            this._client = clientFactory.CreateClient("broker");
+
+            DataContext = new BackgroundServiceSettingsViewModel();
+            Ping();
+        }
+
+        public async void Ping()
+        {
+            while (true)
+            {
+                if (DataContext is not BackgroundServiceSettingsViewModel viewModel)
+                    throw new System.Exception("Wrong viewmodel class!");
+
+                try
+                {
+                    var result = await this._client.InvokeAsync(x => x.Ping("ping"));
+
+                    if (result == "pong")
+                        viewModel.UpdateStatus(true, "All good");
+                    else
+                        viewModel.UpdateStatus(false, "Not running");
+                }
+                catch (System.Exception)
+                {
+                    viewModel.UpdateStatus(false, "Not running");
+                }
+
+                var autostartresult = await this._client.InvokeAsync(x => x.IsAutoStartEnabled());
+                viewModel.UpdateAutostartStatus(autostartresult);
+
+                await Task.Delay(1000);
+            }
+        }
+
+        public void Start(object sender, RoutedEventArgs args)
+        {
+            //TODO: fix the path. This will depend on the deployment structure.
+            System.Diagnostics.Process.Start("hass-worstation-service.exe");
+        }
+
+        public void EnableAutostart(object sender, RoutedEventArgs args)
+        {
+            this._client.InvokeAsync(x => x.EnableAutostart(true));
+        }
+
+        public void DisableAutostart(object sender, RoutedEventArgs args)
+        {
+            this._client.InvokeAsync(x => x.EnableAutostart(false));
+        }
+
+        private void InitializeComponent()
+        {
+            AvaloniaXamlLoader.Load(this);
+        }
+    }
+}
\ No newline at end of file