CA1820: Test for empty strings using string length

TypeName TestForEmptyStringsUsingStringLength
CheckId CA1820
Category Microsoft.Performance
Breaking change Non-breaking

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 MSIL 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.

Example

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

using System;

namespace PerformanceLibrary
{
   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.");
         }
      }
   }
}