perf: 优化性能更新单元测试

Author: ArgoZhang

src/BootstrapBlazor/Utils/CssBuilder.cs

@@ -4,6 +4,7 @@
 // Maintainer: Argo Zhang([email protected]) Website: https://www.blazor.zone
 
 using System.Globalization;
+using System.Text;
 
 namespace BootstrapBlazor.Components;
 
@@ -12,7 +13,8 @@ namespace BootstrapBlazor.Components;
 /// </summary>
 public class CssBuilder
 {
-    private readonly List<string> stringBuffer;
+    private readonly StringBuilder _builder = new();
+    private bool _hasConent;
 
     /// <summary>
     /// Creates a CssBuilder used to define conditional CSS classes used in a component.
@@ -28,8 +30,11 @@ public class CssBuilder
     /// <param name="value"></param>
     protected CssBuilder(string? value)
     {
-        stringBuffer = [];
-        AddClass(value);
+        if (!string.IsNullOrEmpty(value))
+        {
+            _builder.Append(value);
+            _hasConent = true;
+        }
     }
 
     /// <summary>
@@ -39,7 +44,18 @@ protected CssBuilder(string? value)
     /// <returns>CssBuilder</returns>
     public CssBuilder AddClass(string? value)
     {
-        if (!string.IsNullOrEmpty(value)) stringBuffer.Add(value);
+        if (!string.IsNullOrEmpty(value))
+        {
+            if (_hasConent)
+            {
+                _builder.Append(' ');
+            }
+            else
+            {
+                _hasConent = true;
+            }
+            _builder.Append(value);
+        }
         return this;
     }
 
@@ -101,8 +117,8 @@ public CssBuilder AddClassFromAttributes(IDictionary<string, object>? additional
     {
         if (additionalAttributes != null && additionalAttributes.TryGetValue("class", out var c))
         {
-            var classList = c?.ToString();
-            AddClass(classList);
+            var classString = Convert.ToString(c, CultureInfo.InvariantCulture);
+            AddClass(classString);
         }
         return this;
     }
@@ -127,5 +143,5 @@ public CssBuilder AddStyleFromAttributes(IDictionary<string, object>? additional
     /// Finalize the completed CSS Classes as a string.
     /// </summary>
     /// <returns>string</returns>
-    public string? Build() => stringBuffer.Count > 0 ? string.Join(" ", stringBuffer) : null;
+    public string? Build() => _hasConent ? _builder.ToString() : null;
 }

test/UnitTest/Utils/CssBuilderTest.cs

@@ -21,51 +21,6 @@ public void AddClass_When()
         Assert.Contains("cls_test", classString);
     }
 
-    [Fact]
-    public void AddStyle_When()
-    {
-        var classString = CssBuilder.Default()
-            .AddStyle("width", () => "cls_test", () => false)
-            .Build();
-        Assert.DoesNotContain("widht: cls_test;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle("width", () => "cls_test", false)
-            .Build();
-        Assert.DoesNotContain("widht: cls_test;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle("width", "cls_test", false)
-            .Build();
-        Assert.DoesNotContain("widht: cls_test;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle("width", () => "cls_test", () => true)
-            .Build();
-        Assert.Contains("width: cls_test;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle("width", () => "cls_test", true)
-            .Build();
-        Assert.Contains("width: cls_test;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle("width", "cls_test", true)
-            .Build();
-        Assert.Contains("width: cls_test;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle("width", "cls_test", () => true)
-            .Build();
-        Assert.Contains("width: cls_test;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle("width", "cls_test_width")
-            .AddStyle("height", "cls_test_height")
-            .Build();
-        Assert.Equal("width: cls_test_width; height: cls_test_height;", classString);
-    }
-
     [Fact]
     public void AddClass_Builder()
     {
@@ -82,27 +37,6 @@ public void AddClass_Builder()
         Assert.Contains("cls_test", classString);
     }
 
-    [Fact]
-    public void AddStyle_Builder()
-    {
-        var builder = CssBuilder.Default("width: cls_test_width;");
-
-        var classString = CssBuilder.Default()
-            .AddStyle(builder, false)
-            .Build();
-        Assert.DoesNotContain("width: cls_test_width;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle(builder, () => true)
-            .Build();
-        Assert.Contains("width: cls_test_width;", classString);
-
-        classString = CssBuilder.Default()
-            .AddStyle(builder, true)
-            .Build();
-        Assert.Contains("width: cls_test_width;", classString);
-    }
-
     [Fact]
     public void AddStyleFromAttributes_Ok()
     {