我已经为日志库创建了包装器,我想在其中记录抛出异常的外部方法的参数:
public void MethodA(string param1, int param2)
{
try
{
throw new Exception();
}
catch(Exception ex)
{
Logger.Error(ex);
}
}
...
public static void Error(Exception ex)
{
...
}
正如您所看到的,我希望获得Error方法级别的MethodA信息。我很乐意使用:
ParameterInfo[] pi = new StackFrame(1).GetMethod().GetParameters();
因为它有一切,但我只是读了很多关于性能差的帖子。在我看来,这可能不是最好的方法(另一方面,我们谈论的是异常处理——在这一点上,它已经不太重要了)。然而,我找不到使用反射的传统方法来解决这个问题。也许有人能开导我一下?
或者你仍然会使用StackFrame解决方案?这是一个asp.net应用程序,对超级性能的要求相当小。
Edit1:也许我不是很确定我想写什么。我正在使用StackFrame解决方案,即使它可能需要20毫秒才能获得一帧,但我真的很好奇如何通过反射获得类似的结果。
您可能会发现我的这个实用方法很有用,因为它还解决了您可能遇到的另一个问题:当您想从构造函数中获取当前堆栈帧时,情况有点奇怪。
using Collections = MikeNakis.Abstract.Collections;
using Diagnostics = System.Diagnostics;
using Generic = System.Collections.Generic;
using Reflection = System.Reflection;
[...]
///<summary>Collect stack frames</summary>
///<param name="frames_to_skip">Number of frames to skip (usually 1)</param>
///<param name="type">When invoking from within a constructor, specify the type of the containing class so as to
///skip any other constructors.</param>
public static Generic.IList<Diagnostics.StackFrame> CollectStackframes( int frames_to_skip, System.Type type = null )
{
var frames = new Diagnostics.StackTrace( frames_to_skip + 1, fNeedFileInfo:true ).GetFrames();
int i = 0;
if( type != null )
{
for( ; i < frames.Length; i++ )
{
Reflection.MethodBase method = frames[i].GetMethod();
if( !method.IsConstructor )
break;
//Does not work: if( method.DeclaringType == type )
//Does not work: if( Equals( method.DeclaringType, type ) )
//Does not work: if( Equals( method.DeclaringType.TypeHandle, type.TypeHandle ) )
if( Equals( method.DeclaringType.GUID, type.GUID ) )
{
i++;
break;
}
}
}
var list_of_frame = new Generic.List<Diagnostics.StackFrame>( frames.Length - i );
for( ; i < frames.Length; i++ )
list_of_frame.Add( frames[i] );
return Collections.Util.NewListReadonly( list_of_frame );
}
(注意:Collections.Util.NewListReadonly()
是我的一个静态实用方法,它创建一个只读列表并返回它的IList<T>
接口)
我不知道new Diagnostics.StackTrace()
的性能:我猜它和使用反射一样慢,(它可能算反射,)但可能比抛出异常快一点。
关于参数部分,我将让您失望:我没有找到在运行时获取方法参数内容的方法。因此,您所希望得到的只是参数的类型和名称,而不是它们的值。如果你有办法,请告诉我。
这个问题有几个不同的方面:
我想记录外部方法的参数
您可以使用基于拦截的技术构建在您最喜欢的IoC容器上,但老实说,我发现最好的方法通常只是将参数传递到异常信息中:
throw new Exception("Something went wrong..." + new{param1, param2});
在调用堆栈上的活动方法之前检索方法的MethodInfo ?
据我所知,您只打算在抛出异常时使用它来记录错误。在这种情况下,我暂时坚持使用基于stackframe的解决方案,因为这种情况不太可能经常发生。当c# 5发布时,您将能够使用[CallerMemberName]
属性:
public static void Error(Exception ex, [CallerMemberName] string caller = "") {
...
}
给聪明人的一句话
限制捕获此类异常的位置的数量是一种良好的实践。一般来说,最好是让异常自己传递到链上,直到可能的最后一刻,这时您可以优雅地告诉用户发生了意想不到的事情,并在此时记录错误。这为您提供了异常本身的更多信息的堆栈跟踪。如果你想在过程中的某个时刻传递额外的信息,只需将异常包装在内部异常中并抛出:
catch(Exception ex)
{
throw new Exception("Something went wrong..." + new{param1, param2}, ex);
}
或者你仍然会使用StackFrame解决方案?这是asp.net对超性能要求不高的应用
只要您只在错误条件下使用异常,而不是作为方法的"返回值",我将使用这种方法。(参见异常与返回值)。如果你不抛出大量异常——大多数应用程序都不会——那么你就会很好。
我们的团队编写交易软件,我们严格监控其性能。只要抛出的异常不发生在关键路径中,就没问题。通常,如果抛出一个未处理的异常,则性能无关紧要:用户不等待响应。或者响应因为异常而无法到达。
请注意,您可能希望在使用硬编码的新StackFrame(1)之前进行索引检查。
这些都不是。我会用温莎城堡之类的拦截方案。这样就很容易获得被截获的方法的参数。