Using GetCurrentMethod in (supposedly) high-performance code

Dan Tao picture Dan Tao · Sep 23, 2009 · Viewed 8.9k times · Source

For logging purposes, some methods in our application include the following line:

Dim Log As ILog = GetLog(Reflection.MethodBase.GetCurrentMethod().DeclaringType)

I have what might be described as an irrational fear of reflection, which I try to keep in check. However, calls like this in methods that are executed potentially a hundred times a second concern me. I don't know as much as I should about reflection; but from looking briefly over the documentation, it looks to me like I could replace the following with:

Dim Log As ILog = GetLog(Me.GetType())

My question is three-fold:

  1. Does Me.GetType() actually return the same Type as GetCurrentMethod().DeclaringType?
  2. Does Me.GetType() actually do anything differently from GetCurrentMethod().DeclaringType, or is it doing the same thing under the hood?
  3. Should I not even be worried about this at all? Performance is critical in this application; the program runs fine, but the nature of our business is such that if we can shave off even a few microseconds here and there, that is useful.

Answer

JaredPar picture JaredPar · Sep 23, 2009

Does Me.GetType() return the as GetCurrentMethod().DeclaringType?

It depends. Me.GetType will always return the actual type of an object. GetCurrentMethod().DeclaringType will return the type in which the method was declared. These values can be different in inheritance scenarios.

Consider the following

Class C1
  Public Sub Foo() 
    ..
  End Sub
End Class
Class C2 
  Inherits C1
  ..
End Class

Inside method Foo the two expressions would be equal if you were dealing with an instance of C1. But if it was C2 they would be different.

Does Me.GetType() do anything differently from GetCurrentMethod().DeclaringType

Yes these are very different functions. Me.GetType determines the runtime type of the current instance of the class. GetCurrentMethod.DeclaringType determines in what type was this method declared.

Should I not even be worried about this at all?

If this is a performance critical scenario then yes you make sure you profile APIs you do not understand. Especially those that appear to involve reflection. But only a profiler will tell you which is definitively faster. My money is on Me.GetType though.