.Net 4.0 Optimized code for refactoring existing “if” conditions and “is” operator

Question

I have following C# code. It works fine; but the GetDestination() method is cluttered with multiple if conditions by using is operator.

In

Answer 1

One way to do it would be to use a map instead of an if:

//(psuedocode)
private Dictionary RoleMap;

void SomeInitializationCodeThatRunsOnce()
{
  RoleMap.Add(typeof(Manager), @"\ManagerHome");
  RollMap.Add(typeof(Accountant), @"\AccountantHome");
  // ect...
}

string GetDestination(Role x)
{
  string destination;
  if(!RoleMap.TryGet(x.GetType(), out destination))
    destination = @"\Home";
  return destination;
}

Further reading: http://www.hanselman.com/blog/BackToBasicsMovingBeyondForIfAndSwitch.aspx