[Socket] Fermeture correcte

Version imprimable

Bonjour,

vous l'aurez peut-être remarqué que tous mes problèmes sont basés sur le concept des sockets, cette fois-ci je cherche le meilleur moyen de fermer son (client/serveur) après un transfert de fichiers, le mien m'affiche l'erreur : (Connection reset by peer) je pense que j'interrompe la connexion avant qu'elle ait fini, mais je ne sais pas comment résoudre ça, voici mon code :

Envoi :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
 OutputStream os = null;
        DataOutputStream dos = null;
        Socket sock = null;
 
        try {
            sock = new Socket("127.0.0.1", 80);
 
 
            File myFile = new File("D:\\Zest\\1.png");
            byte[] mybytearray = new byte[(int) myFile.length()];
 
            FileInputStream fis = new FileInputStream(myFile);
            BufferedInputStream bis = new BufferedInputStream(fis);
 
 
            DataInputStream dis = new DataInputStream(bis);
            dis.readFully(mybytearray, 0, mybytearray.length);
 
            os = sock.getOutputStream();
 
                 dos = new DataOutputStream(os);
            dos.writeUTF(myFile.getName());
            dos.writeLong(mybytearray.length);
            dos.write(mybytearray, 0, mybytearray.length);
            dos.flush();
 
 
            os.write(mybytearray, 0, mybytearray.length);
            os.flush();
 
            os.close();
           dos.close();  
          sock.close();  
 
        } catch (IOException ex) {
 
            Logger.getLogger(Envoyer.class.getName()).log(Level.SEVERE, null, ex);
 
            System.out.print("Erreur 1: "  + ex.getMessage());
 
        }

Réception:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
   public class thread_envoi extends Thread {
 
        @Override
        public void run() {
 
             try {
            int bytesRead;
            int current = 0;
 
            ServerSocket serverSocket = null;
            serverSocket = new ServerSocket(80);
 
            while(true) {
                Socket clientSocket = null;
                clientSocket = serverSocket.accept();
 
                InputStream in = clientSocket.getInputStream();
 
                DataInputStream clientData = new DataInputStream(in);
 
                String fileName = clientData.readUTF();
                OutputStream output = new FileOutputStream("D:\\"+fileName);
                long size = clientData.readLong();
                byte[] buffer = new byte[1024];
                while (size > 0 && (bytesRead = clientData.read(buffer, 0, (int)Math.min(buffer.length, size))) != -1)
                {
                    output.write(buffer, 0, bytesRead);
                    size -= bytesRead;
                }
 
 
                in.close();
                clientData.close();
                output.close(); 
 
            }   } catch (IOException ex) {
            Logger.getLogger(Recevoir.class.getName()).log(Level.SEVERE, null, ex);
        }
 
 
        }
 
    }

Merci pour votre aide !

Je dirai a priori que l'ordre de tes fermetures est naze puisque tu fermes os avant de fermer dos. Mais je peux me tromper.

Je te rappelle que si tu utilises Java 7, le try-with-resources peut t’éviter de devoir explicitement invoquer close() et que les fermetures auront lieu 1) implicitement et 2) dans l'ordre inverse de leur déclaration :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 try (Socket sock = [...]; OutputStream os = [...]; DataOutputStream dos = [...]) { [...] try (FileInputStream fis = [...]; BufferedInputStream bis = [...]) { [...] } [...] }
De plus, toujours depuis Java 7, la classe utilitaire java.nio.files.Files dispose de methodes permettant de recopier le contenu de fichiers/paths dans des flux sans se casser la tête.

09/09/2015, 10h31
thelvin

En l'occurrence, le problème est qu'il envoie deux fois son fichier alors qu'il ne le lit qu'une fois et que le lecteur ferme de son côté après première lecture.

Mais ce problème ne serait pas arrivé avec des ouvertures et fermetures rendues claires et nettes par une bonne utilisation de try-with-resource.
09/09/2015, 16h03
guatto

Merci tous les deux pour vos explication, l'erreur venait d'un bout de code en plus, t'avais bien vu "thelvin", et toi aussi "bouye" concernant le (try-with-resources) ! ;)