8

我想要一个从异常中提取信息的中心位置,将我需要的所有信息设置为其消息参数,然后将该信息作为相同类型的异常重新抛出。

更好的解决方案可能是在最终处理异常(并记录其消息)的地方执行此操作,但是..我可以控制抛出异常的地方,而不是接收异常的地方,并且只有记录其消息内容。

除了该设计决策并且鉴于该消息是只读属性,我将(?)以某种方式创建一个新的异常对象,有没有办法使新的异常对象与原始对象的类型相同?

这是我的代码,它无法编译 - 它在抛出线(我尝试动态转换对象)上绊倒。

public static void RethrowExceptionWithFullDetailInMessage(string msg, Exception ex)
{
    Exception curEx = ex;
    int cnt = 0;
    while (curEx != null)
    {
        msg += "\r\n";
        msg += cnt++ + " ex.message: " + curEx.Message + "\r\n";
        msg += "Stack: " + curEx.StackTrace;
        curEx = curEx.InnerException;
    }
    object newEx = Convert.ChangeType(new Exception(msg), ex.GetType());
    throw (ex.GetType())newEx;
}

做这个

throw (Exception)newEx;

保留类型?(它编译。)

Convert.ChangeType 是否确保我得到正确类型的异常?

4

5 回答 5

12

您在这里尝试做的事情并不像看起来那么容易,并且有很多陷阱需要考虑。

请记住,Convert.ChangeType() 会将一种类型转换为另一种类型(假设存在这样的路径,例如将字符串转换为 int)。大多数例外都不会这样做(他们为什么会这样做?)

为了实现这一点,您必须在运行时使用 GetType() 方法检查异常类型,并找到具有您可以满足的要求的构造函数并调用它。在这里要小心,因为您无法控制所有异常的定义方式,因此无法保证您可以访问“标准”构造函数。

话虽这么说,如果你想成为一个打破规则的人,你可以做这样的事情......

void Main()
{
    try
    {   
        throw new Exception("Bar");
    }
    catch(Exception ex)
    {
        //I spit on the rules and change the message anyway
        ex.GetType().GetField("_message", BindingFlags.Instance | BindingFlags.NonPublic).SetValue(ex, "Foo");
        throw ex;
    }
}
于 2012-07-26T14:30:06.417 回答
4

您可以这样做以动态调用异常类型的构造函数:

object newEx = Activator.CreateInstance(ex.GetType(), new object[] { msg });

您的原始代码将在运行时失败,因为要工作Convert.ChangeType,异常类型必须实现IConvertible并支持转换为其他异常类型,我对此表示怀疑。

于 2012-07-26T14:27:26.010 回答
4

可能有点晚了,但这对你有用吗?

catch (Exception ex)
{
    throw new Exception("New message", ex);
}
于 2014-09-22T22:53:28.290 回答
0

您可以像这样通过反射更改异常消息...

Exception exception = new Exception("Some message.");
var type = typeof(Exception);
var flags = BindingFlags.Instance | BindingFlags.NonPublic;
var fieldInfo = type.GetField("_message", flags);
fieldInfo.SetValue(exception, message);

所以你可以创建一个扩展方法......

namespace ExceptionExample
{
    public static class ExceptionExtensions
    {
        public static void SetMessage(this Exception exception, string message)
        {
            if (exception == null)
                throw new ArgumentNullException(nameof(exception));

            var type = typeof(Exception);
            var flags = BindingFlags.Instance | BindingFlags.NonPublic;
            var fieldInfo = type.GetField("_message", flags);
            fieldInfo.SetValue(exception, message);
        }
    }
}

然后用它...

...
using static ExceptionExample.ExceptionExtensions;

public class SomeClass
{
    public void SomeMethod()
    {
        var reader = AnotherClass.GetReader();
        try
        {
            reader.Read();
        }
        catch (Exception ex)
        {
            var connection = reader?.Connection;
            ex.SetMessage($"The exception message was replaced.\n\nOriginal message: {ex.Message}\n\nDatabase: {connection?.Database}");
            throw; // you will not lose the stack trace
        }
    }
}

你必须记住,如果你使用“throw ex;” 堆栈跟踪将丢失

为避免这种情况,您必须使用“throw;” 无一例外

于 2019-06-05T11:35:54.057 回答
0

补充评论。

这些都可以补充异常消息,但我发现使用“throw”并没有保留 StackTrace - 最后一个跟踪指向实际的“throw”语句(删除根本原因位置)。

从其他地方的讨论中,很明显,由于 CLR 堆栈限制,某些情况下 throw 不会保留

抛出并保留堆栈跟踪不像代码分析所描述的那样

解决方案:在每个异常中转储 StackTrace(例如并添加到错误消息中)和/或转储到日志记录

于 2019-08-14T16:56:24.610 回答