From: Rick DeNatale Date: 2009-02-17T22:12:31+09:00 Subject: Re: rescue and principle of least surprise --0016364583788f1d2204631d0e4c Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit On Mon, Feb 16, 2009 at 11:21 PM, matt neuburg wrote: > 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. -- Rick DeNatale Blog: http://talklikeaduck.denhaven2.com/ Twitter: http://twitter.com/RickDeNatale --0016364583788f1d2204631d0e4c--