【问题标题】:Sockets, sent pdf file always arrives zero bytes size套接字,发送的 pdf 文件总是到达零字节大小
【发布时间】:2014-01-07 07:06:20
【问题描述】:

我正在将 pdf 文件从 android 平板电脑客户端发送到在 Windows 7 上运行的 Java 应用程序。该文件始终以零字节大小到达。这里有什么问题?

在客户端向服务器发送pdf文件之前,客户端向服务器发送的文件大小为long值,这个大小是正确的并且总是到达服务器。对于 pdf 文件,我用于此测试的大小为 566718 字节。

如何让 pdf 文件以正确的大小到达?

服务器代码

 public class Server {
 ServerSocket serverSocket;
 Socket socket;
 boolean runner = true;

Server() throws IOException{

    serverRunner();
    System.out.println("server constructor started");

} // Server() constructor

public void serverRunner() throws IOException {

    System.out.println("serverrunner started");

     try {

      serverSocket = new ServerSocket(6789, 100);

      runner = true;

        while (runner) {

            socket = serverSocket.accept();

            MultiThreader multi = new MultiThreader(socket);
            Thread t = new Thread(multi);
            t.start();

        }  // while runner
       } catch (IOException ex) {

    }

 } // serverRunner()

 } // class Server


public class MultiThreader implements Runnable {
Socket socket;
public int fileSizeFromClient;
FileOutputStream fos = null;
BufferedInputStream bis = null;
BufferedOutputStream bos = null;
DataInputStream dis = null;
DataOutputStream dos = null;

public MultiThreader(Socket socket){
    System.out.println("print out from multithreader class");
    this.socket = socket;

} // multiThreader

@Override
public void run() {

    System.out.println("multi threader started");

    // action #1 read file from client =====================================
    // transfer.pdf read this file sent from android device to this computer

    int bufferSize = 0;

    try {

     bis = new BufferedInputStream(socket.getInputStream());
     dis = new DataInputStream(bis);

     fileSizeFromClient = dis.readInt();
     System.out.println("file size from client is " + fileSizeFromClient);

      File fileDirectory = new File("C:/DOWNLOAD/");
        if (!fileDirectory.exists()) {
            fileDirectory.mkdir();
      }
      File file = new File("C:/DOWNLOAD/transfer.pdf");
      file.createNewFile();

       fos = new FileOutputStream(file);
       bos = new BufferedOutputStream(fos);
       dos = new DataOutputStream(bos);

       byte[] buffer = new byte[fileSizeFromClient];

      int totalBytesRead = 0;

        while(totalBytesRead < fileSizeFromClient){
            int bytesRemaining = fileSizeFromClient = totalBytesRead;
            int bytesRead = dis.read(buffer, 0, (int) Math.min(buffer.length, bytesRemaining));
            if(bytesRead == -1) {
                break;
            } else {
                dos.write(buffer, 0, bytesRead); 
                    totalBytesRead += bytesRead;
            }
        } // while

      } catch (IOException ex) {
        Logger.getLogger(MultiThreader.class.getName()).log(Level.SEVERE, null, ex);
      } finally {
        try {
          //  socket.close();
        } catch (IOException ex) {
            Logger.getLogger(MultiThreader.class.getName()).log(Level.SEVERE, null, ex);
        }
    }

} // run

} // MultiThreader

客户端代码

 public class MainActivity extends Activity implements Runnable {

TextView textViewOne;
Button buttonOne;
Socket socket;
private String serverIP = "192.XXX.X.X";
FileInputStream fis;
FileOutputStream fos;
private File file;
DataInputStream dis;
DataOutputStream dos;
BufferedInputStream bis;
BufferedOutputStream bos;
long length;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

   textViewOne = (TextView) findViewById(R.id.textView1);
   buttonOne = (Button) findViewById(R.id.button1);

   buttonOne.setOnClickListener(new OnClickListener() {

    @Override
    public void onClick(View v) {
        Thread myThread = new Thread(MainActivity.this);
        myThread.start();

    }

   });

} // oncreate

