Impossible de transferer des fichiers long

Version imprimable

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:

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:

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

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:

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 ;)

24/03/2010, 17h06
feldene

est ce que vous pouvez me proposer une solution directement sur mon code... car j'ai pas compris la solution
Merci
24/03/2010, 17h30
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)

Code:

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.