过犹不及

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