Comments by Trellium (Top 4 by date)

Using names that start with an underscore or not has nothing to do with "being cool", and is easily proved by the fact that neither method will help you get girls nor procreate (a typical outcome of great coolness).

There is no particular reason to use m_Name rather than _Name. There is no magic property to the prefix "m_", and replacing it with a simple underscore removes no readability nor understanding of the concept as long as it is used consistently.

I use the m_ prefix for private member variables. But I use only the underscore when it is a private member variable for which there is a property with the same name (minus the underscore).

Looking at any section of code, I can tell if a member variable is truly private, or if it is exposed as a property. If I see it start with m_, I would know there is no corresponding property without the m_. The reason for doing this is so I can see if the code should be bypassing the property access code or not. I ignore anything starting with m_ since that is the only way to access the variable (since there is no property).

That said, standards are there so everyone does the same thing. The scope of the project determines the standards to some extent. The more public the project the more important the adherance to common standards.

Reason for my vote of 3
Very rough code, needs better looping and recovery. Good generally though.

Reason for my vote of 3
See alternative 1