Atkelar (atkelar) wrote,

WTF coding #6 - "open source"...

This post is not my typical rant about the concept of open source but rather a note about how funny it can be to read things that other people write... I don't dare say "code" because of the level of stupidity involved sometimes.

Two fragments of C# code I came across yesterday are currently on the top of my list. Here's one (I changed the names of the variables to protect the guilty party):

if (xxx != null
     && (false
         || (true
             && string.Equals(xxx.yyy, 

I suspect I know how this came about... there may have been other conditions that got removed and instead of fixing the nested parentheses, they replaced them with constants. Still, that's how you make your code hard to read, hard to maintain and hard to understand. And hard doesn't equal "solid" ;)

And how about this conditional branch a few lines onward?

if (xxx == null)
    return yyy;
return yyy;

I had a friend in my school wayyyy back when DOS and Turbo Pascal was the thing who did something like that to "make sure it gets done the right way"... as if the computer would decide to randomly skip a statement just because it is not written in double.

Bottom line: if changes to your code render parts of it useless - especially when they are THAT obvious - remove them! It will save you headaches in the long run!
Tags: coding, software development, wtf

  • Christmas video!

    OK - I planned so many things ahead this year but Christmas time is stressful in it self and the most busy time in my business... So I planned to…

  • Christmas stress 2010

    This year, christmas stress is even worse than last year... my schedule is so full until christmas, I have to juggle by the hour almost. That…

  • Merry Christmas!

    And just barely in time, I finished my Christmas video for you - it's not very long but I hope it's funny enough:…

  • Post a new comment


    Anonymous comments are disabled in this journal

    default userpic

    Your reply will be screened

    Your IP address will be recorded