Question

In replies to one of my questions, I received a number of answers saying that style 2 may perform better than style 1. I don’t understand how, since I believe they should emit essentially the same machine instructions (if written in C++). Could you please explain why style 2 might perform better?

I’ll rewrite the two styles here for easier reference:

Style 1:

while (!String.IsNullOrEmpty(msg = reader.readMsg())) {     RaiseMessageReceived();     if (parseMsg)     {         ParsedMsg parsedMsg = parser.parseMsg(msg);         RaiseMessageParsed();         if (processMsg)         {             process(parsedMsg);             RaiseMessageProcessed();         }     } } 

Style 2:

while (!String.IsNullOrEmpty(msg = reader.readMsg())) {     RaiseMessageReceived();     if (!parseMsg) continue;      ParsedMsg parsedMsg = parser.parseMsg(msg);     RaiseMessageParsed();     if (!processMsg) continue;      process(parsedMsg);     RaiseMessageProcessed(); } 

Answer 1

I had to check this.

Here is my version of the code:

using System; using System.Collections.Generic;  namespace ConsoleApplication2 {     class Program     {         static void Main(string[] args)         {             Tester t=new Tester();             t.Method1(new Stack<string>(), new MsgParser(), true, true);             t.Method2(new Stack<string>(), new MsgParser(), true, true);         }     }     class Tester     {         public void Method1(Stack<string> strings, MsgParser parser, bool parseMsg, bool processMsg)         {             string msg;             while (!String.IsNullOrEmpty(msg = strings.Pop()))             {                 RaiseMessageReceived();                 if (parseMsg)                 {                     ParsedMsg parsedMsg = parser.ParseMsg(msg);                     RaiseMessageParsed();                     if (processMsg)                     {                         process(parsedMsg);                         RaiseMessageProcessed();                     }                 }             }         }          public void Method2(Stack<string> strings, MsgParser parser, bool parseMsg, bool processMsg)         {             string msg;             while (!String.IsNullOrEmpty(msg = strings.Pop()))             {                 RaiseMessageReceived();                 if (!parseMsg) continue;                  ParsedMsg parsedMsg = parser.ParseMsg(msg);                 RaiseMessageParsed();                 if (!processMsg) continue;                  process(parsedMsg);                 RaiseMessageProcessed();             }          }          private void RaiseMessageProcessed()         {             Console.WriteLine('Done');         }          private void process(ParsedMsg msg)         {             Console.WriteLine(msg);         }          private void RaiseMessageParsed()         {             Console.WriteLine('Message parsed');         }          private void RaiseMessageReceived()         {             Console.WriteLine('Message received.');         }     }      internal class ParsedMsg     {     }      internal class MsgParser     {         public ParsedMsg ParseMsg(string msg)         {             return new ParsedMsg();         }     } } 

I built it with code optimization (default Release configuration), and disassembled the assembly using Reflector. The result verifies that the two styles are identical:

internal class Tester {     // Methods     public void Method1(Stack<string> strings, MsgParser parser, bool parseMsg, bool processMsg)     {         string msg;         while (!string.IsNullOrEmpty(msg = strings.Pop()))         {             this.RaiseMessageReceived();             if (parseMsg)             {                 ParsedMsg parsedMsg = parser.ParseMsg(msg);                 this.RaiseMessageParsed();                 if (processMsg)                 {                     this.process(parsedMsg);                     this.RaiseMessageProcessed();                 }             }         }     }      public void Method2(Stack<string> strings, MsgParser parser, bool parseMsg, bool processMsg)     {         string msg;         while (!string.IsNullOrEmpty(msg = strings.Pop()))         {             this.RaiseMessageReceived();             if (parseMsg)             {                 ParsedMsg parsedMsg = parser.ParseMsg(msg);                 this.RaiseMessageParsed();                 if (processMsg)                 {                     this.process(parsedMsg);                     this.RaiseMessageProcessed();                 }             }         }     }      private void process(ParsedMsg msg)     {         Console.WriteLine(msg);     }      private void RaiseMessageParsed()     {         Console.WriteLine('Message parsed');     }      private void RaiseMessageProcessed()     {         Console.WriteLine('Done');     }      private void RaiseMessageReceived()     {         Console.WriteLine('Message received.');     } }