2010-12-17 3 views
3

내 응용 프로그램에서 분리 된 동작 조각 방법을 고민하고 있습니다. 기본적으로 사용자가 수행 할 작업을 선택하는 양식이 있습니다. 어떤 행동은 혼자 서 있고 내가 순서대로 수행 한 대기열에 쉽게 갈 수 있습니다.Winforms 응용 프로그램에서 변수 그룹 작업을 처리하는 방법에 대한 아이디어가 필요합니다.

Create Website 
- DoesWebsiteExists (Validation) 
- CreatePhyiscalDir 
- CreateWebsite 
- CreateVirDirectories 

이러한 모든 행동이 분리되어 서로에 대해 모르는 그러나, 일부 작업은 기본 작업이 다른 작업을 필요로 먼저 처리하기 위해 여러 관련 작업을 필요로한다.

원래 동작에서 유효성 검사를 분리하기 전에 모든 동작과 필요한 반복 설정을 수행하는 설정 개체를 전달하고 반복 작업을 수행하는 큐를 만들고 각 동작에 대해 Execute() 메서드를 호출했습니다.

Queue<IAction> Actions 
    [0] = CreateDirectory.Execute() 
    [1] = CreateWebsite.Execute() 

하지만 이제는 이러한 모든 작업을 제거 했으므로 어떻게 재구성 할 수 있는지 잘 모르겠습니다.

목록 목록이 있습니까? 어디에서 조치가 독립 실행 형 조치 일 경우 1 요소의 목록일까요? 그리고 4 액션 (4 개의 다른 객체)이 필요한 액션 인 경우 4 개의 요소 목록이됩니다.

거기에 추가 레이어를 던지려면 - 폼에서 선택한 선택에 따라 주요 액션이 필요하거나 때로는 액션이 ​​필요할 수도 있습니다. 예를 들어 사용자가 IIS에 응용 프로그램 풀을 만들도록 선택한 경우 목록에 있거나 없을 수 있으며 목록의 우선 순위를 지정할 수 있습니다.

아이디어가 있으십니까?

답변

3

IAction 인터페이스 (액션 대원과 혼동해서는 안 됨)는 Command Pattern의 구현과 매우 유사합니다. 과거에는 명령을 함께 그룹화 (.NET v1.0)하기를 원했을 때 Composite 패턴을 만들어 복합 명령을 작성하거나 사용자의 경우 복합 액션을 작성했습니다. 그래서 그 구현 세부 사항은 Add 메소드를 가지며 Execute는 다음과 유사합니다.

이것은 목록의 아이디어 목록과 일치합니다.

이 작업을 수행 할 수 있지만 Execute가 void를 반환하기 때문에 명령 체인이 작동하지 않게하는 유일한 방법은 예외를 throw하거나 이상한 커플 링을 사용하는 것입니다 (DoesWebSiteExist에서 일부 IsValid 값을 업데이트 한 다음 모든 Action에서 해당 값을 검사합니다.)

대신이 방법을 좀 더 깔끔하게 처리 할 수있는 Chain of responsibility을 구현할 수 있습니다. 그냥 부울을 반환하는 기존의 방법을 수정할 수있는 경우

또 다른 대안, 당신이 Func을

를 사용하여 너무 많은 어려움없이 원하는 결과를 얻을 수 있습니다 여기에 좋은 것은 방법이 부울을 반환 한 그대로 이 구조에 참여할 수 있습니다. 나쁜 것은 인터페이스를 통해 강제되지 않기 때문에 메소드가 목적을 명확하게 가지지 않아 코드에 약간의 냄새가 있음을 의미합니다.

우선 순위를 정하면 우선 건물을 세우고 우선 순위를 정하는 것이 좋습니다.

업데이트List<Func<bool>의 방법을 명확히하기 위해 어디에서든지 가능합니다. 예를 들어 당신이이만큼 당신이 범위에 CPD와 같이 작동합니다

CreatePhyiscalDir cpd = new CreatePhyiscalDir(); 
    CompositeActions.Add(() => DoesWebsiteExists()); 
    CompositeActions.Add(() => cpd.Execute()); 

이 클래스

