需要选择设计模式的帮助

Question

目前我有一堆if if语句来根据每个集合中有多少项目来设置CategoryId。

例如，

public class TeamWork 
{ 
    public string EmployeeName { get; set; } 
    public int CategoryId { get; set; } 
} 

public class BLL 
{ 
    public void SetCategoryId(ICollection<TeamWork> Converted, ICollection<TeamWork> Sourced) 
    { 
     if (Converted.Count == 1 && Sourced.Count == 1) 
     {     
      if (String.Compare(Sourced.First().EmployeeName, Converted.First().EmployeeName) == 0) 
      { 
       // set category id to 1 
       Converted.First().CategoryId = 1; 
       Sourced.First().CategoryId = 1;            
      } 
      else 
      { 
       // set category id to something     
      } 
     } 
     else if (Sourced.Rows.Count == 1 && Converted.Rows.Count > 1) 
     { 
      // set category id to something   
     } 
     // more if else statements... 
    } 
} 

我想有一个更好的方式，通过应用一些设计模式来做到这一点也许。有什么建议么？谢谢！

Answer 1

Chain of responsibility是要走的路。

所以这个对象被传递给一系列的命令对象，直到一个能够执行和设置状态。

Answer 2

一个战略模式浮现在脑海。试着将这些规则分解成一系列“如果这个条件是真的，那么类别ID就是这个”。使这些方法中的每一个都成为方法，然后将这些方法作为代理添加到List<Func<ICollection<TeamWork>, ICollection<TeamWork>, bool>>或可比较的索引集合中。然后，您SetCategoryId（）的代码如下所示：

public void SetCategoryId(ICollection<TeamWork> Converted, ICollection<TeamWork> Sourced) 
{ 
    foreach(var categoryRule in CategoryRules) 
    { 
     var category = test(Converted, Sourced); 
     if(category != 0) 
     { 
      Converted.First().CategoryId = Sourced.First().CategoryId = category; 
      break; 
     } 
    } 
} 

上面的代码就永远不会不管改变您添加或删除多少规则。但是，对于if-else结构，您的一系列规则可能会依赖于顺序，因此在设置列表中的规则时要小心。