Socket InputStream readObject > Connection Reset

[Raildex1]

Moin moin!

Ich programmier gerade einen multifunktionalen Chat und läuft soweit alles ganz gut.

Das Schicken von Nachrichten funktioniert auch alles.
Es geht mir aber um eine Exception die auftritt, wenn die Methode

readObject();

vom Socket-InputStream gerade lesen möchte, aber der Client abbricht.
Das kann ja durchaus gerne mal passieren, dass der Client einfach mal das Programm ausschaltet.
Die Exception lautet wie folgt:

java.net.SocketException: Connection reset
at java.net.SocketInputStream.read(SocketInputStream.java:196)
at java.net.SocketInputStream.read(SocketInputStream.java:122)
at java.net.SocketInputStream.read(SocketInputStream.java:210)
at java.ibjectInputStream$PeekInputStream.peek(ObjectInputStream.java:2293)
at java.ibjectInputStream$BlockDataInputStream.peek(ObjectInputStream.java:2586)
at java.ibjectInputStream$BlockDataInputStream.peekByte(ObjectInputStream.java:2596)
at java.ibjectInputStream.readObject0(ObjectInputStream.java:1318)
at java.ibjectInputStream.readObject(ObjectInputStream.java:370)
at de.borstgaitzsch.transfer.ReceiveListener.run(ReceiveListener.java:50)

und die Zeile ist diese hier:

        JeezyMessage receive = (JeezyMessage) ois.readObject();

wie gesagt, ich möchte nur die Exception loswerden, so dass das Socket geschlossen werden kann und der Server die Verbindung wieder freigibt.

 

[Sen-Mithrarin]

dafür brauchen wir erstmal mehr code ... vor allem die stelle des read-loops um zu gucken wie die exception gefangen wird
sinnvoll wäre, wenn Java7 genutzt wird, try-with-resources

 

[Raildex1]

 @Override
  public void run()
  {
    while (socket.isConnected())
    {
      try
      {
        System.out.println("Waiting for incoming message...");
        JeezyMessage receive = (JeezyMessage) ois.readObject();
        if (receive != null)
        {
          System.out.println("Message received...");
          client.forwardMessage(receive);
        }
      }
      catch (IOException | ClassNotFoundException ex)
      {
        Logger.getLogger(JeezyServer.class.getName()).
          log(Level.SEVERE, null, ex);
      }
    }
    try
    {
      ois.close();
      socket.close();
    }
    catch (IOException ex)
    {
      Logger.getLogger(ReceiveListener.class.getName()).log(Level.SEVERE, null,
        ex);
    }

  }

das ist meine run-Methode.

 

[Sen-Mithrarin]

Socket.isConnected() ... schrott
grund : isConnected() übeprüft NICHT ob ein Socket noch verbunden ist ... sondern nur ob er bei abfrage korrekt verbunden war

public boolean isConnected()

Returns the connection state of the socket.
Note: Closing a socket doesn't clear its connection state, which means this method will return true for a closed socket (see isClosed()) if it was successfuly connected prior to being closed.

Returns: true if the socket was successfuly connected to a server
Since: 1.4

gleich vorweg : Socket.isClosed() brauchst du garnicht erst zu versuchen ... wird nur gesetzt wenn man mit Socket.close() den Socket selbst schließt ... und nicht wenn er abbricht

public boolean isClosed()

Returns the closed state of the socket.

Returns: true if the socket has been closed
Since: 1.4
See Also: close()

try-catch innerhalb eines loops der SO nie beendet wird ... schrott
wenn du schon ein while(FLAG) baust ... dann setze dieses im catch-block auch auf false
außerdem : in einen catch-block gehört IMMER ein Throwable.printStackTrace() ... völlig egal ob man logging nutzt !

nachträgliches try-catch um bei getrennter verbindung (was ja wie gesagt bei diesem code SO niemals passieren wird) um die sockets zu schließen ? ... wie gesagt : Java7 und try-with-resources




ergo : du bekommst du Exception völlig zurecht und dein code räumt auch nie auf ... weil du ihn so programmiert hast
google mal wie man richtig mit connections umgeht ... SuFu hilft auch weiter