Is there a better way to write new List<string> {"a", "b"}.Contains(str) ?

Question

I want to test whether a certain string is contained in a short list of strings. Currently the code is like this:

if (new List<string> { "A", "B", "C" }.Contains (str)) {

However, this seems bloated. For instance, iirc, in Java I could simply write {"A", "B", "C"}.Contains(str) which would be much preferable to the above.

I'm sure there is a better way in C#. Could you point it out?

Answer 1

I guess you could shorten it down to:

if ((new []{ "A", "B", "C" }).Contains (str)) {

Don't know how much of an actual difference it would make though.

Update: if you know that you will be testing for exactly one letter, I see no reason to make a list or array of it:

if ("ABC".Contains(str)) {

That code is both shorter and faster. But then again I guess that the single-letter strings were merely samples...

Answer 2

What about this approach:

"A;B;C".Split(';').Contains(str);

Answer 3

If your short list of strings are constant, you should use a static readonly string array.

The benefit is it's easy to write and it does not instantiate new List every time you need to perform the check.

private static readonly string[] Names = new string[] { "A", "B", "C" };

...

if (Names.Contains(str)) {

However, this solution is not scalable as the search is done in a linear fashion. Alternatively, you can define your constant array in a sorted manner and use BinarySearch over the array.

// this has to be sorted
private static readonly string[] Names = new string[] { "A", "B", "C" };

...

if (Array.BinarySearch(Names, str) >= 0) {

Answer 4

To completely change it up:

switch(str){
    case "A":
    case "B":
    case "C":
       contains = true;
       break;

    default:
       contains = false;
       break;
}

Answer 5

You could write an extension method:

public static bool In<T>(this T obj, params T[] candidates)
{
    return obj.In((IEnumerable<T>)candidates);
}

public static bool In<T>(this T obj, IEnumerable<T> candidates)
{
    if(obj == null) throw new ArgumentNullException("obj");
    return (candidates ?? Enumerable.Empty<T>()).Contains(obj);
}

Which you could then use to do:

if(str.In("A", "B", "C")) { ... }