Strutsによるバイナリファイル出力

質問

FileInputStream stream =new FileInputStream("C:/Test.exe");
String filename = "Test1.exe";
int bytesRead = 0;
byte[] buffer = new byte[256];
HttpResponse.setContentType("application/octet-stream; charset=UTF-8");
HttpResponse.setHeader("Content-Disposition","attachment; filename=\""+filename+"\"");
ServletOutputStream sos =HttpResponse.getOutputStream();
while ((bytesRead = stream.read(buffer,0,256) != -1) {
　sos.write(buffer);
}
sos.flush();
sos.close();
stream.close();

以上のコードでローカル内にあるファイルを名前を変え転送していますが
クライアントにより正常なファイルだったり異常なファイルだったりします。
異常なファイルはクライアントによりばらばらですが、ファイルサイズが正常のものより少し大きかったりします。
バッファーではなくバイト単位での転送も試みたのですが
結果は同じでした。

何かおかしい点ありましたらご指摘いただけますでしょうか。

回答No.1

ukiuni · ベストアンサー

OutputStreamの使い方の問題ですね。
で、OutPutStream#write(byte[] buffer)は、bufferのすべてを書き込む、と言うものです。

記載のコードの問題はInputStreamで読み込んでいない分のbufferまで書き込まれてしまうものです。
例えば、「ABCDEFGH」と言うファイルがあり、bufferが3文字読み込めるとして
ABC
DEF
GH
のようにbufferには読み込まれますが、bufferは三文字分なので、「GH」を読み込んだ際に、前回読み込んでいたFが上書きされずに残ります。書き込んだときは
ABC
DEF
GHF
となります。

正しいコードは
sos.write(buffer);⇒sos.write(buffer,0,bytesRead);
と、読み込んだ分だけ、出力するようにします。

お礼

ukiuni様
ご回答ありがとうございました。
非常に丁寧で分かりやすいご解説感謝いたします。

ukiuni様が書かれた様に修正を行ったのですが
やはりクライアントにより、できたりできなかったりします。
ですが、以前のように同クライアントでできたりできなかったりという現象はなくなりました。

こうなるとクライアントの環境次第のような気もするのですが
ありえないという気もします…

なにか他に気付かれた点ございませんでしょうか