API

IndexOf function should check the first character

Docs > Code Analysis > Static Analysis Rules > IndexOf function should check the first character

Metadata

ID: csharp-best-practices/indexof-checks

Language: C#

Severity: Warning

Category: Error Prone

Description

When using Indexof or LastIndexOf, using > 0 may miss the first item of the collection (string, list, etc). Instead, the code should use >=0 to take the first element into account.

Non-Compliant Code Examples

using System.Net;

class MyClass {
    public static void routine(string str, string str2)
    {
        str.IndexOf(str2)>0;
        str.IndexOf(str2)>0;
        str.LastIndexOf(str2)>0;
    }
}

Compliant Code Examples

using System.Net;

class MyClass {
    public static void routine(string str, string str2)
    {
        str.IndexOf(str2)>=0;
        str.IndexOf(str2)>=0;
        str.LastIndexOf(str2)>=0;
        if (serverPackages.Any(x => x.Version.CompareTo(currentVersion) > 0)) {
            
        }
    }
}

How to use this rule

1
2
rulesets:
  - csharp-best-practices     # Rules to enforce C# best practices.

Create a static-analysis.datadog.yml with the content above at the root of your repository
Use our free IDE Plugins or add Code Analysis scans to your CI pipelines
Get feedback on your code

For more information, please read the Code Analysis documentation

VS Code Extension

Identify code vulnerabilities directly in your
VS Code editor

JetBrains Plugin

Identify code vulnerabilities directly in
JetBrains products