just look at this beauty:

m_fooDictionary[m_fooList[p_fooIndex].BarTypeEnum][oldState]--;

the interesting thing is that this line is repeated numerous times across a 1500+ lines class, sometimes with "++" or "--" at the end.. truly a great way to start off your day

got to love cpp...

:laugh:

haha, the last one made me laugh

you know what they say.. 'measure twice, cut once'

mission impossible...

:laugh: :laugh: :laugh: feel sorry for you..

guess hes really performance-oriented, not wanting to allocate memory for the eventArgs at actual runtime...

probably can cut the guy a little slack, at least he didn't locked on 'this' or 'typeof'..

my god, i really thought I wouldn't live to see the day someone actually have the nerve to use the goto statement in C#...

:laugh:

using try-catch blocks on a whole function may be ok in some situation (for example, if function is executed on a thread from the .Net Thread Pool). The reall coding horror here is that the method is more than 200 lines long (can't wait to debug that baby..)

Lovely. A simple proof that there's infinite ways to code even the most simple task.

I think it's pretty obvious. It's just that the good fellas down at Redmond decided to allow this so we could enjoy some hilarious coding horror posts

Code Reviewer: "wtf dude?!" Programmer: "ah... but it works!" indeed, if code written like THIS can actually work, than there must be some king of supreme being..

[Message Deleted]