public class CreatePhyiscalDir 
    { 

     public bool Execute() 
     { 
      Console.WriteLine("CreatePhyiscalDir"); 
      return true; 
     } 
    } 

당신은 위의 내용을 변경할 수 있었다면. 나는이 강도가 List<Func<bool>는 것을 강조하고 싶습니다

은 ...

  1. 그것은 당신이 거의 모든 것을 할 수있는 부울을 리턴하는 메소드 호출에서 구조를 만들 수 있습니다. 이는 기존 코드 기반이있는 경우 특히 유용합니다.
  2. 이전 작업이 성공하지 않는 한 다음 작업이 실행되지 않도록 종속성을 만듭니다. 당신은 자신이 return true; 그냥 달리 없을 것 구조에 맞는 방법을 쓰기 찾을 수 있습니다

    그러나 약점입니다 ....

  3. 메서드 또는 클래스 간의 관계가 명확하지 않습니다.

그래서 당신은 세 가지 옵션이 의무 일 또는 파이프 라인의 List<Func<bool>> 솔루션 또는 복합 Actionl SnOrfus 같은 대답 같은 의사 파이프 라인 구현 체인의 실제 구현이있다. 당신이 선택한 것은 아마 지금까지 작성한 내용과 변경해야하는 부분에 따라 다릅니다.

마지막 생각. 먼저 각 액션이 다음 액션을 취해야하는지 또는 루핑 코드를 결정할지 결정할 수 있습니다. 그것은 당신이 어떻게 구현하는지 알려줄 수 있습니다.

+0

감사합니다. 이것은 제가 생각한 것과 비슷한 (여러분이 지적한 것처럼) 다소입니다. 당신의 예제에서 저를 괴롭히는 유일한 부분은 여러 가지 방법과 그것들이 캡슐화되지 않은 방법을 보는 것입니다. 이 작업을 개별 개체 (예 : Iis6CreateWebsite)로 생각했습니다.함수를 목록에 추가하는 예제를 통해 단일 클래스의 메서드에서 가져온 것입니다. 문제는 IIS5/6/7 용 CreateWebsite (매개 변수)를 갖게된다는 것입니다. 내가 뭘 놓치고 있니? – pghtech

+0

작업을 클래스로 캡슐화 할 수 있습니다. 내 대답을 업데이트했습니다. 희망적으로이를 보여줍니다. –

+0

감사합니다. 글쎄, 새로운 프로그래머가되고 좋은 OO 원칙을 철저히 준수하려고 노력할 때 나는 각 Execute() 메소드를 처리하는 프로세서에 다음 액션을 수행하는 결정을 남겨두고 모든 IAction은 로직을 수행하도록 남겨두고 싶다. 그들의 프로세스. – pghtech

1

나는 콘라드 Frix에 의해 게시 솔루션을 좋아하지만, 나뿐만 아니라 다른 옵션을 제안합니다 :

나는 비슷한 짓을했는지, 그리고 내 경우에는 내가 대신있는의 책임 패턴의 체인을 사용 행동의 수집.

public abstract SystemAction 
{ 
    public SystemAction nextAction { get; set; } 

    public void Execute() 
    { 
     this.ExecuteAction(); 
     if (this.nextAction != null) 
      this.nextAction.Execute(); 
    } 

    protected abstract void ExecuteAction(); 

    public SystemAction ThenDo(SystemAction action) 
    { 
     this.nextAction = action; 
     return this.nextAction; 
    }    
} 

그래서 당신의 작업을 수행 할 다음 작업으로 연결 수 :

따라서, 귀하의 경우에는 내가처럼 보이는 추상 기본 클래스로하여 IAction을 변경합니다. 클라이언트 코드처럼 보이는 끝 :

SystemAction setupSite = new CreatePhyiscalDir(); 
setupsSite 
    .ThenDo(new CreateWebsite()) 
    .ThenDo(new CreateVirDirectories()); 

setupSite.Execute(); 

나는이 콘래드의 솔루션보다 더 말할 것이지만, 또 다른 옵션이다. 필자는 그것을 내 시스템에서 효과적으로 사용하도록했습니다 (큐 메커니즘을위한 필터 체인 구축).

+0

+1 - 유창한 표현을 사용하여 연결하는 것이 좋음 ... – decyclone

관련 문제