过犹不及
在程序中有一個HandlerManager類需要管理一些對象，這些對象都從同一個基類 IHandler派生，擁有MethodName和TypeName兩個字符串屬性。Manager類所做的主要工作就是提供兩個方法，一個添加 IHandler,一個通過MethodName和TypeName查找IHandler
interface?IHandlerManager
{
????public?void?AddHandler(IHandler);
????public?IHandler?GetHandler(String?methodName,?String?typeName);
}
因為一般情況下添加的Handler不會超過二十個，所以我沒有做什么優化,就簡單的實現如下
{
????private?List<IHandler>?m_handlers?=?new?List<IHandler>();
????public?void?AddHandler(IHandler?handler)
????{
????????m_handlers.Add(handler);
????}
????public?IHandler?GetHandler(String?methodName,?String?typeName)
????{
????????foreach?(IHandler?handler?in?m_handlers)
????????{
????????????if?(handler.MethodName?==?methodName?
????????????????????&&?handler.TypeName?==?typeName)
????????????{
????????????????return?handler;
????????????}
????????}
????????return?null;
????}
}
新來一個同事，就說這代碼簡單循環，直接比較字符串，看著很丑陋。想改進一下。我問怎么改進，他說可以用Dictionary.這里用Dictionary就很合適么？因為有兩個字段需要比較，這兩個字段怎么插入Dictionary做Key？
?
一種方法是用wrapper類把兩個字符串包裝到一個類里面做key,這樣就需要引入一個額外的wrapper類，這個wrapper類還需要實現Equal方法和GetHashCode方法。
?
另外一種方法是用嵌套的Dictionary,即Dictionary<String, Dictionary<String, IHandler>>.外層Dictionary把MethodName索引到一個內層的Dictionary<String, IHandler>。內層的Dictionary則把TypeName映射到IHandler對象。這樣的一個問題每次需要判斷外層 MethodName鍵值是否存在，直接dict[methodName][typeName]會引發空引用異常。
?
其實這里直接比較是最簡單的實現方法。不超過二十個元素，順序查找并不比Dictionary慢多少，這里的代碼又非常簡明易懂，誰都能一眼看明白。為了追求"優雅"的設計而花額外精力做一個wrapper類，或者使用嵌套Dictionary而增加額外鍵值存在判斷，都會引入額外的代碼復雜度，而為這么一個小接口花這些額外的力氣，實在不值。設計的度一定要把握好，過猶不及。
轉載于:https://www.cnblogs.com/MichaelPeng/archive/2009/12/24/1631719.html
總結
