简化策略模型的代码

在我们编写代码时，经常遇到一次策略模式（俺不会背那个啥设计模式，暂时叫他策略模式吧），例如，在反序列化时，已知一个名称和命名空间，获取其对应的类型，使用下面的策略：

尝试从绑定期中获取，

如果不成功，尝试从基类获取；

如果还不成功，尝试播发事件获取。

看起来，一个个尝试，如果不成功，下一个。代码是这个样子的。

?

private IEntityType BindToType(XElement element,IEntityType baseType, out IEnumerable<DcxmlBinder.CustomAttribute> attributes)

{

    //第一级策略

    var name = element.Name;

    attributes = from p in element.Attributes()

                 select new DcxmlBinder.CustomAttribute() { Namespace = p.Name.NamespaceName, Name = p.Name.LocalName, Value = p.Value };

    IEntityType dt = _binder.BindToType(name.NamespaceName, name.LocalName,attributes);

 

    if (dt == null)

    {

        //第二级策略:如果没有，看看是否有缺省类型。

        if (baseType != null)

        {

            //检查此类型的确正好是此名称

            string n1, n2;

            _binder.BindToName(baseType, out n1, out n2);

            if (string.Equals(name.NamespaceName, n1, _binder.StringComparison) &&

                string.Equals(name.LocalName, n2, _binder.StringComparison))

            {

                //不用检测派生关系，直接返回。

                return baseType;

            }

        }

         

        //第三级策略

        //TODO:应该先播发event System.Xml.Serialization.XmlElementEventHandler UnknownElement事件。外界获取类型。

        throw new ApplicationException();

    }

 

    //检查派生关系是否正确，即dt必须是baseDt的派生类型。

    if ((baseType != null) && (!baseType.IsAssignableFrom(dt)))

    {

        throw new ApplicationException("设计错误，返回的数据类型没有派生自baseType");

    }

 

    return dt;

}

　　

有点乱糟糟的，老实说，我也觉得乱糟糟的。于是我也想优化这样的代码，最终，我利用了C# 的或短路的特性，代码如下：

?

private IEntityType BindToType(XElement element,IEntityType baseType, out IEnumerable<DcxmlBinder.CustomAttribute> attributes)

        {

            IEntityType dt;

            if (TryBindFromBinder(element,out dt,out attributes) ||

                TryBindFromBase(element,baseType,out dt) ||

                TryBindFromEvent(element,baseType,out dt))

            {

                //检查派生关系是否正确，即dt必须是baseDt的派生类型。

                if ((baseType != null) && (!baseType.IsAssignableFrom(dt)))

                {

                    throw new ApplicationException("设计错误，返回的数据类型没有派生自baseType");

                }

            }

             

            return dt;

        }

 

        private bool TryBindFromBinder(XElement element, out IEntityType dt, out IEnumerable<DcxmlBinder.CustomAttribute> attributes)

        {

            //第一级策略

            var name = element.Name;

            attributes = from p in element.Attributes()

                         select new DcxmlBinder.CustomAttribute() { Namespace = p.Name.NamespaceName, Name = p.Name.LocalName, Value = p.Value };

            dt = _binder.BindToType(name.NamespaceName, name.LocalName,attributes);

 

            return (dt != null);

        }

 

        private bool TryBindFromBase(XElement element, IEntityType baseType, out IEntityType dt)

        {

            //第二级策略:如果没有，看看是否有缺省类型。

            if (baseType != null)

            {

                var name = element.Name;

                //检查此类型的确正好是此名称

                string n1, n2;

                _binder.BindToName(baseType, out n1, out n2);

                if (string.Equals(name.NamespaceName, n1, _binder.StringComparison) &&

                    string.Equals(name.LocalName, n2, _binder.StringComparison))

                {

                    //不用检测派生关系，直接返回。

                    dt = baseType;

                    return true;

                }

            }

 

            dt = null;

            return false;

        }

 

        private bool TryBindFromEvent(XElement element, IEntityType baseType, out IEntityType dt)

        {

            //第三级策略

            //TODO:应该先播发event System.Xml.Serialization.XmlElementEventHandler UnknownElement事件。外界获取类型。

            throw new ApplicationException();

        }

　　虽然代码比以前多了，但是看起来清晰了很多。而且，通常第一个策略都能完成搜索，所以就不会调用后面的方法，也就不会初始化其变量了。