Question

鉴于： 复杂的算法。

switch obj.field1 + "-" + obj.field2
case "A-A":
    return Sample(obj){Error = "Type can't be matched"}
case "A-B":
    {
    if(obj.field3 != null){
        var match = otherObjectCollection.FirstOrDefault(otherOb.field2 == obj.field3);
        return mathc != null? Sample(obj,match) : Sample(obj){Error="Can't find match"}
    }else{use field4...}
}
case "A-C":
{
var related = objectCollection.FirstOrDefault(parent.field4 == obj.field3);
if(related == null)
 return Sample(obj){Error = "Can't find parent"}
else {
  if(related.field3 != null){
       var match = otherObjectCollection.FirstOrDefault(otherOb.field2 == related.field3);
       return mathc != null? Samble(obj,match) : Sample(obj){Error="Can't find match"}
  }else{ use field 4 ...}

}
    }

等等。很多棘手的规则。我想出了下一个决定：

abstract class AggregationChain{
    protected MyObj Obj;
    abstract string Type{get;}
    abstract Priority Priority{get;}
    abstract bool Decide(MyObj obj);
    abstract Sample Aggregate(ICollection<MyObj> objects,ICollection<OtherObj> otherobjects);
    bool CanDecide(MyObj obj){
        if(Decide(obj)){
           Obj = obj;
           return true;
        }
        return false;
    }
 }

所以现在我可以使用ChainLinks：

 class ABAggregationChainLink{

        string Type{get{return "A-B"}}
        Priority Priority{get{return Priority.High}}
        bool Decide(MyObj obj){ 
           return obj.fiel2 != null;
        }
        Sample Aggregate(ICollection<MyObj> objects,ICollection<OtherObj> otherobjects){
            var match = OtherObjectCollection.FirstOrDefault(otherOb.field2 == obj.field3);
            return mathc != null? Samble(obj,match) : Sample(obj){Error="Can't find match"}
        }

     }

在这个例子中，我需要再创建一个A-B ChainLink来处理＆＃34; else＆＃34;情况。对于所有交换机情况，我需要创建不同的ChainLink。这当然增加了类的数量和实现的时间，但是更适合于单元测试的类，从我的观点来看，更加可靠和灵活。

问题：

我在思考 - 可能是我被Open-Close和＃34; Good过度兴奋编程＆＃34;而对于现实生活中的应用，最好只是创造有开关的方法和照顾可重复使用的部件？
我有填写这里可能是更好的解决方案吗？

PS。这不是C＃代码，我只是试图解释一下主要的逻辑。

Answer 1

在分析这样的事情时，你不能只看它现在是什么。

该算法是否可以轻松维护？
它有多容易在不改变现有功能的情况下添加新规则？
多么容易是测试吗？
您是否需要在运行时添加新规则（插件架构）？

在3个案例中有一些if elses，它可能并不重要。在40例子案例和新要求都需要单元测试？我更喜欢按规则清理隔离类（或者更可能是将lambda表达式添加到规则集合中的泛型类）

您的链接类看起来没问题，但您仍然需要链类来保存所有链接并确定要处理的链接。这最终可能会有一个结构替换掉这样的开关语句，如：

Dictionary<String, Dictionary<String, List<Link>>> index;

if (index.ContainsKey(obj.Field1))
{
   var subIndex = index[obj.Field1];
   if (subIndex.ContainsKey(obj.Field2))
   {
      var linkList = subIndex[obj.Field2];
      foreach(var Link in linkList)
      {
          if (Link.Decide(obj))
          {
              return Link.Aggregate(objects, otherObjects);
          } 
      }
   }
}

如果您将决定签名更改为Func<bool, MyObj>和汇总Func<Sample, ICollection<MyObj>,ICollection<OtherObj>>

然后你可以添加像

这样的规则

RuleChain.AddRule("A", "A", (o) => true, (objs, others) => Sample(obj){Error = "Type can't be matched"})
RuleChain.AddRule("A", "B",...

现实生活中的程序员应该有多少OOP_Crazy？

1 个答案: