19

我正在使用以下方式InputStream写入File

private void writeToFile(InputStream stream) throws IOException {
    String filePath = "C:\\Test.jpg";
    FileChannel outChannel = new FileOutputStream(filePath).getChannel();       
    ReadableByteChannel inChannel = Channels.newChannel(stream);
    ByteBuffer buffer = ByteBuffer.allocate(1024);
    
    while(true) {
        if(inChannel.read(buffer) == -1) {
            break;
        }
        
        buffer.flip();
        outChannel.write(buffer);
        buffer.clear();
    }
    
    inChannel.close();
    outChannel.close();
}

我想知道这是否是使用 NIO 的正确方法。我读过一个方法FileChannel.transferFrom,它需要三个参数:

  1. ReadableByteChannel src
  2. 多头头寸
  3. 长数

在我的情况下,我只有src,我没有positionand count,有什么办法可以使用这种方法来创建文件?

同样对于 Image 是否有更好的方法来仅从InputStreamNIO 创建图像?

任何信息都会对我非常有用。这里有类似的问题,在 SO 中,但我找不到适合我的情况的任何特定解决方案。

4

2 回答 2

49

我会使用 Files.copy

Files.copy(is, Paths.get(filePath));

至于你的版本

  1. ByteBuffer.allocateDirect更快 - Java 将尽最大努力直接在其上执行本机 I/O 操作。

  2. 关闭是不可靠的,如果第一次失败,第二次将永远不会执行。使用 try-with-resources 代替,ChannelsAutoCloseable也是。

于 2013-05-16T08:02:41.210 回答
7

不,这是不正确的。您冒着丢失数据的风险。规范的 NIO 复制循环如下:

while (in.read(buffer) >= 0 || buffer.position() > 0)
{
  buffer.flip();
  out.write(buffer);
  buffer.compact();
}

请注意更改的循环条件,它负责在 EOS 上刷新输出,而使用 ofcompact()而不是clear(),负责短写入的可能性。

类似地,规范transferTo()/transferFrom()循环如下:

long offset = 0;
long quantum = 1024*1024; // or however much you want to transfer at a time
long count;
while ((count = out.transferFrom(in, offset, quantum)) > 0)
{
    offset += count;
}

它必须在循环中调用,因为不能保证传输整个量程。

于 2013-05-16T07:51:55.413 回答