org.sonar.plugins.csharp.S3256.html Maven / Gradle / Ivy
Why is this an issue?
Using string.Equals
to determine if a string is empty is significantly slower than using string.IsNullOrEmpty()
or
checking for string.Length == 0
. string.IsNullOrEmpty()
is both clear and concise, and therefore preferred to laborious,
error-prone, manual null- and emptiness-checking.
Noncompliant code example
"".Equals(name); // Noncompliant
!name.Equals(""); // Noncompliant
name.Equals(string.Empty); // Noncompliant
Compliant solution
name != null && name.Length > 0 // Compliant but more error prone
!string.IsNullOrEmpty(name)
string.IsNullOrEmpty(name)
© 2015 - 2024 Weber Informatics LLC | Privacy Policy