Does the C# compiler optimize foreach blocks when its contents have the Conditional attribute?

Question

I'm writing some debug code at work, and I'm wondering if what I'm doing might hurt performance or not.

Let's get to the code:

foreach (var item in aCollection)
    Debug.WriteLine(item.Name);

I know that the Debug class uses the Conditional attribute to avoid compilation in release mode (or whenever DEBUG is undefined), but would this end up as a futile/empty iteration when compiled in release mode or would it be optimized by the compiler?

Answer 1

You are not leveraging any compiler symbols here. Wrap it inside these:

#if DEBUG

    // your code here

#endif

Advantages of this approach:

• readability is greatly decreased by the [Conditional] method attribute. It is not obvious on the invoking side that an invocation will not take place in the compiled code. Teams should refrain from that practice in favor of more explicit conditional compilation methods. Even commenting is not advisable because there's always that someone in large teams who forgets to comment stuff like this. The example above, instead, is easy to read (VS2010+ even shades the text when it is not part of the current build profile).
• constructing collections might be very expensive (e.G. constructing 1000 items with data from a database). In those cases compiler symbols allow for easier and cleaner culling of the interested code, without leaving an empty loop or the construction of the collection.

Answer 2

That foreach will not be wiped away.

The compiled code will be something like:

foreach (var item in aCollection)
{
   ;
}

the collection will be enumerated anyway.