Java FileInputStream FileOutputStream разница в прогоне

FileInputStream read() метод следует этой логике:

Читает байт данных из этого входного потока. Этот метод блокируется, если ввод еще не доступен.

Итак, присвоив значение его возврата переменной, например:

while((len = fis.read())!= -1) 

Избегает того, чтобы байт данных, только что прочитанных из потока, был забыт, поскольку каждый вызов read() будет назначен вашей переменной len.

Вместо этого этот код пропускает один из каждых двух байтов из потока, поскольку read(), выполняемый в условии while, никогда не присваивается переменной. Таким образом, поток продвигается без чтения половины байтов (присвоенных len):

while (fis.read() != -1) {      // reads a byte of data (but not saved)
   int len = fis.read();        // next byte of data saved
   fos.write(len);              // possible -1 written here    
}

@aran и другие уже указали решение вашей проблемы.

Однако у этого есть и другие стороны, поэтому я расширил ваш пример:

import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;

public class FileCopyFisFos {

    public static void main(final String[] args) throws IOException {
        final File src = new File("d:/Test1/OrigFile.MP4");
        final File sink = new File("d:/Test2/DestFile.mp4");

        {
            final long startMS = System.currentTimeMillis();
            final long bytesCopied = copyFileSimple(src, sink);
            System.out.println("Simple copy transferred " + bytesCopied + " bytes in " + (System.currentTimeMillis() - startMS) + "ms");
        }
        {
            final long startMS = System.currentTimeMillis();
            final long bytesCopied = copyFileSimpleFaster(src, sink);
            System.out.println("Simple+Fast copy transferred " + bytesCopied + " bytes in " + (System.currentTimeMillis() - startMS) + "ms");
        }
        {
            final long startMS = System.currentTimeMillis();
            final long bytesCopied = copyFileFast(src, sink);
            System.out.println("Fast copy transferred " + bytesCopied + " bytes in " + (System.currentTimeMillis() - startMS) + "ms");
        }

        System.out.println("Test completed.");
    }

    static public long copyFileSimple(final File pSourceFile, final File pSinkFile) throws IOException {
        try (
                final FileInputStream fis = new FileInputStream(pSourceFile);
                final FileOutputStream fos = new FileOutputStream(pSinkFile);) {

            long totalBytesTransferred = 0;
            while (true) {
                final int readByte = fis.read();
                if (readByte < 0) break;

                fos.write(readByte);
                ++totalBytesTransferred;
            }
            return totalBytesTransferred;
        }
    }

    static public long copyFileSimpleFaster(final File pSourceFile, final File pSinkFile) throws IOException {
        try (
                final FileInputStream fis = new FileInputStream(pSourceFile);
                final FileOutputStream fos = new FileOutputStream(pSinkFile);
                BufferedInputStream bis = new BufferedInputStream(fis);
                BufferedOutputStream bos = new BufferedOutputStream(fos);) {

            long totalBytesTransferred = 0;
            while (true) {
                final int readByte = bis.read();
                if (readByte < 0) break;

                bos.write(readByte);
                ++totalBytesTransferred;
            }
            return totalBytesTransferred;
        }
    }

    static public long copyFileFast(final File pSourceFile, final File pSinkFile) throws IOException {
        try (
                final FileInputStream fis = new FileInputStream(pSourceFile);
                final FileOutputStream fos = new FileOutputStream(pSinkFile);) {

            long totalBytesTransferred = 0;
            final byte[] buffer = new byte[20 * 1024];
            while (true) {
                final int bytesRead = fis.read(buffer);
                if (bytesRead < 0) break;

                fos.write(buffer, 0, bytesRead);
                totalBytesTransferred += bytesRead;
            }
            return totalBytesTransferred;
        }
    }

}

Подсказки, сопровождающие этот код:

• Существует пакет java.nio, который обычно делает эти вещи намного быстрее и с меньшим объемом кода.
• Копирование отдельных байтов в 1 000–40 000 раз медленнее, чем массовое копирование.
• Использование try / resource / catch - лучший способ избежать проблем с зарезервированными / заблокированными ресурсами, такими как файлы и т. Д.
• Если вы решаете что-то довольно банальное, я предлагаю вам поместить это в собственный служебный класс или даже в свою собственную библиотеку.
• Существуют вспомогательные классы, такие как BufferedInputStream и BufferedOutputStream, которые в значительной степени заботятся об эффективности; см. пример copyFileSimpleFaster ().
• Но, как обычно, больше всего влияет на реализацию качество концепции; см. пример copyFileFast ().
• Существуют даже более продвинутые концепции (похожие на java.nio), которые учитывают такие концепции, как поведение кэширования ОС и т. Д., Что придаст производительности еще один удар.

Проверьте мои результаты или запустите его самостоятельно, чтобы увидеть разницу в производительности:

Simple copy transferred 1608799 bytes in 12709ms
Simple+Fast copy transferred 1608799 bytes in 51ms
Fast copy transferred 1608799 bytes in 4ms
Test completed.