CA1820: Test for empty strings using string length

Artikkel
09/05/2023

Property	Value
Rule ID	CA1820
Title	Test for empty strings using string length
Category	Performance
Fix is breaking or non-breaking	Non-breaking
Enabled by default in .NET 9	No

Cause

A string is compared to the empty string by using Object.Equals.

Rule description

Comparing strings using the String.Length property or the String.IsNullOrEmpty method is faster than using Equals. This is because Equals executes significantly more CIL instructions than either IsNullOrEmpty or the number of instructions executed to retrieve the Length property value and compare it to zero.

For null strings, Equals and <string>.Length == 0 behave differently. If you try to get the value of the Length property on a null string, the common language runtime throws a System.NullReferenceException. If you perform a comparison between a null string and the empty string, the common language runtime does not throw an exception and returns false. Testing for null does not significantly affect the relative performance of these two approaches. When targeting .NET Framework 2.0 or later, use the IsNullOrEmpty method. Otherwise, use the Length == 0 comparison whenever possible.

How to fix violations

To fix a violation of this rule, change the comparison to use the IsNullOrEmpty method.

When to suppress warnings

It's safe to suppress a warning from this rule if performance is not an issue.

Suppress a warning

If you just want to suppress a single violation, add preprocessor directives to your source file to disable and then re-enable the rule.

#pragma warning disable CA1820
// The code that's violating the rule is on this line.
#pragma warning restore CA1820

To disable the rule for a file, folder, or project, set its severity to none in the configuration file.

[*.{cs,vb}]
dotnet_diagnostic.CA1820.severity = none

For more information, see How to suppress code analysis warnings.

Example

The following example illustrates the different techniques that are used to look for an empty string.

public class StringTester
{
    string s1 = "test";

    public void EqualsTest()
    {
        // Violates rule: TestForEmptyStringsUsingStringLength.
        if (s1 == "")
        {
            Console.WriteLine("s1 equals empty string.");
        }
    }

    // Use for .NET Framework 1.0 and 1.1.
    public void LengthTest()
    {
        // Satisfies rule: TestForEmptyStringsUsingStringLength.
        if (s1 != null && s1.Length == 0)
        {
            Console.WriteLine("s1.Length == 0.");
        }
    }

    // Use for .NET Framework 2.0.
    public void NullOrEmptyTest()
    {
        // Satisfies rule: TestForEmptyStringsUsingStringLength.
        if (!String.IsNullOrEmpty(s1))
        {
            Console.WriteLine("s1 != null and s1.Length != 0.");
        }
    }
}

Del via