[Ryano121]

Just came across this little gem today. Not necessarily a nightmare, but pretty bad nonetheless. It should be noted that it has gone through several revisions where the try/catch was used, but whoever left it like this was having a bad day.

try
{
   // do something
}
catch(SomeException se)
{
    throw se;
}
catch(SomeOtherException soe)
{
    throw soe;
}
catch(Exception e)
{
    throw e;
}

[MentalFloss]

This one kind of depends on what is calling this code. Perhaps it was decided that your company wanted the calling object to handle the exception instead. Do you see this kind of action taken elsewhere? I would be wary about changing it before talking with your lead.