Welcome to the Lounge

Luiz Felipe Stangarlin wrote:

I want to kill you if you don't delete old code.

mark merrens wrote:

People that tell you their code is 'self-commenting'.

// check if user is valid
if(IsUserValid(user))
{
   // update the user
   UpdateUser(user);
}
else
{
    // show a messagebox with an error
    MessageBox(error);
}

n+1. comments that state the obvious

Nish Sivakumar wrote:

Sometimes, it is though.
 

// check if user is valid
if(IsUserValid(user))
{
   // update the user
   UpdateUser(user);
}
else
{
    // show a messagebox with an error
    MessageBox(error);
}

// better validate user before we get too far into the process...

Roger Wright wrote:

I don't know if that last one is common anymore, but it used to drive me nuts, and I found it in a lot of code.

const int FiveHundred= 500;