需要選擇設計模式的幫助

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結構，您的一系列規則可能會依賴於順序，因此在設置列表中的規則時要小心。