检索在调用堆栈上活动方法之前的方法的MethodInfo



我已经为日志库创建了包装器,我想在其中记录抛出异常的外部方法的参数:

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)之前进行索引检查。

这些都不是。我会用温莎城堡之类的拦截方案。这样就很容易获得被截获的方法的参数。

相关内容

  • 没有找到相关文章

最新更新