更好的 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< /code> 被声明为循环外，并且 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()一次。我从来没有发现这是一个问题——虽然我通常更喜欢声明范围尽可能小的变量，但这更多的是一个一般的清洁问题。如果您想进一步限制范围，您可以将整个内容放在大括号中，或者将其提取到自己的方法中，就像艾伦的方法一样。以下是我实现它的方法：

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 循环，其中值要测试的内容在循环内初始化。据我所知，这是不可能的。

另一种方法是使用提取方法设计模式将该循环提取到完全独立的方法，即

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) {
  ...
}

但在我看来，不太清晰。