Daniel
Daniel

Reputation: 181

collection/string .Contains vs collection/string.IndexOf

Is there a reason to use .Contains on a string/list instead of .IndexOf? Most code that I would write using .Contains would shortly after need the index of the item and therefore would have to do both statements. But why not both in one?

if ((index = blah.IndexOf(something) >= 0)
    // i know that Contains is true and i also have the index

Upvotes: 5

Views: 1717

Answers (3)

Matt Hamilton
Matt Hamilton

Reputation: 204129

One reason you'd want to use Contains is that it has an overload which lets you specify an IEqualityComparer<String> (in case you want the comparison to be case-insensitive or something). IndexOf has no such overload.

Upvotes: 2

luke
luke

Reputation: 14788

You are right that IndexOf is a more general operation than Contains. However, Contains is still very useful because it represents the operation explicitly.

if(blah.IndexOf(something) >=0)
{
}

isn't as obvious an operation as

if(blah.Contains(something))
{
}

so if you need the index, then you should use the IndexOf operation, if you only need to know if the string contains the substring then use Contains.

Use the tool for the job it was created for.

Upvotes: 7

Keith Nicholas
Keith Nicholas

Reputation: 44288

they are different functions intended for different purposes, but you can use IndexOf to mimic Contains.

The reason to use Contains? In Linq, especially when translated into sql, gets turned into a relatively simple sql statement, IndexOf is more complicated to translate

Upvotes: 0

Related Questions