rescue and principle of least surprise

M

matt neuburg

It came as a big surprise to me when I discovered that this code:

begin
# do stuff
rescue
# do rescue stuff
end

....could fail to catch exceptions thrown in the "do stuff" section. A
bare "rescue" looks to me like it ought to mean: catch every exception.
Instead it turns out to mean: catch a certain subset of exceptions. It
doesn't catch LoadError, it doesn't catch SyntaxError, etc. etc.

Of course now I know better (and I commonly write "rescue Exception"),
but it still feels wrong, especially in view of the "principle of least
surprise". And I see by a quick Google search that people regard this as
an annoying "gotcha". Is it worth proposing an actual change in the
language - that the default rescue be Exception instead of
StandardError? m.
 
R

Rick DeNatale

[Note: parts of this message were removed to make it a legal post.]

It came as a big surprise to me when I discovered that this code:

begin
# do stuff
rescue
# do rescue stuff
end

...could fail to catch exceptions thrown in the "do stuff" section. A
bare "rescue" looks to me like it ought to mean: catch every exception.
Instead it turns out to mean: catch a certain subset of exceptions. It
doesn't catch LoadError, it doesn't catch SyntaxError, etc. etc.

Of course now I know better (and I commonly write "rescue Exception"),
but it still feels wrong, especially in view of the "principle of least
surprise". And I see by a quick Google search that people regard this as
an annoying "gotcha". Is it worth proposing an actual change in the
language - that the default rescue be Exception instead of
StandardError? m.

No, I don't think so. The distinction between StandardErrors and the others
is carefully thought out, and makes sense.

Exceptions which are not StandardErrors representation situations which are
normally not handled by a 'normal' Ruby program, either because it's usually
better to handle it the standard way (e.g. a SignalException because someone
issued a kill command, or a SystemExit exception), or difficult for the
program to recover from (e.g. ScriptErrors, and NoMemoryErrors).

The fact that you can explicitly rescue these exceptional exceptions means
that you don't HAVE to let them be handled in the standard way, but you
still have to carefully consider how that rescue clause needs to be written.

Also, if feel the need to rescue one or more of these low level Exceptions,
I it's usually wise to rescue them specifically rather than rescuing
Exception in general.
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

Forum statistics

Threads
473,744
Messages
2,569,484
Members
44,903
Latest member
orderPeak8CBDGummies

Latest Threads

Top