Pls help: server send a file upon client's request

Question

I just started learning Java and I'm trying to write a simple file transfer program. My problem should be in the transfer part. Code as follows:
Client side:
package p2pclient;
import java.io.*;
import java.net.*;
public class P2PClient {
public static void main(String argv[]) throws Exception
  String filename;
  try {
  Socket clientSocket = new Socket("localhost", 6783);
  BufferedReader inFromUser = new BufferedReader( new InputStreamReader(System.in));
  filename = inFromUser.readLine();
  DataOutputStream outToServer = new DataOutputStream(clientSocket.getOutputStream());
  outToServer.writeBytes(filename);
  String newname = "C:\\" + filename;
  File file = new File(newname);
  byte[] Byte = new byte[1120];
  DataInputStream inFromSocket = new DataInputStream(clientSocket.getInputStream());
  FileOutputStream fout = new FileOutputStream(file);
  int reads;
  while((reads = inFromSocket.read(Byte)) != -1){
  fout.write(Byte, 0, Byte.length);
  clientSocket.close();
  catch (java.io.IOException e) {
      System.err.println("error connecting to " + "Daniel-Ding" + ":" + e);
      return;
}On the server side:
package p2pserver;
import java.io.*;
import java.net.*;
import java.util.logging.*;
class FileTransfer implements Runnable {
  private Socket connectionSocket;
  FileTransfer(Socket connectionSocket) {
    this.connectionSocket = connectionSocket;
  public void run(){
        try {
            String fileName;
            BufferedReader inFromClient = new BufferedReader(new InputStreamReader(connectionSocket.getInputStream()));
            fileName = inFromClient.readLine();
            System.out.println("received filename: "+ fileName);
            File fl = new File(fileName);
            byte[] filebuffer = new byte[1120];
            FileInputStream fin = new FileInputStream(fl);
            int bytesRead;
        while ((bytesRead = fin.read(filebuffer)) != -1) {
            connectionSocket.getOutputStream().write(filebuffer, 0, bytesRead);         
            connectionSocket.close();
        } catch (IOException ex) {
            Logger.getLogger(FileTransfer.class.getName()).log(Level.SEVERE, null, ex);
class P2PServer {
    public static void main(String argv[]) throws Exception
        ServerSocket welcomeSocket = new ServerSocket(6783);
        while(true) {
            Socket connectionSocket = welcomeSocket.accept();
            FileTransfer ft;
            ft = new FileTransfer(connectionSocket);
            Thread thread = new Thread(ft);
            thread.start();
}I can see an empty file with the name of 'newname' in my C drive but no data transfered.
Thanks a lot in advance.

Answer 1

Hi Peter,
Sorry to bother you again. I tried to increase the size of the buffer. The speed improved but the transfered file quality became intolerable. I tested it with some mp3 files and found that with large buffer size there was a lot of noise in the transfered mp3s. Any idea what I can do to optimize this trade-off?
Many thanks.
Best,
Daniel