search

sgrassie / sgrassie.github.io

temporalcohesion.co.uk - my blog
0 stars 1 forks source link

Damn you empty catch block #109

Open sgrassie opened 4 years ago

sgrassie commented 4 years ago

Comment thread for the post Damn you empty catch block

sgrassie commented 4 years ago

Comment by Calum Leslie on Tuesday, 9 November 2010 16:04 (imported from Disqus):

At a previous workplace of mine I found that a wrapper, which had only a "try" and a "finally" (it was designed to clean up after exceptions), was eating exceptions. I investigated to find that between the lovingly-crafted "try" and "finally" blocks, someone had inserted the following:

catch( Exception e ) {
  int i = 0;
}


An svn blame later and I was over at the culprits desk. Apparently the code was there so that he "could debug it"; he just needed a line of code where an exception would be caught that he could put a breakpoint on.

sgrassie commented 4 years ago

Comment by Bob on Thursday, 13 January 2011 00:53 (imported from Disqus):

COM interop makes you do it. There are some horrendous COM APIs out there and those that dont return S_OK throw an excaption in .net, which is horrible. I recently had to use the Task Scheduler COM API and it had an unqueryable dictionary - the only way to test was to Get and catch the exception which simply was the HRESULT saying key not found.

Sometimes an empty catch block is better than a shitty COM API bombing out your app when the HRSEULT is actually perfectly valid......

sgrassie commented 4 years ago

Comment by stuart on Thursday, 13 January 2011 01:28 (imported from Disqus):

COM Interop is a dirty, and necessary evil, and I've had to do my fair share of it. Exception handling can be a pain, admittedly. A lot of the time though, I'd still rather add a log or trace message rather than just leave it empty.

This blog post was more aimed toward the more general case of someone swallowing an exception either because they were lazy or didn't see the benefit of logging the error or some such.