Discussion :

[feldene]

bonjour tout le monde
voila mon probleme je travaille sur le transfert d'un fichier .. d'un serveur vers un client via les sockets java... mais le probleme qui se pose actuellement c'est que je ne peus pas transferer des fichier grand genre 5Mo.. je peus transfere que des petit fichier 250ko ....
voila mon code

serveur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
package serv4;
 
import java.net.*;
import java.io.*;
 
public class Serveur {
  public static void main (String [] args ) throws IOException {
    // create socket
    ServerSocket servsock = new ServerSocket(13267);
    while (true) {
      System.out.println("Waiting...");
 
      Socket sock = servsock.accept();
      System.out.println("Accepted connection : " + sock);
 
      // sendfile
      File myFile = new File ("c:\\livres.pdf");
      byte [] mybytearray  = new byte [(int)myFile.length()];
      FileInputStream fis = new FileInputStream(myFile);
      BufferedInputStream bis = new BufferedInputStream(fis);
      bis.read(mybytearray,0,mybytearray.length);
      OutputStream os = sock.getOutputStream();
      System.out.println("Sending...");
      os.write(mybytearray,0,mybytearray.length);
      os.flush();
      sock.close();
      }
    }
}

Client :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
package serv4;
 
import java.net.*;
import java.io.*;
 
public class Client{
  public static void main (String [] args ) throws IOException {
    int filesize=602238; // filesize temporary hardcoded
 
    long start = System.currentTimeMillis();
    int bytesRead;
    int current = 0;
    // localhost for testing
    Socket sock = new Socket("127.0.0.1",13267);
    System.out.println("Connecting...");
 
    // receive file
    byte [] mybytearray  = new byte [filesize];
    InputStream is = sock.getInputStream();
    FileOutputStream fos = new FileOutputStream("d:\\livre.pdf");
    BufferedOutputStream bos = new BufferedOutputStream(fos);
    bytesRead = is.read(mybytearray,0,mybytearray.length);
    current = bytesRead;
 
 
    do {
       bytesRead =
          is.read(mybytearray, current, (mybytearray.length-current));
       if(bytesRead >= 0) current += bytesRead;
    } while(bytesRead > -1);
 
    bos.write(mybytearray, 0 , current);
    bos.flush();
    long end = System.currentTimeMillis();
    System.out.println(end-start);
    bos.close();
    sock.close();
  }
}

Merci

[Grimly_old]

Bonjour,

Les buffers par défaut sont trop petits pour ton application.
Tes méthodes fonctionnent très bien sans eux d'ailleurs.

En gros, retire "BufferedInputStream bis = new BufferedInputStream(fis);" et "BufferedOutputStream bos = new BufferedOutputStream(fos);"

EDIT :
Et pour la recopie d'un InputStream vers un OutputStream, je te conseil plutôt ce bout de code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
public static void recopier(InputStream in, OutputStream out) {
    byte[] buf = new byte[1024];
    // Pas besoin de monter la taille du buffer trop haut, les perfs n'evoluent pas tant que ça.
    int lu = 0;
    while ((lu = in.read(buf)) > 0)
        out.write(buf, 0, lu);
}

Salut

[feldene]

est ce que vous pouvez me proposer une solution directement sur mon code... car j'ai pas compris la solution
Merci

[professeur shadoko]

explication: ce qu'il y a d'anormal dans ton code c'est de créer un buffer de la taille du fichier: tu dois transférer bloc par bloc. (d'où la proposition d'utiliser 1024 et de boucler)

[tchize_]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
    do {
       bytesRead =
          is.read(mybytearray, current, (mybytearray.length-current));
       if(bytesRead >= 0) current += bytesRead;
    } while(bytesRead > -1);
 
    bos.write(mybytearray, 0 , current);
    bos.flush();

avec ce code, si le buffer est trop petit, tu va vite te retrouver à court de place.

Faut envoyer au fur et à mesure que tu lit dans le buffer, pas après avoir tenté de lire tout le fichier.