Art of Code Reviewing

This is a true story and happened on one of my projects (which shall remain nameless). Below is the feedback from one of the client as they were doing our code reviews.

Line 66: there seem to be a few too many words starting with "f" in the salutations comment. [This is really picky – the commenting standard in this file has otherwise been really good so far.]

And incase you were wondering what was the code in question, here it is:

/// <param name="salutations">The selection of Titles for which to choose from for a Contact</param>

So, what is the problem you ask? Who said there was a problem. Geeked

Published by

Amit Bahree

This blog is my personal blog and while it does reflect my experiences in my professional life, this is just my thoughts. Most of the entries are technical though sometimes they can vary from the wacky to even political – however that is quite rare. Quite often, I have been asked what’s up with the “gibberish” and the funny title of the blog? Some people even going the extra step to say that, this is a virus that infected their system (ahem) well. [:D] It actually is quite simple, and if you have still not figured out then check out this link – whats in a name?

Leave a Reply

This site uses Akismet to reduce spam. Learn how your comment data is processed.