옵션 구성 요소 기능 대 SRP

Question

현재 발생하는 디자인 문제가 있습니다.

구성 요소의 계층 구조가 있다고 가정 해 보겠습니다. 이러한 구성 요소의 각이 같은 보이는 추상적 인 Component 유형에서 파생 :

public abstract class Component 
{ 
    public abstract Component Parent { get; } 
    public abstract ComponentCollection Children { get; } 
} 

가 지금은 이러한 구성 요소에 몇 가지 옵션 기능을 추가 할을, 구성 요소 계층 구조 내에서 검색 할 수있는 타고 내에서 구성 요소를 선택할 수 있습니다 예를 들어 계층 구조. 는 AND "를 선택-능력을" "-능력을 검색하는 것은"예에 의해 파생 된 구성 요소에서 관리되는 동안

public abstract class Component 
{ 
    // Other members 

    public abstract bool IsSearchable { get; } 
    public abstract bool Search(string searchTerm); 

    public abstract bool IsSelectable { get; } 
    public abstract bool Select(); 
} 

:

는이 같은 기본 클래스에서 그 옵션 기능을 제공하기 위해 나쁜 관행으로 간주됩니다 전략 패턴 사용?

어떻게 든 내게는 SRP 위반 인 것처럼 보이지만 제 생각에는 유일한 대안은 각 선택적 기능에 대한 인터페이스를 갖고이 기능을 지원하는 구성 요소에서만 구현하는 것입니다.

public bool Search(Component component, string searchTerm) 
{ 
    ISearchable searchable = component as ISearchable; 
    if(searchable != null) 
    { 
     searchable.Search(searchTerm); 
    } 
} 

당신이 선택하거나 할 것이 전략 어떤이

: 구성 요소가 특정 기능을 제공하는 경우이 내가 확인하려면이 매번 같은 코드를 작성해야하는 단점이있을 것입니다 내 생각에

더 나은 아이디어?

미리 감사드립니다.

Answer 1

가능한 옵션 : 검색 가능성/선택성의 구현 전략 패턴 (의존성 주입)을 통해 제공되는 경우 당신이 말한대로

이

, 그때 ISearchable 및 ISelectable에 대한 인터페이스가 더 나은 아이디어를 생각합니다.

이러한 인터페이스에서 전략 객체를 파생시키고 기본 구성 요소 클래스에서 getter를 구현할 수 있습니다. GetSearchable(), GetSelectable() - Component의 기본 구현이 null을 반환하거나 만약 당신이 null을 싫어하는 인터페이스).

Answer 2

데코레이터를 사용하지 않는 이유는 무엇입니까?

Component c = new Component(); 
var selectableAndSearchableOne = new SelectableComponent (new SearchableComponent (c)); 

Answer 3

다른 한가지 : 이번에는 구성 요소의 확장 점도 알고 있습니다. 방문자와 비슷한 패턴으로

public interface IHasExtensions 
    { 
     List<Extension> Extensions { get; } 
     void Extend (Extension ext); 
    } 

    public class Component : IHasExtensions 
    { 
     List<Extension> exts = new List<Extension>(); 

     public List<Extension> Extensions 
     { 
      get { return exts; } 
     } 

     public void Extend (Extension ext) 
     { 
      exts.Add (ext); 
     } 

     void Draw() { } 
    } 

    public abstract class Extension 
    { 
     readonly protected Component _Component; 

     public Extension(Component component) 
     { 
      _Component = component; 
     } 
    } 

    public class SearchExtension : Extension 
    { 
     public SearchExtension (Component component) : base (component) 
     { 

     } 
    } 

    public class SelectionExtension : Extension 
    { 
     public SelectionExtension (Component component) : base (component) 
     { 

     } 
    } 

    public class test_fly 
    { 
     void start() 
     { 
      Component c = new Component(); 
      c.Extend (new SearchExtension (c)); 
      c.Extend (new SelectionExtension (c)); 

      var exts = c.Extensions; // I Know the extensions now 
     } 
    }