24

HttpServletResponseに書き込みたいInputStreamがあります。このアプローチがありますが、byte[]を使用しているため時間がかかりすぎます

InputStream is = getInputStream();
int contentLength = getContentLength();

byte[] data = new byte[contentLength];
is.read(data);

//response here is the HttpServletResponse object
response.setContentLength(contentLength);
response.write(data);

スピードと効率の観点から、それを行うための最良の方法は何でしょうか。

4

3 に答える 3

51

最初にJavaのメモリに完全にコピーするのではなく、ブロックに書き込むだけです。以下の基本的な例では、10KB のブロックで書き込みます。このようにして、コンテンツ全体の長さではなく、10KB のみの一貫したメモリ使用量になります。また、エンドユーザーがコンテンツの一部をより早く入手できるようになります。

response.setContentLength(getContentLength());
byte[] buffer = new byte[10240];

try (
    InputStream input = getInputStream();
    OutputStream output = response.getOutputStream();
) {
    for (int length = 0; (length = input.read(buffer)) > 0;) {
        output.write(buffer, 0, length);
    }
}

パフォーマンスに関しては、クレーム デ ラ クレームとして、NIOChannelsと直接割り当てられた を使用できますByteBuffer。いくつかのカスタム ユーティリティ クラスで次のユーティリティ/ヘルパー メソッドを作成しますUtils。

public static long stream(InputStream input, OutputStream output) throws IOException {
    try (
        ReadableByteChannel inputChannel = Channels.newChannel(input);
        WritableByteChannel outputChannel = Channels.newChannel(output);
    ) {
        ByteBuffer buffer = ByteBuffer.allocateDirect(10240);
        long size = 0;

        while (inputChannel.read(buffer) != -1) {
            buffer.flip();
            size += outputChannel.write(buffer);
            buffer.clear();
        }

        return size;
    }
}

次に、以下のように使用します。

response.setContentLength(getContentLength());
Utils.stream(getInputStream(), response.getOutputStream());
于 2012-04-13T14:21:58.510 に答える
1 
BufferedInputStream in = null;
BufferedOutputStream out = null;
OutputStream os;
os = new BufferedOutputStream(response.getOutputStream());
in = new BufferedInputStream(new FileInputStream(file));
out = new BufferedOutputStream(os);
byte[] buffer = new byte[1024 * 8];
int j = -1;
while ((j = in.read(buffer)) != -1) {
    out.write(buffer, 0, j);
}
于 2012-04-13T14:18:40.930 に答える
0

それが最善の方法に非常に近いと思いますが、次の変更をお勧めします。固定サイズのバッファ (たとえば 20K) を使用してから、ループで読み取り/書き込みを行います。

ループの場合、次のようなことを行います

byte[] buffer=new byte[20*1024];
outputStream=response.getOutputStream();
while(true) {
  int readSize=is.read(buffer);
  if(readSize==-1)
    break;
  outputStream.write(buffer,0,readSize);
}

ps: read は与えられた配列全体を常に満たすとは限らないため、プログラムが常にそのまま機能するとは限りません。

于 2012-04-13T14:20:46.403 に答える