Merge pull request #26 from marthijn/feature/objectpool

⚡️ Optimize StringBuilder usage with ObjectPool

Author: marthijn

src/Sidio.OpenGraph/OpenGraphBuilder.cs

@@ -1,5 +1,6 @@
 using System.Text;
 using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.ObjectPool;
 
 namespace Sidio.OpenGraph;
 
@@ -77,13 +78,22 @@ public void AddRange(IEnumerable<OpenGraphMetaTag> openGraphMetas)
     /// <inheritdoc />
     public string GetPrefixAttributeValue()
     {
-        var sb = new StringBuilder();
-        foreach (var ns in Namespaces)
+        var pool = ObjectPool.Create<StringBuilder>();
+        var sb = pool.Get();
+
+        try
         {
-            sb.Append($"{ns.Prefix}: {ns.SchemaUri} ");
-        }
+            foreach (var ns in Namespaces)
+            {
+                sb.Append($"{ns.Prefix}: {ns.SchemaUri} ");
+            }
 
-        return sb.ToString().Trim();
+            return sb.ToString().Trim();
+        }
+        finally
+        {
+            pool.Return(sb);
+        }
     }
 
     /// <inheritdoc />

src/Sidio.OpenGraph/OpenGraphExtensions.cs

@@ -1,4 +1,5 @@
 using System.Text;
+using Microsoft.Extensions.ObjectPool;
 
 namespace Sidio.OpenGraph;
 
@@ -14,12 +15,21 @@ public static class OpenGraphExtensions
     /// <returns>A HTML <see cref="string"/>.</returns>
     public static string MetaTagsToHtml(this OpenGraph openGraph)
     {
-        var builder = new StringBuilder();
-        foreach (var property in openGraph.MetaTags)
+        var pool = ObjectPool.Create<StringBuilder>();
+        var builder = pool.Get();
+
+        try
         {
-            builder.Append($"<meta property=\"{property.Property}\" content=\"{property.Content}\" />");
-        }
+            foreach (var property in openGraph.MetaTags)
+            {
+                builder.Append($"<meta property=\"{property.Property}\" content=\"{property.Content}\" />");
+            }
 
-        return builder.ToString();
+            return builder.ToString();
+        }
+        finally
+        {
+            pool.Return(builder);
+        }
     }
 }

src/Sidio.OpenGraph/Sidio.OpenGraph.csproj

@@ -25,6 +25,7 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="9.0.7" />
+    <PackageReference Include="Microsoft.Extensions.ObjectPool" Version="9.0.8" />
     <PackageReference Include="Microsoft.SourceLink.GitHub" Version="8.0.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>