@Override
public void run() {

    MainActivity.this.runOnUiThread(new Runnable(){
        @Override
        public void run() {
           textViewOne.setText("run method started");
        }
        });

    try {
        socket = new Socket(InetAddress.getByName(serverIP), 6789);

        if (socket == null) {
            return;
        } else {
            MainActivity.this.runOnUiThread(new Runnable(){
                @Override
                public void run() {
                   textViewOne.setText("connected");
                }
                });

        }

        file = new File(Environment.getExternalStorageDirectory().getPath() + File.separator + "transfer.pdf");

        length = file.length();

        fis = new FileInputStream(file);
        bis = new BufferedInputStream(fis);
        dis = new DataInputStream(bis);

        bos = new BufferedOutputStream(socket.getOutputStream());
        dos = new DataOutputStream(bos);

        dos.writeInt((int) length); // sends the length as number bytes is file size

        int count = 0;
        byte[] buffer = new byte[(int) length];

        while ((count = bis.read(buffer)) > 0)
        {
            bos.write(buffer, 0, count);
        }

        bos.flush();
        bis.close();
        socket.close();

    } catch (UnknownHostException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    } catch (IOException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }

}

 } // mainactivity

【问题讨论】:

  • 是否记录了任何异常?为什么不关闭服务器上的输出流?您是否尝试过在复制循环中添加诊断信息?
  • 你不需要调用 createNewFile(),但是你需要关闭你的输出流。

标签: java android sockets tcp


【解决方案1】:

这就是问题所在,我相信。

int bytesRemaining = fileSizeFromClient = totalBytesRead;

这是做两个赋值,所以你立即将 0 赋值给 fileSizeFromClient,然后退出循环。

你的意思是:

int bytesRemaining = fileSizeFromClient - totalBytesRead;

这是一个非常微妙的错字,你很不幸这是一个仍然导致有效代码的错字:(

鉴于无论如何您都立即关闭套接字,尚不清楚为什么要先发送文件大小。如果您在客户端和服务器上都具有相同的“从输入流复制到输出流,直到输入用完数据”,您的代码可能会更简单,只是从 FileInputStreamSocket OutputStream 在客户端,SocketInputStream 到服务器上的 FileOutputStream(可能带有缓冲包装器)。

我还建议关闭 all 流 - 如果您使用的是 Java 7,则只需使用 try-with-resources 语句即可;在早期版本中,您应该关闭 finally 块中的流。

【讨论】:

    【解决方案2】:

    -根据经验法则,始终在写入/读取数据流后关闭数据流

    -服务器端关闭流。

    -SocketInputStreamScanner 一起使用,可以在套接字之间轻松进行数据传输。 (这就是我的感受,正在尝试使用套接字。)

    【讨论】:

    • 我肯定会避免在这里使用Scanner。这里绝对不需要。
    • Mr.Skeet,我已经尝试过 Scanner、InputStream 和 InputStreamReader 的组合...我觉得它对我有用几次...当我尝试组合 Socket、InputStream和扫描仪,它击中了目标......我是你的忠实粉丝......如果可以的话,我很想纠正我使用扫描仪的错误,请让我知道它的不良影响。我有义务
    • 好吧,当你没有使用Scanner时,我们真的无法判断你做错了什么,尽管如果你在非文本上使用InputStreamReader数据,那将是一个问题。但是我发现Scanner API 的许多 问题与预期不符,而对于仅传输数据,InputStreamOutputStream API 确实非常简单。
    • 我的意思是如果我使用它会产生问题Scanner reader = new Scanner(socket.getInputStream());
    • 很可能不会,但是当您只是尝试将数据从一个流复制到另一个流时,它会增加复杂性而没有明显的好处。
    猜你喜欢
    • 2017-03-14
    • 1970-01-01
    • 2014-05-28
    • 1970-01-01
    • 2016-08-01
    • 2012-07-12
    • 1970-01-01
    • 2014-06-19
    • 2011-08-01
    相关资源
    最近更新 更多