Блог

Главная - Вопросы по программированию - Почему моя процедура копирования потока неуклонно снижает производительность

Почему моя процедура копирования потока неуклонно снижает производительность

#java #java-io

#java #java-io

Вопрос:

Этот код работает все медленнее и медленнее при копировании больших файлов. Я делаю что-то не так?

     InputStream ms2 = new BufferedInputStream(new FileInputStream("/home/fedd/Videos/homevid.mp4"));
    OutputStream fos2 = new BufferedOutputStream(new FileOutputStream("testfile2.mp4", true));

    try {
        int byt;
        int i = 0;
        long time = System.currentTimeMillis();
        while ((byt = ms2.read()) != -1) {
            fos2.write(byt);
            i  ;
            if (i > 100000) {
                i = 0;
                long took = System.currentTimeMillis() - time;
                System.out.println("100000 bytes took "   took   " milliseconds which means "   (100000000 / took)   " bytes per second");
            }
        }
        fos2.close();
        ms2.close();
    } catch (Exception e) {
        throw new RuntimeException(e);
    }

Моя Java:

openjdk 10.0.2 2018-07-17 Среда выполнения OpenJDK (сборка 10.0.2 13-Ubuntu-1ubuntu0.18.04.4)

64-разрядная серверная виртуальная машина OpenJDK (сборка 10.0.2 13-Ubuntu-1ubuntu0.18.04.4, смешанный режим)

Комментарии:

1. Что такое ms2 (ms2.read())? Проверьте свой оператор while, я думаю, что здесь опечатка.

2. Вы измеряете время с самого начала цикла. Вам нужно сбросить временную метку начала после печати: time = System.currentTimeMillis(); .

3. Кстати: для большинства дисков / SSD довольно нормально, что производительность значительно снижается через некоторое время. Это связано с кэшированием записи либо на уровне ядра, либо даже на аппаратном уровне. Поэтому я бы также просто проверил, насколько быстро вы можете скопировать файл, используя другие инструменты…

4. Могу ли я удалить свой вопрос? 🙂

5. Вам следует использовать инструкцию try with resources (см. docs.oracle.com/javase/tutorial/essential/exceptions/… ) или закройте потоки в блоке finally, иначе это приведет к возможной утечке памяти из-за открытых потоков.

Ответ №1:

Вам нужно сбрасывать базовое «время» после каждого сравнения. Попробуйте использовать это:

 if (i > 100000) {
    i = 0;
    long took = System.currentTimeMillis() - time;
    time = System.currentTimeMillis();
    System.out.println("100000 bytes took "   took   " milliseconds which means "   (100000000 / took)   " bytes per second");
}

Ответ №2:

Ваша производительность снижается, потому что ваш расчет неверен. Для второго фрагмента вы вычисляете байты в секунду исходя из размера второго фрагмента, но из времени обоих фрагментов. Попробуйте добавить time = System.currentTimeMillis(); после long took = ...

Комментарии:

1. Вы имеете в виду time , не took , но, черт возьми, да