이 코드를 작성하는 더 좋은 방법이 있습니까?

Question

값이 before, after 또는 none 인 클래스가 enum DoStuff입니다. 이 이름들은 실제 이름이 아니지만 그 의미를 알 수 있습니다. 클래스는 메소드 foo을 가지고 있습니다.

public [type] MyProperty { 
    get { 
     if(enumValue == DoStuff.Before) 
      foo(); 

     [type] result = //Do calculations here 

     if(enumValue == DoStuff.After) 
      foo(); 

     return result; 
    } 
} 

이러한 전/후 계산 부른다 추출하는 방법이 있나요 : 다음 무엇

다음과 같이 각각의 모양, 다른 유형의 읽기 전용 속성의 집합입니다? 나는 현재이 개 솔루션을 생각할 수 있습니다 :

• 가 대리자를 취하고, 적절한 장소에 foo를 호출하는 개인 방법을 만듭니다. 이 글을 쓰고있는 플랫폼에 제네릭이 없기 때문에 복잡합니다.

• 는 어느 래퍼 호출과 uninstantiable 기본 클래스를 확인하고 잘 알려진 패턴이있는 전에 적절한 장소

의 호출로, 기본 클래스의 속성에 액세스 서브 클래스 후 도출 이런 종류의 구조?

Answer 1

이 구문을 사용하는 방법 만듭니다

다음

delegate void Handler(); 

void DoHandler(Handler handler) 
{ 
     if(enumValue == DoStuff.Before) 
      foo(); 

     handler(); 

     if(enumValue == DoStuff.After) 
      foo(); 
} 

당신의 속성을

public [type] MyProperty 
{ 
    get 
    { 
     [type] result = default(type); 
     DoHandler(() => 
     { 
      int a = 5; 
      int b = 6; 
      result = a + b; 
     }); 
     return result; 
    } 
} 

Answer 2

나는 enumValue를 지금 설정하고있는 곳이면 어디서나 적절한 장소에 Before and After 이벤트/위임자를 만들고 slop foo()를 두려워 할 것입니다. 그런 다음 전화 :

get { 
    BeforeDelegate(); 
    // Do calcs 
    AfterDelegate(); 
} 

무엇이 실제적인 용도입니까? 왜이 패턴이 필요한가요?

사이드 노트 : 그런 종류의 논리를 사용하는 게터가 있다면 그 방법에 넣을 가능성이 큽니다. 사람들을 놀라게 할 가능성은 적습니다. 메서드는 스칼라 값을 노출하는 것 이상의 값을 얻기 위해 무언가를 아마 수행했을 것이라고 암시합니다. EnumValue는 변경이 적절한 이벤트를 해고 할 때

Answer 3

나는 enamvalue 클래스를 사용하는 클래스, 에서 이벤트에 EnamumValue 및 이벤트 전후라는 이름의 한 속성이 별도의 클래스에서 열거 값을 넣어이 문제를 처리하기 위해 더 나은 생각

public class EnumValueClass 
{ 

    public event BeforeDelegate OnBefore(); 
    public event AfterDelegate OnAfter(); 
    private EnumType enumValue; 
    public EnumType EnumValue 
    { 
     get { 
      return enumValue; 
     } 
     set{ 
      this.enumValue = value; 
      if(enumValue == DoStuff.Before) 
       if(OnBefore!=null) 
        OnBefore(); 

      if(enumValue == DoStuff.After) 
       if(OnAfter!=null) 
        OnAfter(); 
     } 


    } 
}