small improvements

sonnemaf · Aug 3, 2020 · 46778d6 · 46778d6
1 parent 12c7a3c
commit 46778d6
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/MvvmDemo/Messages/AsyncYesNoMessage.cs b/MvvmDemo/Messages/AsyncYesNoMessage.cs
@@ -7,7 +7,7 @@
 using Windows.UI.Popups;
 
 namespace MvvmDemo.Messages {
-    public sealed class AsyncYesNoMessage : RequestMessage<Task<bool>> {
+    public sealed class AsyncYesNoMessage : AsyncRequestMessage<bool> {
 
         public AsyncYesNoMessage(string text) {
             Text = text;

diff --git a/MvvmDemo/ViewModels/MainViewModel.cs b/MvvmDemo/ViewModels/MainViewModel.cs
@@ -18,7 +18,7 @@ public class MainViewModel {
 
         public static MainViewModel Current => Ioc.Default.GetService<MainViewModel>();
 
-        
+
         private readonly ILogger _logger;
 
         public ObservableCollection<Employee> Employees { get; } = new ObservableCollection<Employee>();
@@ -37,19 +37,17 @@ public MainViewModel(ILogger logger) {
         }
 
         private async Task OnDelete(Employee emp) {
-            if (emp is object) {
-                if (await Messenger.Default.Send(new AsyncYesNoMessage($"Delete {emp.Name}?")).Response) {
-                    _logger.Log($"Delete: {emp.Name}");
-                    Employees.Remove(emp);
-                }
+            if (await Messenger.Default.Send(new AsyncYesNoMessage($"Delete {emp.Name}?"))) {
+                _logger.Log($"Delete: {emp.Name}");
+                Employees.Remove(emp);
             }
         }
 
         private void OnRaiseSalary(Employee emp) {
-            if (emp is object) {
-                _logger.Log($"OnRaiseSalary: {emp.Name}");
-                emp.Salary += 100;
-            }
+            _logger.Log($"OnRaiseSalary: {emp.Name}");
+            emp.Salary += 100;
         }
     }
+
+
 }