다형성을 통한 예외 처리

Question

이 디자인에서는 예외 처리를 제안하고 있습니다.

1. 던져 새로운 오류 예외 < BusinessRuleViolationFault> (...)
2. 던져 새로운 오류 예외 < SomeOtherViolationFault> (...)
:

나는 다른 오류 예외의 유형을 던졌습니다 WCF 서비스를 excep

3. ...
4. ...

이 모든 결함 클래스는 IViolationFault라는 인터페이스를 구현합니다.이 인터페이스에는 단일 HandleException 메소드가 있습니다. 각 메소드의 구현은 클래스에 따라 다릅니다. 예를 들어 이러한 예외 클래스가 클라이언트에서 발견되면 HandleException() 메서드를 호출하면 오류 유형을 구분하는 IF 조건을 작성하지 않고도 처리 할 수 ​​있습니다. BusinessRuleViolationFault의 HandleException() 메소드에 난 그냥 다른 하나에 내가 어딘가를 기록뿐만 아니라 다른 액션을 트리거 할 수 있습니다 동안 ...,

catch (FaultException<BusinessRuleViolationFault> ex) 
{ 
ex.HandleException(); 
} 
catch (FaultException<SomeOtherViolationFault> ex) 
{ 
ex.HandleException(); 
} 

질문

을 화면에 메시지를 표시 할 수 있습니다

1. 접근 방법에 문제가 있습니까?
2. catch 블록 수를 줄일 수있는 방법이 있습니까? 하나의 catch 블록이 모든 예외 유형을 처리합니까?
3. 예외를 처리하는 동안 이것이 다형성을 유지하는 데 좋은 객체 지향적 인 방법입니까?

EDIT

필자는베이스 클래스로부터 상속 대신 인터페이스를 구현하는 코드를 변경. 내 BusinessRuleViolationFault 클래스에는 HandleException 메서드가 있지만 클라이언트 catch 블록에서 HandleException 메서드를 가져올 수 없습니다. 무엇이 잘못 되었습니까? 이는이 날이에 대한 당신의 생각을하자 DO 내 BusinessRuleViolationFault 코드

[DataContract] 
public class BusinessRuleViolationFault : BaseFault 
{ 
    public BusinessRuleViolationFault(string message) 
     : base(message) 
    { 

    } 


    [OperationContract] 
    public override string HandleException() 
    { 
     return "BusinessRuleViolationFault executed"; 
    } 
} 



[DataContract] 
public abstract class BaseFault 
{ 
    public BaseFault(string message) 
    { 
     Message = message; 
    } 

    [DataMember] 
    public string Message { get; set; } 

    [OperationContract] 
    public abstract string HandleException(); 
} 

인 서비스

BusinessRuleViolationFault bf = new BusinessRuleViolationFault(""); 
throw new FaultException<BusinessRuleViolationFault>(bf, new FaultReason(new FaultReasonText("Fault reason here"))); 

에 슬로우됩니다 방법이다. 시간 내 주셔서 감사합니다 ...

Answer 1

편집 : 올바른 샘플 코드 예를 들어.

적어도 WCF에서 사용자 지정 오류를 구현하는 방법은 어디에도 없습니다. 비록 당신이 다형성을 잃어 버렸지 만, 이런 식으로는 할 수 없듯이, FaultExceptions를 잡기 위해서는

// Does not work 
try 
{ 
} 
catch (FaultException<IVoliationFault> ex) 
{ 
    ex.HandleException(); 
} 

을 잡아라.

기본 클래스 System.ServiceModel.FaultException<TDetail> 또는 심지어 System.ServiceModel.FaultException을 HandleException() 메서드를 포함하도록 변경할 수 없습니다 (분명히).

당신이 할 수있는 일은 어떤 경우 오류 예외의 TDetail를 추출하는 반사와 함께 작업 한 다음 해당 작업을한다 :

 try 
     { 
     } 
     catch (FaultException ex) 
     { 
      var detail = ex.GetDetail<IViolationFault>(); // EDIT: Specify type 

      if (detail != null) 
      { 
       detail.HandleException(); 
      } 
      else 
      { 
       // Either not a FaultException<TDetail>, or not FaultException<IViolationFault>. 
       throw; 
      } 
     } 
... 

public static class FaultExceptionExtensions 
{ 
    public static T GetDetail<T>(this FaultException exception) 
    { 
     if (exception == null) 
      throw new ArgumentNullException("exception"); 

     Type type = exception.GetType(); // EDIT: use "exception" instead of "ex" 
     if (!type.IsGenericType) 
     { 
      return default(T); 
     } 

     Type genType = type.GetGenericArguments()[0]; 

     PropertyInfo pi = type.GetProperty("Detail", genType); 
     Debug.Assert(pi != null, "FaultException<" + genType + ">.Detail property is missing"); 

     object val = pi.GetValue(exception, null); 

     if (!typeof(T).IsInstanceOfType(val)) 
     { 
      return default(T); 
     } 

     return (T)val; 
    } 

} 

Answer 2

이 접근법에는 아무런 문제가 없습니다. 단지 다형성을 정상적으로 사용하는 것입니다.

모든 사용자 정의 예외 클래스가 인터페이스를 구현하는 대신 기본 예외 클래스에서 파생되도록함으로써 catch 블록 수를 줄일 수 있습니다.그냥 다음에 모든 예외를 잡을 수

public abstract class FaultException : Exception 
{ 
    public abstract void HandleException() 
} 

public class Faultexception<T> : FaultException 
{ 
    public override void HandleException() 
    { 
     //your code here 
    } 
} 

이 방법 :

catch (FaultException ex) 
{ 
    ex.HandleException(); 
} 

Answer 3

당신은 문서화 된 기능적 접근을 살펴 할 수 있습니다 this blog post에서 try/catch 논리를 좀 더 명확하게/간결하게 만들 수 있습니다. 예 :

TryCatch.Try(() => 
     { 
      using (var stream = File.OpenRead(FileTextBox.Text)) 
      { 
       Trace.WriteLine(stream.Length); 
      } 
     }) 
.Catch<FileNotFoundException, 
    DirectoryNotFoundException, 
    UnauthorizedAccessException> 
    (ex => 
    { 
     MessageBox.Show(ex.Message, "Fail"); 
    });