Question

我有一段用于流复制的代码。

OutputStream os = ...;
InputStream is = ...;
int bufferLength;
byte[] buffer = new byte[1024];
while ((bufferLength = is.read(buffer)) != -1) {
   os.write(buffer, 0, bufferLength);
}

如果我对它运行PMD，我会收到以下警告http://pmd.sourceforge.net/rules/controversial.html#AssignmentInOperand。

现在我希望摆脱那个警告，但我能想到的唯一选择就是

OutputStream os = ...;
InputStream is = ...;
int bufferLength;
byte[] buffer = new byte[1024];
bufferLength = is.read(buffer);
while (bufferLength != -1) {
   os.write(buffer, 0, bufferLength);
   bufferLength = is.read(buffer);
}

我并不喜欢这样，因为我最终会复制代码。是否有更优雅的方式来满足此PMD规则？

Answer 1

我只是建议您使用Commons IO：

IOUtils.copy(is, os);

然后我快速浏览了copy()的源代码：

byte[] buffer = new byte[DEFAULT_BUFFER_SIZE];
int n = 0;
while (-1 != (n = input.read(buffer))) {
    output.write(buffer, 0, n);
}

我认为你的代码是有效的，并保持原样。或者也许do-while循环可以解决这个问题？

Answer 2

最优雅的方式是抑制警告。

PMD附带了许多规则，您的想法是选择要在自己的代码中使用的规则。如果您认为，操作数中的分配是正常的，只需suppress the warning：

@SuppressWarnings("PMD.AssignementInOperand")

顺便说一句，无论如何，这都是在有争议的规则集中定义的。我根本不会激活它。

有争议的规则集包含的规则无论出于何种原因都被认为是有争议的。它们在这里被分开，以允许人们通过自定义规则集包含他们认为合适的内容。这个规则集最初是为了回应汤姆喜欢的不必要的构造规则的讨论而创建的，但是大多数人真的不喜欢:-)

在使用PMD一段时间之后，您应该开始考虑包含所有规则的自定义规则集，并且只考虑您同意的那些规则。

Answer 3

while (true) {
   int bufferLength = is.read(buffer);
   if (bufferLength == -1) 
      break;
   os.write(buffer, 0, bufferLength);
}

Answer 4

也许其他的递归方法可以解决此警告：

private static void writeToInputStream(final InputStream is, final OutputStream os) throws IOException {
    writeToInputStream(is, os, new byte[8388608]); // 8388608 bits = 1024 * 1024 * 8 = 1MB
    os.flush();
}

private static void writeToInputStream(final InputStream is, final OutputStream os, final byte[] dados) throws IOException {
    final int read = is.read(dados, 0, dados.length);
    if (read != -1) {
        os.write(dados, 0, read);
        writeToInputStream(is, os, dados);
    }
}

由于您必须以一定的长度初始化缓冲区，所以我看不到其他方法可以不重复代码或使用两种方法。

java中符合PMD的流副本

4 个答案: