Forum: Ruby TCPServer.new().close

Posted by Mamba Black (mamba)
on 2013-01-30 15:54
Hi,
  Please correct if my understanding is wrong for the below code:-
Code to check if a port is free or not.

def port_available?(port)
  begin
    TCPServer.new('localhost', port).close
    rescue
      return false
    else
     return true
  end
end

Port will be closed. While closing if an exception occurs, it
returns false. Else true.

Thanks
Posted by Robert Klemme (robert_k78)
on 2013-01-30 17:53
(Received via mailing list)
On Wed, Jan 30, 2013 at 3:54 PM, Mamba Black <lists@ruby-forum.com> 
wrote:
>      return true
>   end
> end

I personally prefer to put the "true" before the "rescue" because this
is the regular control flow.  You can even change that to a one liner

def port_alive? port
  (TCPServer.new('localhost', port).close; true) rescue false
end

> Port will be closed. While closing if an exception occurs, it
> returns false. Else true.

But note that your system may have multiple network addresses and this
check verifies just one.  The same port could be occupied on another
address.

Plus, if you do the check to decide whether to start a server on that
port the port may be closed the very moment you try to open the port
"for real".  In these situations it's usually better to just open the
port and deal with the error because you need to do that anyway.

Kind regards

robert
Please log in before posting. Registration is free and takes only a minute.
Existing account (Switch to SSL-encrypted connection)
NEW: Do you have a Google/GoogleMail or Yahoo account? No registration required!
Log in with Google account | Log in with Yahoo account
No account? Register here.