From e4c621aa708702d03b5bbfc329e26912a2ba1e17 Mon Sep 17 00:00:00 2001 From: bforget163 <33746058+bforget163@users.noreply.github.com> Date: Thu, 17 Apr 2025 15:33:48 +0800 Subject: [PATCH] Update ca2249.md and fix typo in the example code --- .../code-analysis/quality-rules/ca2249.md | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/docs/fundamentals/code-analysis/quality-rules/ca2249.md b/docs/fundamentals/code-analysis/quality-rules/ca2249.md index e3ea320011d16..5a0924a5ef91a 100644 --- a/docs/fundamentals/code-analysis/quality-rules/ca2249.md +++ b/docs/fundamentals/code-analysis/quality-rules/ca2249.md @@ -110,22 +110,22 @@ class MyClass found = str.Contains('x'); // No comparisonType in string overload, adds StringComparison.CurrentCulture to resulting fix - found = !string.Contains("text", StringComparison.CurrentCulture); - found = string.Contains("text", StringComparison.CurrentCulture); + found = !str.Contains("text", StringComparison.CurrentCulture); + found = str.Contains("text", StringComparison.CurrentCulture); // comparisonType equal to StringComparison.Ordinal, removes the argument - found = !string.Contains('x'); - found = string.Contains('x'); + found = !str.Contains('x'); + found = str.Contains('x'); - found = !string.Contains("text"); - found = string.Contains("text"); + found = !str.Contains("text"); + found = str.Contains("text"); // comparisonType different than StringComparison.Ordinal, preserves the argument - ;found = !string.Contains('x', StringComparison.OrdinalIgnoreCase) - found = string.Contains('x', StringComparison.CurrentCulture); + found = !str.Contains('x', StringComparison.OrdinalIgnoreCase); + found = str.Contains('x', StringComparison.CurrentCulture); - found = !string.Contains("text", StringComparison.InvariantCultureIgnoreCase); - found = string.Contains("text", StringComparison.InvariantCulture); + found = !str.Contains("text", StringComparison.InvariantCultureIgnoreCase); + found = str.Contains("text", StringComparison.InvariantCulture); // This case had to be manually fixed if (!str.Contains("text"))