在程序中有一个HandlerManager类需要管理一些对象，这些对象都从同一个基类 IHandler派生，拥有MethodName和TypeName两个字符串属性。Manager类所做的主要工作就是提供两个方法，一个添加 IHandler,一个通过MethodName和TypeName查找IHandler

interface

 IHandlerManager

{

    

public

 

void

 AddHandler(IHandler);

    

public

 IHandler GetHandler(String methodName, String typeName);

}

因为一般情况下添加的Handler不会超过二十个，所以我没有做什么优化,就简单的实现如下

class

 HandlerManager:IHandlerManager

{

    

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而增加额外键值存在判断，都会引入额外的代码复杂度，而为这么一个小接口花这些额外的力气，实在不值。设计的度一定要把握好，过犹不及。