There's a bit more subtlety to rethrowing exceptions than most developers realize. Although this topic is covered very nicely at The .NET Guy blog, here's another example:
Try session = smgr.getSession(_strDocbaseName) Catch ex As Exception If ex.Message.IndexOf("authentication failed") > 0 Then Throw New Exception("more info about the exception", ex) Else Throw End If End Try
The important thing here is to preserve the call stack, and that means
- when throwing your more-informative exception, include the original exception as the InnerException (second parameter) for reference
- when you decide you can't handle the exception, re-throw the original exception as is.
... would do the same thing, but less code is almost always better, IMO.
So then the next natural question that most developers ask is, "When should I catch exceptions"? And it's a very good question. Here are some guidelines that I have found useful.
- Unless you have a very good reason to catch an exception, DON'T. Exceptions are supposed to be exceptional, just like the dictionary meaning: uncommon, unusual. When in doubt, let the calling routine, or the global exception handler, deal with it. This is the golden rule. The hardest kinds of exceptions to troubleshoot are the ones that don't even exist, because a developer upstream of you decided to consume it.
- If you can correct the problem implied by the exception. For example, if you try to write to a file and it is read-only, try removing the read-only flag from the file. In this case you handled the exception and fixed the problem, so you should eat the exception. It doesn't exist, because you fixed it.
- If you can provide additional information about the exception. For example, if you fail to connect via HTTP to a remote website, you can provide details about why the connection failed: was the DNS invalid? Did it time out? Was the connection closed? Did the site return 401 unauthorized, which implies that credentials are needed? In this case you want to catch the exception, and re-throw it as an inner exception with more information. This is a very good reason to catch an exception, but note that we are still re-throwing it!
- Always try to catch specific exceptions. Avoid catching
System.Exceptionwhenever possible; try to catch just the specific errors that are specific to that block of code. Catch