디자인 패턴을 선택할 때 도움이 필요합니다.

Question

현재 각 컬렉션에 몇 개의 항목이 있는지에 따라 CategoryId를 설정하는 else 문이 여러 개 있습니다. 예를 들어

,

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 if 구조를 사용하면 일련의 규칙이 순서에 따라 달라 지므로 목록에 규칙을 설정할 때주의해야합니다.