更好的 Java 循环？

Question

我有一段代码可以读取InputStream 并将内容写入OutputStream：

BufferedInputStream in = new BufferedInputStream(...);
FileOutputStream outStream = new FileOutputStream outStream(...);

int read = in.read(buffer, 0, bufferSize);
while (read != -1) {
    outStream.write(buffer, 0, read);
    read = in.read(buffer, 0, bufferSize);
}

它可以工作，但我不喜欢它，因为变量read 是在循环外声明的，而read() 方法被写了两次。 修改后的版本：

 for (int read = 0; read != -1; read = in.read(buffer, 0, bufferSize)) {
      outStream.write(buffer, 0, read);
 }

它看起来更好但不够好，因为第一次迭代在 read=0 时是无用的（并且可能有害）。

你有更好的解决方案吗？

Answer 1

就我个人而言，对于这类事情，我打破了正常的“条件无副作用”规则：

int bytesRead;
while ((bytesRead = in.read(buffer, 0, bufferSize)) != -1)
{
    outStream.write(buffer, 0, bytesRead);
}

编辑：如上所述，它确实涉及在循环外声明read，但它只调用一次read()。我从来没有发现它是一个问题——虽然我通常更喜欢用尽可能小的范围声明变量，但这更像是一个一般的清洁问题。如果您想进一步限制范围，您可以将整个内容放在大括号中，或者将其提取到自己的方法中，例如 Alan 的方法。以下是我的实现方式：

public static void copyStream(InputStream input, OutputStream output)
    throws IOException {
  byte[] buffer = new byte[1024 * 16]; // Reasonable general size

  int bytesRead;
  while ((bytesRead = in.read(buffer, 0, buffer.length)) != -1) {
    outStream.write(buffer, 0, bytesRead);
  }
}

或者，您可以提供缓冲区长度作为参数。请注意，这现在可以进入实用程序库，您无需再次编写代码。

或者，您可以使用它已经在其他实用程序库中可用的事实，例如 Guava 作为 ByteStreams.copy

Answer 2

你可以这样做：

BufferedInputStream in = new BufferedInputStream(...);
FileOutputStream outStream = new FileOutputStream outStream(...);

while (true) { // can i use for(;;) in Java ???
    int read = in.read(buffer, 0, bufferSize);
    if (read == -1) break;
    outStream.write(buffer, 0, read);
}

不过，它使用break。有人说break 风格不好/不太好。

Answer 3

这不是很出色，但是通过一个简单的块，您可以阻止read 变量在稍后的方法中被访问：

BufferedInputStream in = new BufferedInputStream(...);
FileOutputStream outStream = new FileOutputStream outStream(...);

{
    int read = in.read(buffer, 0, bufferSize);
    while (read != -1)
    {
        outStream.write(buffer, 0, read);
        read = in.read(buffer, 0, bufferSize);
    }
}

// ...rest of your code

...但我同意，我也经常想要一个 while 循环，其中要测试的值在循环内初始化。据我所知，这是不可能的。

另一种方法是使用extract method design pattern 将该循环拉出到一个完全独立的方法中，即

public void yourMethod() {
    BufferedInputStream in = new BufferedInputStream(...);
    FileOutputStream outStream = new FileOutputStream outStream(...);

    this.writeToOutputStream(in, outStream);
}

private void writeToOutputStream(InputStream in, OutputStream outStream) {
    int read = in.read(buffer, 0, bufferSize);
    while (read != -1)
    {
        outStream.write(buffer, 0, read);
        read = in.read(buffer, 0, bufferSize);
    }
}

Answer 4

这种形式比较常见：

while ((read = in.read(buffer, 0, bufferSize)) != -1) {
  ...
}

但不太清楚 IMO。