0

元々文字列しか送信できないAndroidコードを修正しようとしています。タスクは、写真を転送できるようにすることです。これを実装するには非常に多くの異なる方法があることを私は知っています。しかし、接続には問題はありませんが、写真を送信できない理由が本当にわかりません。教えてくれる人はいますか？本当に感謝します。私はただの初心者です。下手なプログラミング スキルは忘れてください。

以下はサーバーの主要部分です

private Runnable socket_server = new Runnable(){
    public void run(){
        handler.post(new Runnable() {
            public void run() {
                test.setText("Listening...." + getMyIp());

            }
        });
        try{ 
            serverSocket = new ServerSocket(1234);

            while (true) {
                Socket client = serverSocket.accept();

                handler.post(new Runnable() {
                    public void run() {
                        test.setText("Connected.");
                    }
                });
                try {
                     File myFile = new File ("/sdcard/DCIM/img2.jpg");
                      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 = client.getOutputStream();
                      os.write(mybytearray,0,mybytearray.length);
                      os.flush();
                      client.close();
                    test.setText("Received.");

                } catch (Exception e) {
                    handler.post(new Runnable() {
                        public void run() {
                            test.setText("Sending erro");
                        }
                    });
                }
            }
        }catch(IOException e){
            handler.post(new Runnable() {
                public void run() {
                    test.setText("Fail to buitl the socket");
                }
            });
        }
    }
};

ここにクライアントの一部があります

new Thread() 
            {
                @Override 
                public void run() { 
                    // TODO Auto-generated method stub 
                    InetAddress serverAddr = null;
                    SocketAddress sc_add = null;
                    Socket socket = null;

                    try 
                    { 
                        serverAddr = InetAddress.getByName("192.168.1.105");
                        sc_add= new InetSocketAddress(serverAddr,1234);
                        socket = new Socket();
                        socket.connect(sc_add,2000);

                        File myFile = new File("/sdcard/DCIM/img2.jpg");
                        InputStream fis = new FileInputStream("/sdcard/DCIM/img2.jpg");
                        OutputStream outputStream = socket.getOutputStream();

                        byte [] buffer = new byte[(int)myFile.length()];

                        int temp = 0 ;  
                        while((temp = fis.read(buffer)) != -1){  
                            outputStream.write(buffer, 0, temp);  
                        }  
                        outputStream.flush();
                        socket.close();


                } catch (UnknownHostException e) {

                    //TextView01.setText("InetAddress fail");

                } catch (SocketException e) {

                    //TextView01.setText("fail to develop socket");

                } catch(IOException e) {

                    //TextView01.setText("fail to sending");

                }
                } 

            }.start();
4

1 に答える 1

2

これがあなたの問題です：

byte [] mybytearray  = new byte [(int)myFile.length()];
// ...
bis.read(mybytearray,0,mybytearray.length);
OutputStream os = client.getOutputStream();
os.write(mybytearray,0,mybytearray.length); // <- This one.

ここで一度これを書きました:バッファリングされていないストリームの作業

問題は、このコードがバッファが完全にいっぱいであると想定していることです。これは必ずしもそうではありません。InputStream.read(byte[], int, int) のドキュメントには次のように記載されています。

入力ストリームから最大len バイトのデータをバイト配列に読み取ります。len バイトまで読み取ろうとしますが、それより少ない数が読み取られる可能性があります。実際に読み取られたバイト数は整数として返されます。[…]

そのため、バッファがいっぱいになる保証はありません。これは、OutputStream.write(byte[], int, int)-method を使用して読み取りバイトを出力ストリームに書き込むときに問題になります。ドキュメントには次のように書かれています。

オフセット off から始まる指定されたバイト配列から [正確に] len バイトをこの出力ストリームに書き込みます。[…]

これが実用的な実装です：

byte buffer[] = new byte[1024];
int read_count = 0;
while ((read_count = input.read(buffer, 0, buffer.length)) != -1) {
    output.write(buffer, 0, read_count); // Now writes the correct amount of bytes
}

また、ファイルをチャンクで転送する方が効率的であるため、バッファーはファイル全体ほど大きくしないでください。詳細な説明については、この古い質問を参照してください: FileInputStream を使用する場合、理想的なバッファー サイズをどのように決定しますか?

于 2013-08-16T21:44:42.943 に答える