|
Srinivas Kalabarigi wrote: You mean to say its not good practice to catch all....?
It's a known anti-pattern; catching all is the worst practice, a remnant of the "On Error Resume Next" lazyness.
Catching an unexpected exception might lead to unexpected behaviour (since your app is in an unknown state), data-loss, and generally makes it harder to solve bugs.
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
Eddy Vluggen wrote: Catching an unexpected exception might lead to unexpected behaviour... (
Except of course that in servers
1. Not catching it means it is lost.
2. Most of the time, most errors are non-disruptive. For example null pointer errors, and in a correctly designed layered system such errors, although unexpected, will not stop the application.
|
|
|
|
|
It is a good practice to catch exceptions; but if someone is sure about some exception then it is better to work on your code so that it doesn't throw any exception; let's say if divide by zero throws an exception then you shouldn't allow your code to reach to that point so that division by zero may occur. In that case it is not recommended to use the try-catch block but it is recommended to not let your code reach to that point so that that exception may occur.
|
|
|
|
|
I dont pay much attention to practices when it comes to exceptions.. I guess it depends on what you are coding...
However, I have coded both pro app and pro web sites..
And I must say....
The best use of catch exception code for me has always been...
If Website, catch just about all exceptions and store them into a database for review and easy lookup for when users report bugs and problems that may be very hard to reproduce on your own..
In applications, catch exception and present user with interface to report exception to staff..
That has been the extend of my exception code...
=)
|
|
|
|
|
Forbiddenx wrote: catch just about all exceptions
Not using a try-catch construction; there's an event that's raised for unhandled exceptions; that's the place where one would log them.
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
Yes, I know what your talking about..
But I typical do this.. where i want to log data because it gives me more control.
And sometimes depending on what the exception is and where it happened, i don't log it...
So, if i did it in the exception event.. it would have less control.
But I do see what your saying.. and its also a method that works.
Try
{
some code
}
catch (Exception exc)
{
log it, or present it to the user exc...
}
=)
|
|
|
|
|
Forbiddenx wrote: where i want to log data because it gives me more control.
And sometimes depending on what the exception is and where it happened, i don't log it...
How do you have "more" control?
I do hope that you catch specific exceptions, and aren't advocating a pokemon-handler? Swallowing exceptions is a worst-practice:
catch (Exception exc)
{
log it, or present it to the user exc...
}
And how is it preferable to have your code littered with repeating code, consisting of merely a try-and-log? Simply log from the one place where all unhandled exceptions end up, and be done with it.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
I think we are misunderstanding each other.
I do use specific exceptions but at times I also use the generic all exceptions..
But, I also bomb out if I use the generic all exception and stop everything in its tracks.
catch (Exception exc)
{
log it, or present it to the user exc...
BOMB OUT.... alert.. done.
I never resume operation that would be bad!
}
I only resume operation if I
catch (ExceptionTYPE exc)
{
try to recover..
}
Yes, it does add some extra code, but it is better then resuming you want to log any exception that occurs globally.
=)
|
|
|
|
|
Forbiddenx wrote: But, I also bomb out if I use the generic all exception and stop everything in its tracks.
There's the problem; handling a "specific" exception means that you cannot use the generic exception to handle it - it'd be passing in other exceptions that weren't meant to be handled that way. Nice example is a Sql Server that's offline, and a client-app that keeps whining about a password being incorrect (since that was all that's handled locally).
Hence, the best practice would be to catch and handle what you expect, and to log everything else on a higher level.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Eddy Vluggen wrote: that's the place where one would log them.
But only at the top-most layer.
|
|
|
|
|
Forbiddenx wrote: If Website, catch just about all exceptions and store them into a database fo
And what happens when the database throws an exception?
|
|
|
|
|
That hardly ever happens. 
|
|
|
|
|
I am going to compound a little on Eddy's
Don't catch what you can't handle
Don't throw what you can catch
Expected exceptions are not exceptions test first, your code will be more rigorous.
|
|
|
|
|
Ennis Ray Lynch, Jr. wrote: Don't throw what you can catch
Unless you are a value-added re-thrower.
|
|
|
|
|
I catch everything on layer boundaries. What happens there depends on the layer and where I catch it.
I catch everything on adapter boundaries, although that is just a specialization of a layer boundary. However I also tend to catch everything at the adapter top level as well.
When I catch everything then handling involves the following
1. log it, and take appropriate action which might include throwing a different exception WITHOUT including the original exception.
2. Don't log it and take appropriate action which might include throwing a different exception INCLUDING the original exception in the new exception.
I do NOT use catch alls which do nothing but log and then rethrow. That is pointless and a waste of time since the caller should be catching and will then end up logging as well.
There are of course extreme conditions where the above can be problematic. For example out of memory errors. Those can cause the log to fail as well. However without some logging figuring out an out of memory occurred is impossible. There are things like stack over flow in C# which CANNOT be caught so one is kind of out luck there.
This however does NOT mean that one can ignore the code that one is working with. If I am working on a communications layer then I expect communication exceptions. And I expect to take specific actions with regards to those.
I suppose what it really comes down to is that I use catch all because I am almost certain that some unexpected exception will occur at some point and I need to still behave in some reasonable manner. At other times I am not certain and thus it is inappropriate to take such action. Which is why boundary layers are often the only place this occurs.
|
|
|
|
|
Srinivas Kalabarigi wrote: i am sure it throws an exception
That's silly; how do you do that?
Srinivas Kalabarigi wrote: use it in most places of your code
No, only where it makes sense.
Srinivas Kalabarigi wrote: what is the best practice
That depends on your situation; different domains require different techniques.
I work mostly in the back-end with libraries and databases, which is very different from the front-end.
Most times I have a catch, it's a catch-all; I rarely need to have special handling for different Exceptions.
The main thing I usually need to do is to add information to the Data collection of the Exception and rethrow it.
Many times, I also need to rollback a transaction or perform some other clean up task.
Also, most times that a problem happens in a database all I get is a System.Data.DataException and I have to investigate it to see what the problem was -- deadlock, timeout, duplicate key, etc. (and no two database systems report it the same way) -- then I can wrap the DataException in a more detailed custom Exception -- DeadlockException, TimeoutException, DuplicateException, etc. and throw it so the caller can decide how to procede.
The main thing here is to add information to make the caller's job easier; but the ultimate decision of what to do is up to the caller.
|
|
|
|
|
Use try-catch unless you want your clients complaining "i got this error and then the entire application shut down".
|
|
|
|
|
I'd rather have them complain once and explain that it's a bug that will never resurface, than having a lot of exceptions (and invalid states) that I don't know of.
I love the PHP-approach; either do or die, don't fake it.
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
I'm with Abhinav - use try catch, but report a problem to the user.
Nothing, but nothing, is so damn annoying as losing an hour's work because a programmer didn't bother to allow for an alpha character in a numeric field!
Have you never sworn because an app went "...caused an exception and needs to close" and you lose a bunch of work?
This message is manufactured from fully recyclable noughts and ones. To recycle this message, please separate into two tidy piles, and take them to your nearest local recycling centre.
Please note that in some areas noughts are always replaced with zeros by law, and many facilities cannot recycle zeroes - in this case, please bury them in your back garden and water frequently.
|
|
|
|
|
OriginalGriff wrote: Nothing, but nothing, is so damn annoying as losing an hour's work because a programmer didn't bother to allow for an alpha character in a numeric field!
Yes, it's loosing a day because that exception is swallowed somewhere deeply within the system.
OriginalGriff wrote: Have you never sworn because an app went "...caused an exception and needs to close" and you lose a bunch of work?
There's an auto-save, but my Ctrl-S reflex is still alive.
And yes, I'd rather see my app die, than pretend nothing happened (and have it continue with corrupted data that gets saved later)
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
I'm not saying "swallow it" and "pretend nothing happened".
I'm saying use the tools and locate it while you (as a programmer) can still prevent it adversely affecting the user. Then report it, log it (both is good) and give them a chance to do something. Just letting your program crash is just being lazy! 
This message is manufactured from fully recyclable noughts and ones. To recycle this message, please separate into two tidy piles, and take them to your nearest local recycling centre.
Please note that in some areas noughts are always replaced with zeros by law, and many facilities cannot recycle zeroes - in this case, please bury them in your back garden and water frequently.
|
|
|
|
|
OriginalGriff wrote: Just letting your program crash is just being lazy!
Aight.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Eddy Vluggen wrote: than having a lot of exceptions (and invalid states) that I don't know of.
Been writing servers for 15 years, high capacity ones for 10, and I have never seen that.
What I have seen are a lot of null pointer exceptions caused by programming adaptors in plugin architectures doing dynamic processing. Which means that the ALL transactions in flight would then fail simply because a one time data set caused a bug to show up.
And I can note that in most cases the adapter code was not written by me but the plugin architecture was.
I have seen catastrophic failures but they take the server down. And in that case catching the exceptions at least for the purpose of logging is critical.
Eddy Vluggen wrote: I'd rather have them complain once and explain that it's a bug
I rather not have to explain anything when all the customers suffered some sort of failure due to something that could have been avoided.
|
|
|
|
|
Down vote countered - not checking or catching errors is just lazy programming, and is unfair on the people who pay our wages!
This message is manufactured from fully recyclable noughts and ones. To recycle this message, please separate into two tidy piles, and take them to your nearest local recycling centre.
Please note that in some areas noughts are always replaced with zeros by law, and many facilities cannot recycle zeroes - in this case, please bury them in your back garden and water frequently.
|
|
|
|
|
OriginalGriff wrote: Down vote countered
Thanks.
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
