Правильный способ найти самое сокровенное исключение?

Question

Я работаю с некоторыми классами, которые при броске имеют относительно глубокое дерево InnerException. Я хотел бы войти и действовать на основе самого внутреннего исключения, которое имеет реальную причину проблемы.

Я сейчас использую что-то похожее на

public static Exception getInnermostException(Exception e) {
    while (e.InnerException != null) {
        e = e.InnerException;
    }
    return e;
}

Это правильный способ обработки деревьев исключений?

Answer 1

Я думаю, что вы можете получить самое внутреннее исключение, используя следующий код:

public static Exception getInnermostException(Exception e) { 
    return e.GetBaseException(); 
}

Answer 2

Вы можете использовать метод GetBaseException .Очень быстрый пример:

try
{
    try
    {
        throw new ArgumentException("Innermost exception");
    }
    catch (Exception ex)
    {
        throw new Exception("Wrapper 1",ex);
    }
}
catch (Exception ex)
{
    // Writes out the ArgumentException details
    Console.WriteLine(ex.GetBaseException().ToString());
}

Answer 3

Существуют исключения, которые могут иметь несколько основных причин (например, AggregateException и ReflectionTypeLoadException).

Я создал свой собственный класс для навигации по дереву, а затем разные посетители либо собирали все, либо только коренные причины. Пример выходных данных здесь . Соответствующий фрагмент кода ниже.

public void Accept(ExceptionVisitor visitor)
{
    Read(this.exception, visitor);
}

private static void Read(Exception ex, ExceptionVisitor visitor)
{
    bool isRoot = ex.InnerException == null;
    if (isRoot)
    {
        visitor.VisitRootCause(ex);
    }

    visitor.Visit(ex);
    visitor.Depth++;

    bool isAggregateException = TestComplexExceptionType<AggregateException>(ex, visitor, aggregateException => aggregateException.InnerExceptions);
    TestComplexExceptionType<ReflectionTypeLoadException>(ex, visitor, reflectionTypeLoadException => reflectionTypeLoadException.LoaderExceptions);

    // aggregate exceptions populate the first element from InnerExceptions, so no need to revisit
    if (!isRoot && !isAggregateException)
    {
        visitor.VisitInnerException(ex.InnerException);
        Read(ex.InnerException, visitor);
    }

    // set the depth back to current context
    visitor.Depth--;
}

private static bool TestComplexExceptionType<T>(Exception ex, ExceptionVisitor visitor, Func<T, IEnumerable<Exception>> siblingEnumerator) where T : Exception
{
    var complexException = ex as T;
    if (complexException == null)
    {
        return false;
    }

    visitor.VisitComplexException(ex);

    foreach (Exception sibling in siblingEnumerator.Invoke(complexException))
    {
        visitor.VisitSiblingInnerException(sibling);
        Read(sibling, visitor);
    }

    return true;
}

Answer 4

Одним словом, да. Я не могу придумать какой-либо значительно лучший или иной способ сделать это. Если только вы не хотите добавить его в качестве метода расширения, но на самом деле это шесть из одного, полдюжины из другого.