예외에 대한 추가 정보를 제공해야 할 때마다 실제로 어떤 방법이 올바른 방법 인지 궁금 합니다.

이 질문을 위해 예를 썼습니다. Abbreviation속성 을 업데이트하려는 클래스가 있다고 가정 해 봅시다 . SOLID 관점에서 보면 완벽하지는 않지만 일부 서비스를 통해 DI를 통해 작업자 메서드 를 전달하더라도 동일한 상황이 발생합니다. 예외가 발생하면 컨텍스트가 없습니다. 예제로 돌아 가기 ...

class Person
{
    public int Id { get; set; }
    public string Name { get; set; }
    public string Abbreviation { get; set; }
}

그런 다음 클래스의 인스턴스와 작업자 메서드가 호출되는 루프가 있습니다. 던질 수 있습니다 StringTooShortException.

var persons =
{
    new Person { Id = 1, Name = "Fo" },
    new Person { Id = 2, Name = "Barbaz" },
}

public IEnumerable<Person> GenerateAbbreviation(IEnumerable<Person> persons)
{
    foreach (var person in persons)
    {
        try
        {
            person.Abbreviation = GenerateAbbreviation(person.Name);
        }
        catch(Exception ex)
        {
            // ?
        }
    }
    // throw AggregateException...
}

public IEnumerable<string> GenerateAbbreviation(string value)
{
    if (value.Length < 5)
    {
        throw new StringTooShortException(value);
    }

    // generate abbreviation
}

질문은 : Person또는 그 Id(또는 다른 것) 를 추가하는 방법은 무엇입니까?

다음 세 가지 기술을 알고 있습니다.

1- Data속성 사용

장점 :

• 추가 정보를 쉽게 설정
• 더 많은 예외를 만들 필요가 없습니다
• 추가가 필요하지 않습니다 try/catch

단점 :

• 에 쉽게 통합 될 수 없습니다 Message
• 로거는이 필드를 무시하고 덤프하지 않습니다.
• 키와 캐스팅이 필요하기 때문에 값이 object
• 불변의

예:

public IEnumerable<Person> GenerateAbbreviation(IEnumerable<Person> persons)
{
    foreach (var person in persons)
    {
        try
        {
            person.Abbreviation = GenerateAbbreviation(person.Name);
        }
        catch(Exception ex)
        {
            ex.Data["PersonId"] = person.Id;
            // collect ex
        }
    }
    // throw AggregateException...
}

2-사용자 정의 특성 사용

장점 :

• Data속성 과 유사 하지만 강력한 유형
• 에 쉽게 통합 Message

단점 :

• 맞춤 예외가 필요합니다
• 로거는 그들을 무시합니다
• 불변의

예:

public IEnumerable<Person> GenerateAbbreviation(IEnumerable<Person> persons)
{
    foreach (var person in persons)
    {
        try
        {
            person.Abbreviation = GenerateAbbreviation(person.Name);
        }
        catch(Exception ex)
        {
            // not suitable for this exception because 
            // it doesn't have anything in common with the Person
        }
    }
    // throw AggregateException...
}

3-예외를 다른 예외로 랩

장점 :

• Message 예측 가능한 방식으로 형식화 가능
• 로거는 내부 예외를 덤프합니다
• 불변의

단점 :

• 추가가 필요합니다 try/catch
• 중첩 증가
• 집행의 깊이를 증가

예:

public IEnumerable<Person> GenerateAbbreviation(IEnumerable<Person> persons)
{
    foreach (var person in persons)
    {
        try
        {
            try
            {
                person.Abbreviation = GenerateAbbreviation(person.Name);
            }
            catch(Exception ex)
            {
                throw new InvalidPersonDataException(person.Id, ex);
            }
        }
        catch(Exception ex)
        {
            // collect ex
        }
    }
    // throw AggregateException...
}

• 다른 패턴이 있습니까?
• 더 나은 패턴이 있습니까?
• 일부 / 모두 모범 사례를 제안 할 수 있습니까?

Data FTW .

"대비":

• "메시지에 쉽게 통합 될 수 없습니다"

->의 경우 귀하의 예외 유형은 우선 쉬운 충분해야한다 Message그래서 그 않는 법인 (法人)을 Data(가) 만약 내가에만이 고려 것이지만 .. Data메시지입니다 .

• "로거는이 필드를 무시하고 덤프하지 않습니다"

일례로 인터넷 검색을 Nlog 수율 :

예외 레이아웃 렌더러

(...)

format- 출력 형식. 예외 특성 쉼표로 구분이어야 : Message, Type, ShortType, ToString, Method, StackTrace및 Data. 이 매개 변수 값은 대소 문자를 구분하지 않습니다. 기본:message

따라서 쉽게 구성 할 수있는 것 같습니다.

• 값이 객체이기 때문에 키와 캐스팅이 필요합니다

응? 거기에있는 객체를 덤프하고 사용 가능한 ToString()방법 이 있는지 확인하십시오 .

또한 키에 아무런 문제가 없습니다. 온화한 독창성을 사용하면 좋습니다.

면책 조항 : 이것은 질문에서 즉시 볼 수있는 것과 Data15 분 안에 봤습니다 . 나는 그것이 약간 도움이된다고 생각했기 때문에 대답으로 내놓았지만 결코 사용하지 않았 Data으므로 여기 질문자가 나보다 이것에 대해 더 많이 알고있을 것입니다.

왜 예외를 던지나요? 그들이 잡히고 처리하도록.

catch 코드 는 예외 처리 방법 을 어떻게 해결 합니까? Exception 객체에 정의한 속성 사용

Message 속성을 사용하여 예외를 식별하거나 잠재적 인 처리기가 의존해야하는 "정보"를 제공하지 마십시오. 단순히 너무 휘발성이고 신뢰할 수 없습니다.

전에는 "Data"속성을 사용한 적이 없지만 지나치게 일반적으로 들립니다.

생성하지 않는 한 많은 식별 각각의 예외의 클래스, 특정 예외적 인 경우를 어떻게 알 수 있습니까 당신은 예외 잡을 때 "데이터」를 나타냅니다 무엇을? ( "메시지"에 대한 이전 주석 참조).

나는 세 번째 예제를 좋아하지만, 대부분의 "con"을 제거하기 위해 코딩 할 수있는 또 다른 방법이 있습니다.

public IEnumerable<Person> GenerateAbbreviation(IEnumerable<Person> persons)
{
    var exceptions = new List<InvalidPersonDataException>();

    foreach (var person in persons)
    {
        try
        {
            person.Abbreviation = GenerateAbbreviation(person.Name);
        }
        catch(Exception ex)
        {
            exceptions.Add(new InvalidPersonDataException(person.Id, ex));
        }
    }

    if (exceptions.Any())
    {
        throw new AggregateException(exceptions);
    }
}