Question

我想使用Base64.java来编码和解码文件。 Encode.wrap(InputStream)和decode.wrap(InputStream)工作但运行缓慢。所以我使用了以下代码。

public static void decodeFile(String inputFileName,
        String outputFileName)
        throws FileNotFoundException, IOException {

    Base64.Decoder decoder = Base64.getDecoder();
    InputStream in = new FileInputStream(inputFileName);
    OutputStream out = new FileOutputStream(outputFileName);

    byte[] inBuff = new byte[BUFF_SIZE];  //final int BUFF_SIZE = 1024;
    byte[] outBuff = null;
    while (in.read(inBuff) > 0) {
        outBuff = decoder.decode(inBuff);
        out.write(outBuff);
    }
    out.flush();
    out.close();
    in.close();
}

然而，它总是抛出

Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: Input byte array has wrong 4-byte ending unit
    at java.util.Base64$Decoder.decode0(Base64.java:704)
    at java.util.Base64$Decoder.decode(Base64.java:526)
    at Base64Coder.JavaBase64FileCoder.decodeFile(JavaBase64FileCoder.java:69)
    ...

将final int BUFF_SIZE = 1024;更改为final int BUFF_SIZE = 3*1024;后，代码才有效。由于“BUFF_SIZE”也用于编码文件，我认为编码的文件有问题（1024％3 = 1，这意味着在文件中间添加了填充）。

另外，正如@Jon Skeet和@Tagir Valeev所提到的，我不应该忽略InputStream.read()的返回值。所以，我修改了下面的代码。

（但是，我必须提到代码的运行速度比使用wrap()要快得多。我注意到了速度差异，因为我在jdk8之前很久就编码并集中使用了Base64.encodeFile（）/ decodeFile（）现在，我的buffed jdk8代码运行速度和原始代码一样快。所以，我不知道wrap()发生了什么......）

public static void decodeFile(String inputFileName,
        String outputFileName)
        throws FileNotFoundException, IOException
{

    Base64.Decoder decoder = Base64.getDecoder();
    InputStream in = new FileInputStream(inputFileName);
    OutputStream out = new FileOutputStream(outputFileName);

    byte[] inBuff = new byte[BUFF_SIZE];
    byte[] outBuff = null;
    int bytesRead = 0;
    while (true)
    {
        bytesRead = in.read(inBuff);
        if (bytesRead == BUFF_SIZE)
        {
            outBuff = decoder.decode(inBuff);
        }
        else if (bytesRead > 0)
        {
            byte[] tempBuff = new byte[bytesRead];
            System.arraycopy(inBuff, 0, tempBuff, 0, bytesRead);
            outBuff = decoder.decode(tempBuff);
        }
        else
        {
            out.flush();
            out.close();
            in.close();
            return;
        }
        out.write(outBuff);
    }
}

特别感谢@Jon Skeet和@Tagir Valeev。

Answer 1

我强烈怀疑问题在于您忽略了InputStream.read的返回值，而不是检查流的结尾。所以这个：

while (in.read(inBuff) > 0) {
    // This always decodes the *complete* buffer
    outBuff = decoder.decode(inBuff);
    out.write(outBuff);
}

应该是

int bytesRead;
while ((bytesRead = in.read(inBuff)) > 0) {
    outBuff = decoder.decode(inBuff, 0, bytesRead);
    out.write(outBuff);
}

我不会期待这比使用wrap更快。

Answer 2

尝试使用decode.wrap(new BufferedInputStream(new FileInputStream(inputFileName)))。通过缓冲，它应该至少与手动制作的版本一样快。

至于为什么你的代码不起作用：那是因为最后一个块可能短于1024个字节，但你尝试解码整个byte[]数组。有关详细信息，请参阅@JonSkeet答案。

Answer 3

好吧，我换了

＆＃34; final int BUFF_SIZE = 1024;＆＃34;

进入

＆＃34; final int BUFF_SIZE = 1024 * 3;＆＃34;

有效！

所以，我猜可能填充有问题...我的意思是，在编码文件时，（因为1024％3 = 1）必须有填充。这些可能会在解码时引发问题...

Answer 4

您应该记录您已阅读的字节数，此外，
你应该确定你的缓冲区大小可以被3整除，因为在Base64中，每3个字节有4个输出（64是2 ^ 6,3 * 8等于4 * 6），这样做可以避免填充问题。（这样你的输出就没有错误的结尾＆＃34; =＆＃34;）

使用Java8 Base64解码器的IllegalArgumentException

4 个答案: