Improve DialogResult extension with TryGet for data

Add result.Data
b-straub · Jul 1, 2024 · c60b41a · c60b41a
1 parent 6b6849d
commit c60b41a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 6 deletions.
diff --git a/RxMudBlazorLight/Extensions/RxMudBlazorLightExtensionsExtern.cs b/RxMudBlazorLight/Extensions/RxMudBlazorLightExtensionsExtern.cs
@@ -9,5 +9,17 @@ public static bool OK([NotNullWhen(true)] this DialogResult? result)
         {
             return result is not null && !result.Canceled;
         }
+
+        public static bool TryGet<T>(this DialogResult? result, [NotNullWhen(true)] out T? data)
+        {
+            data = default;
+
+            if (result is not null && !result.Canceled)
+            {
+                data = (T?)result.Data;
+            }
+
+            return data is not null;
+        }
     }
 }
diff --git a/RxMudBlazorLightTestBase/CRUD/CRUDItemAddOrUpdate.razor b/RxMudBlazorLightTestBase/CRUD/CRUDItemAddOrUpdate.razor
@@ -25,13 +25,9 @@ else
         var dialog = await DialogService.ShowAsync<CRUDItemDialog>(AddMode ? "Add ToDo" : "Edit ToDo", parameters);
         var result = await dialog.Result;
 
-        if (result.OK())
+        if (result.TryGet<CrudService.CrudItemInput>(out var scope))
         {
-            var scope = (CrudService.CrudItemInput?)result.Data;
-            if (scope is not null)
-            {
-                await scope.SubmitAsync();
-            }
+            await scope.SubmitAsync();
         }
     }
 }