Velocity Reviews

Velocity Reviews (http://www.velocityreviews.com/forums/index.php)
-   Ruby (http://www.velocityreviews.com/forums/f66-ruby.html)
-   -   Catching exceptions (http://www.velocityreviews.com/forums/t819620-catching-exceptions.html)

Mark Probert 02-09-2005 07:15 PM

Catching exceptions
 

Can someone please let me know what I am doing wrong here?
I can't seem to catch the exception ..


$ cat test.rb
require 'socket'

def alive(host, port=80)
s = "host=#{host} : "
t = TCPSocket.new(host, port)
begin
s << " made it!"
rescue Errno::ETIMEDOUT
s << " Timed out -- node unreachable"
rescue Exception => e
s << " exception = #{e}"
ensure
t.close
end
return s
end
puts alive('10.10.10.2')

$ ruby test.rb
test.rb:6:in `initialize': Operation timed out - connect(2) (Errno::ETIMEDOUT)
from test.rb:6:in `new'
from test.rb:6:in `alive'
from test.rb:19

Regards,

-mark. (probertm at acm dot org)



Kent Sibilev 02-09-2005 07:31 PM

Re: Catching exceptions
 
Mark Probert <probertm@acm.org> writes:

> Can someone please let me know what I am doing wrong here?
> I can't seem to catch the exception ..
>
>
> $ cat test.rb
> require 'socket'
>
> def alive(host, port=80)
> s = "host=#{host} : "
> t = TCPSocket.new(host, port)
> begin
> s << " made it!"
> rescue Errno::ETIMEDOUT
> s << " Timed out -- node unreachable"
> rescue Exception => e
> s << " exception = #{e}"
> ensure
> t.close
> end
> return s
> end
> puts alive('10.10.10.2')
>
> $ ruby test.rb
> test.rb:6:in `initialize': Operation timed out - connect(2) (Errno::ETIMEDOUT)
> from test.rb:6:in `new'
> from test.rb:6:in `alive'
> from test.rb:19
>
> Regards,
>
> -mark. (probertm at acm dot org)


Exception is being thrown from the outside of begin...end construct. Try this:

def alive(host, port=80)
s = "host=#{host} : "
t = TCPSocket.new(host, port)
s << " made it!"
return s
rescue Errno::ETIMEDOUT
s << " Timed out -- node unreachable"
rescue Exception => e
s << " exception = #{e}"
ensure
t.close
end

Cheers,
Kent



Gennady Bystritksy 02-09-2005 07:32 PM

Re: Catching exceptions
 
Have you tried to move your TCPSocket.new clause inside begin/rescue/end
block?

Mark Probert wrote:
> Can someone please let me know what I am doing wrong here?
> I can't seem to catch the exception ..
>
>
> $ cat test.rb
> require 'socket'
>
> def alive(host, port=80)
> s = "host=#{host} : "
> t = TCPSocket.new(host, port)
> begin
> s << " made it!"
> rescue Errno::ETIMEDOUT
> s << " Timed out -- node unreachable"
> rescue Exception => e
> s << " exception = #{e}"
> ensure
> t.close
> end
> return s
> end
> puts alive('10.10.10.2')
>
> $ ruby test.rb
> test.rb:6:in `initialize': Operation timed out - connect(2) (Errno::ETIMEDOUT)
> from test.rb:6:in `new'
> from test.rb:6:in `alive'
> from test.rb:19
>
> Regards,
>
> -mark. (probertm at acm dot org)
>





Mark Probert 02-09-2005 11:28 PM

Re: Catching exceptions
 
Hi ..

On Wednesday 09 February 2005 11:31, Kent Sibilev wrote:
> Mark Probert <probertm@acm.org> writes:
> > Can someone please let me know what I am doing wrong here?
> > I can't seem to catch the exception ..
> >

>
> Exception is being thrown from the outside of begin...end construct. Try
> this:
>

Many thanks. The t.close() in the ensure is out of scope, so that the final
result looks like:

require 'socket'
def alive(host, port=80)
s = "host=#{host} : "
t = TCPSocket.new(host, port)
s << " made it!"
t.close
return s

rescue Errno::ETIMEDOUT
s << " Timed out -- node unreachable"
rescue Exception => e
s << " exception = #{e}"
end
puts alive('10.10.10.2')

So, the next question I have is how does this actually work? Is there some
kind of implicit block around the code? If so, why does the explicit
begin .. end construct not work? How can I use an ensure for this?

Also, whilst I am at it ;-), I used the first formulation

t = TCPSocket.new()
begin
<stuff>
rescue <errors>
ensure
t.close
end

because t is not in scope if the creator is inside the begin .. end block. I
recall that this is the subject of much debate. How has it been resolved?

--
-mark. (probertm at acm dot org)



Mark Probert 02-09-2005 11:35 PM

Re: Catching exceptions
 
Hi ..

On Wednesday 09 February 2005 11:32, Gennady Bystritksy wrote:
> Have you tried to move your TCPSocket.new clause inside begin/rescue/end
> block?
>


Yes. The close() in the ensure is then out of scope. Moving it into the
begin .. block like

def alive(host, port=80)
s = "host=#{host} : "
begin
t = TCPSocket.new(host, port)
s << " made it!"
return s
t.close
rescue Errno::ETIMEDOUT
s << " Timed out -- node unreachable"
rescue Exception => e
s << " exception = #{e}"
end
end

Regards,

--
-mark. (probertm at acm dot org)



Gennady Bystritksy 02-10-2005 12:16 AM

Re: Catching exceptions
 
Mark Probert wrote:

> Yes. The close() in the ensure is then out of scope. Moving it into the
> begin .. block like
>
> def alive(host, port=80)
> s = "host=#{host} : "
> begin
> t = TCPSocket.new(host, port)
> s << " made it!"
> return s
> t.close

^^^^^^^
This will not be executed.

> rescue Errno::ETIMEDOUT
> s << " Timed out -- node unreachable"
> rescue Exception => e
> s << " exception = #{e}"
> end
> end
>


begin/rescue/ensure/end are all in the same (enclosing) scope. A
variable used in 'begin' block is available also in 'rescue' and
'ensure' blocks as well.

However, in your particular example, as an exception is raised in 'new',
't' will not reffer to any TCPSocket instance anyways, so doing t.close
does not make much sense (it will give you runtime error, as 't' will be
nil in this case).

Gennady.






All times are GMT. The time now is 03:59 PM.

Powered by vBulletin®. Copyright ©2000 - 2014, vBulletin Solutions, Inc.
SEO by vBSEO ©2010, Crawlability, Inc.