I saw this code over on Ayende’s website. I see at least 4 things wrong with this code, which was found here.
public object DeepCopy (object value)
{
try {
return value;
} catch (Exception ex) {
throw ex;
}
}
See if you can find them all.
.NET, Bad Code
1. It doesn’t actually copy anything (except for the reference).
2. It catches System.Exception, which is generally pointless.
3. It doesn’t actually handle the exception. If you aren’t going to handle it why catch it?
4. It resets the call stack.
Goes to show you how much credibility CodeProject has.