Potential fix for code scanning alert no. 199: Inefficient use of ContainsKey

csharpfritz · github-advanced-security[bot] · web-flow · commit 733cf9086908 · 2026-02-01T17:37:23.000-05:00
Co-authored-by: Copilot Autofix powered by AI &lt;62310815+github-advanced-security[bot]@users.noreply.github.com&gt;
diff --git a/src/BlazorWebFormsComponents/CustomControls/HtmlTextWriter.cs b/src/BlazorWebFormsComponents/CustomControls/HtmlTextWriter.cs
@@ -71,9 +71,8 @@ public void AddAttribute(string name, string value)
 				throw new ArgumentNullException(nameof(name));
 
 			// Special handling for class attribute - concatenate instead of replace
-			if (name.Equals("class", StringComparison.OrdinalIgnoreCase) && _pendingAttributes.ContainsKey("class"))
+			if (name.Equals("class", StringComparison.OrdinalIgnoreCase) && _pendingAttributes.TryGetValue("class", out var existingClass))
 			{
-				var existingClass = _pendingAttributes["class"];
 				if (!string.IsNullOrEmpty(existingClass) && !string.IsNullOrEmpty(value))
 				{
 					_pendingAttributes["class"] = $"{existingClass} {value}";

Original file line number	Diff line number	Diff line change
`@@ -71,9 +71,8 @@ public void AddAttribute(string name, string value)`
`71`	`71`	`throw new ArgumentNullException(nameof(name));`
`72`	`72`
`73`	`73`	`// Special handling for class attribute - concatenate instead of replace`
`74`		`- if (name.Equals("class", StringComparison.OrdinalIgnoreCase) && _pendingAttributes.ContainsKey("class"))`
	`74`	`+ if (name.Equals("class", StringComparison.OrdinalIgnoreCase) && _pendingAttributes.TryGetValue("class", out var existingClass))`
`75`	`75`	`{`
`76`		`- var existingClass = _pendingAttributes["class"];`
`77`	`76`	`if (!string.IsNullOrEmpty(existingClass) && !string.IsNullOrEmpty(value))`
`78`	`77`	`{`
`79`	`78`	`_pendingAttributes["class"] = $"{existingClass} {value}";`