Question

I have a method similar to the following:

StringComparison comparison = StringComparison.CurrentCultureIgnoreCase;

switch(SearchType)
{
case SearchType.Contains:
list = list.Where(a => a.Reference.Contains("test",comparison));
break;

case SearchType.StartsWith:
list = list.Where(a => a.Reference.StartsWith("test",comparison));
break;

case SearchType.EndsWith:
list = list.Where(a => a.Reference.EndsWith("test",comparison));
break;
}

As you can probably guess SearchType is a custom enum I have.

Is there an easier way of doing this, possibily using reflection? The multiple switches seem a bit ugly.

Answer 1

Which part of that is not easy?

I guess you could have a class for each enum instead, inherit from a common SearchType interface and implement a function called ProcessList – not sure what list is to offer a better function.

Something like..

interface SearchType
{
   object ProcessList(object list, string text);
}

class Contains : SearchType
{
   object ProcessList(object list, string text)
   {
      list = list.Where(a => a.Reference.Contains(text, StringComparison.CurrentCultureIgnoreCase));
   }
}

Need to do a class for each enum type.

Then you would need to set the SearchType variable like this…

SearchType searchType = new Contains();//or something else

and your switch could then be replaced with this…

list = searchType.ProcessList(list, "test");

…Not really easier in terms of coding, but you get a more readable code instead of the switch.