过犹不及

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

转载于:https://www.cnblogs.com/MichaelPeng/archive/2009/12/24/1631719.html