Re: Java (android) socket reconnection
On 09/12/2012 21:36, artik allegedly wrote:
Thank You very much! Honestly.
I started analyze Your code but it is partly to hard for me - I'm newbe in Java and it gives me error on invoking method "getWriter" in line:
Writer w = connector.getWriter()
and further exactly inside this method in line "else if( ! socket.isClosed() )". I'm going fight with this in next hours, and don't give up.
One hour ago I write my code as simple as I can and in one thread - everything to avoid problem with synchronization. Effect on the server is still the same (several connection attempts after starting server when client was waiting for it)
I apologize that I'm putting my code again (I hope in good format) instead try to adapt new one from You immediately, but I want to understand this problem.
<snip />
Ah, shoot. I forgot the null check in the line you mention (btw, when
you get an error, always say *what* error it is. Best is to post the
stacktrace of the error; in this case it's obvious, but it's seldom that
easy).
Yes, the format is better this time. Also, you probably have solved your
synchronisation issues for the time being.
After the initial sleep, you should not go into the rest of the loop if
you have been interrupted (being interrupted means "stop doing things",
remember). Likewise, your last code is a regression from before, where
you had the interruptedness as the loop condition (in the while clause),
which is definitely right. In the latest form, AFAICS, the thread won't
exit even if it is interrupted.
As for the problem you mention: it is normal that you'd get several
connection attempts. Do you think there's anything wrong with that?
Client tries, server ain't there, client retries later. Seems okay to
me. What you said before was that your app was crashing after a while in
that case. I'm not sure why that is the case, but a candidate would be
to close() the socket you've created when it fails to connect. I.e.:
try {
sock = new Socket();
sock.connect(new InetSocketAddress(address, 5000), 1000);
out = new BufferedWriter( new
OutputStreamWriter(sock.getOutputStream()));
} catch (IOException e1) {
if( sock != null ){
try { sock.close(); } catch( IOException ioex ){
ioex.printStacktrace(); }
sock = null;
}
}
--
DF.