Yesterday’s episode of The Daily WTF had a great code snippet. Prepare to laugh.


private int GetMessage( int selected )
{
  int index = 0;
  // Get the Message
  switch( selected )
  {
    case 0:
      index = 0;
      break;
    case 1:
      index = 1;
      break;
    case 2:
      index = 2;
      break;
    case 3:
      index = 3;
      break;
  }
  return index;
}
 

gcc really and truly needs a compiler warning flag for utter stupidity. How about what someone in the comments proposed?


private int GetMessage( int selected )
{
  return selected;
}
 

or better still, just use whatever you were going to pass to GetMessage() in the first place… Yikes.