fix(Table): add IsAutoInitializeModelProperty parameter control create instance logic (#5530)

* feat: 增加 IsAutoInitializeModelProperty 参数

* chore: bump version 9.4.6

* test: 增加单元测试

* refactor: 更新方法名

* test: 更新单元测试

* test: 更新单元测试

* refactor: 精简代码

Author: ArgoZhang

src/BootstrapBlazor/BootstrapBlazor.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk.Razor">
 
   <PropertyGroup>
-    <Version>9.4.5</Version>
+    <Version>9.4.6</Version>
   </PropertyGroup>
 
   <ItemGroup>

src/BootstrapBlazor/Components/Table/Table.razor.Edit.cs

@@ -294,18 +294,28 @@ private async Task InternalOnAddAsync()
     [Parameter]
     public Func<TItem>? CreateItemCallback { get; set; }
 
+    /// <summary>
+    /// Get or sets Whether to automatically initialize model properties default value is false.
+    /// </summary>
+    [Parameter]
+    public bool IsAutoInitializeModelProperty { get; set; }
+
     private TItem CreateTItem() => CreateItemCallback?.Invoke() ?? CreateInstance();
 
+    private readonly string ErrorMessage = $"{typeof(TItem)} create instrance failed. Please provide {nameof(CreateItemCallback)} create the {typeof(TItem)} instance. {typeof(TItem)} 自动创建实例失败，请通过 {nameof(CreateItemCallback)} 回调方法手动创建实例";
+
     private TItem CreateInstance()
     {
+        TItem? item;
         try
         {
-            return ObjectExtensions.CreateInstanceWithCascade<TItem>();
+            item = ObjectExtensions.CreateInstance<TItem>(IsAutoInitializeModelProperty);
         }
         catch (Exception ex)
         {
-            throw new InvalidOperationException($"{typeof(TItem)} missing new() method. Please provide {nameof(CreateItemCallback)} create the {typeof(TItem)} instance. {typeof(TItem)} 未提供无参构造函数 new() 请通过 {nameof(CreateItemCallback)} 回调方法创建实例", ex);
+            throw new InvalidOperationException(ErrorMessage, ex);
         }
+        return item!;
     }
 
     /// <summary>

src/BootstrapBlazor/Extensions/ObjectExtensions.cs

@@ -242,37 +242,53 @@ internal static void Clone<TModel>(this TModel source, TModel item)
     /// Creates an instance of a type and ensures all class-type properties are initialized.
     /// </summary>
     /// <typeparam name="TItem">The type to create an instance of.</typeparam>
+    /// <param name="isAutoInitializeModelProperty">Whether to automatically initialize model properties default value is false.</param>
     /// <returns>An instance of the specified type with initialized properties.</returns>
-    public static TItem CreateInstanceWithCascade<TItem>()
+    public static TItem? CreateInstance<TItem>(bool isAutoInitializeModelProperty = false)
     {
         var instance = Activator.CreateInstance<TItem>();
-        instance!.EnsureInitialized();
+        if (isAutoInitializeModelProperty)
+        {
+            instance.EnsureInitialized(isAutoInitializeModelProperty);
+        }
+        return instance;
+    }
+
+    private static object? CreateInstance(Type type, bool isAutoInitializeModelProperty = false)
+    {
+        var instance = Activator.CreateInstance(type);
+        if (isAutoInitializeModelProperty)
+        {
+            instance.EnsureInitialized();
+        }
         return instance;
     }
 
     /// <summary>
     /// Ensures that all class-type properties of the instance are initialized.
     /// </summary>
+    /// <param name="isAutoInitializeModelProperty">Whether to automatically initialize model properties default value is false.</param>
     /// <param name="instance">The instance to initialize properties for.</param>
-    private static void EnsureInitialized(this object instance)
+    private static void EnsureInitialized(this object? instance, bool isAutoInitializeModelProperty = false)
     {
+        if (instance is null)
+        {
+            return;
+        }
+
         // Reflection performance needs to be optimized here
         foreach (var propertyInfo in instance.GetType().GetProperties().Where(p => p.PropertyType.IsClass && p.PropertyType != typeof(string)))
         {
             var type = propertyInfo.PropertyType;
             var value = propertyInfo.GetValue(instance, null);
             if (value is null)
             {
-                var pv = CreateInstance(type);
-                propertyInfo.SetValue(instance, pv);
+                var pv = CreateInstance(type, isAutoInitializeModelProperty);
+                if (pv is not null)
+                {
+                    propertyInfo.SetValue(instance, pv);
+                }
             }
         }
     }
-
-    private static object? CreateInstance(Type type)
-    {
-        var instance = Activator.CreateInstance(type);
-        instance!.EnsureInitialized();
-        return instance;
-    }
 }

test/UnitTest/Components/TableTest.cs

@@ -127,6 +127,7 @@ public async Task Items_Add(InsertRowMode insertMode, bool bind)
         {
             pb.AddChildContent<Table<Foo>>(pb =>
             {
+                pb.Add(a => a.IsAutoInitializeModelProperty, true);
                 pb.Add(a => a.Items, items);
                 if (bind)
                 {

test/UnitTest/Extensions/ObjectExtensionsTest.cs

@@ -303,6 +303,27 @@ public void IsStatic_Ok()
         Assert.True(pi.IsStatic());
     }
 
+    [Fact]
+    public void CreateInstance_Ok()
+    {
+        var exception = Assert.ThrowsAny<Exception>(() => ObjectExtensions.CreateInstance<MockComplexObject>(true));
+
+        var mi = typeof(ObjectExtensions).GetMethod("EnsureInitialized", System.Reflection.BindingFlags.Static | System.Reflection.BindingFlags.NonPublic);
+        Assert.NotNull(mi);
+        mi.Invoke(null, [null, false]);
+
+        var instance = ObjectExtensions.CreateInstance<MockComplexObject>(false);
+        Assert.NotNull(instance);
+        Assert.Null(instance.Test);
+    }
+
+    private class MockComplexObject
+    {
+        public Foo? Foo { get; set; }
+
+        public (string Name, int Count)[]? Test { get; set; }
+    }
+
     private class MockStatic
     {
         private static int _test;