Using Action dictionaries instead of switch statements
<p>I'm just reviewing some of my old code (have some spare time), and I noticed a rather lengthy switch statement.  Due to gaining new knowledge, I have since refactored it in the following form:</p>

<pre><code>private Dictionary&lt;string, Action&gt; createView
    {
        get
        {
            return new Dictionary&lt;string, Action&gt;()
            {
                {"Standard", CreateStudySummaryView},
                {"By Group", CreateStudySummaryByGroupView},
                {"By Group/Time", CreateViewGroupByHour}
            };
        }
    }
</code></pre>

<p>Would you consider this good practise, or is this simply a case of being superflous and unneccessary?  I'm keen to ensure new techniques that I learn, are not being clever just for the sake of it, and that they actually add benefit to the code.   </p>

<p>Thanks.</p>
