Socket Connection Error

Question

Hi,
I m facing some strange issues in a java file. The requirement is something like, i need to process some business operation , connect to a third party vendor and they need to process some operation in 10 secs and they reverted back with the result. The whole process should complete in 15 secs.
So for the first time while connection to the third party it is working fine, but for the second time it fails and showing connection failed in the UI. When i retires it is successfully connects. i checked the log file, there it is throwing exceptions some of the time. Please find the piece of code below. From the third party there is no issues from them , all the operations sucessfully done from their side..
try {
// open socket connection
socket = openSocketConnection(host, port, connectTimeOut, responseTimeout);
if(socket.isConnected()){
// some steps of business processing.....
// write request
// read response
// set pipeline
// set error status
catch(SocketTimeoutException ste){}
catch (UnknownHostException uhe) {}
catch (SocketException soe) {}
catch (Exception e) { }
finally {
try {
if (socket != null)socket.close();
} catch (Exception e) { }
// method for connecting the server....
openSocketConnection(String host, int port, int connectTimeOut, int responseTimeout) throws SocketTimeoutException, ServiceException,IOException {
Socket socket = null;
socket = new Socket();
socket.bind(null);
socket.connect(new InetSocketAddress(host,port),connectTimeOut);
if (socket.isConnected()) {
               socket.setSoTimeout(responseTimeout);
} else {
     throw new ServiceException("Error opening socket connection to Third party Vendor.");<------- mostly in second time throwing exception, but when retires working fine...
return socket;
Can anyone please help me out for this.

Answer 1

Remove the isConnected() test and associated code: it's contributing nothing here. If the socket itself throws an exception then you have a problem. At the moment all you have is a self-inflicted wound.
You can also remove the bind(null) call.