java - 多线程运行速度比单进程慢

标签 java multithreading java.util.scanner bufferedreader

在学校的一项作业中,我被要求创建一个简单的程序来创建 1000 个文本文件,每个文件的行数是随机的,通过多线程\单进程计算有多少行。而不是删除这些文件。

现在在测试过程中发生了一件奇怪的事情——对所有文件进行线性计数总是比以多线程方式对它们进行计数要快一点,这在我的类里面引发了相当多的学术理论讨论。

当使用 Scanner 读取所有文件时,一切都按预期工作 - 以大约 500 毫秒的线性时间和 400 毫秒的线程时间读取 1000 个文件

然而,当我使用 BufferedReader 时,线性时间下降到大约 110 毫秒,线程时间下降到 130 毫秒。

代码的哪一部分导致了这个瓶颈,为什么?

编辑:澄清一下,我不是在问为什么 ScannerBufferedReader 慢。

完整的可编译代码:(尽管您应该更改文件创建路径输出)

import java.io.*;
import java.util.Random;
import java.util.Scanner;

/**
 * Builds text files with random amount of lines and counts them with 
 * one process or multi-threading.
 * @author Hazir
 */// CLASS MATALA_4A START:
public class Matala_4A {

    /* Finals: */
    private static final String MSG = "Hello World";

    /* Privates: */
    private static int count;
    private static Random rand;

    /* Private Methods: */ /**
     * Increases the random generator.
     * @return The new random value.
     */
    private static synchronized int getRand() {
        return rand.nextInt(1000);
    }

    /**
     * Increments the lines-read counter by a value.
     * @param val The amount to be incremented by.
     */
    private static synchronized void incrementCount(int val) {
        count+=val;
    }

    /**
     * Sets lines-read counter to 0 and Initializes random generator 
     * by the seed - 123.
     */
    private static void Initialize() {
        count=0;
        rand = new Random(123);
    }

    /* Public Methods: */ /**
     * Creates n files with random amount of lines.
     * @param n The amount of files to be created.
     * @return String array with all the file paths.
     */
    public static String[] createFiles(int n) {
        String[] array = new String[n];
        for (int i=0; i<n; i++) {
            array[i] = String.format("C:\\Files\\File_%d.txt", i+1);
            try (   // Try with Resources: 
                    FileWriter fw = new FileWriter(array[i]); 
                    PrintWriter pw = new PrintWriter(fw);
                    ) {
                int numLines = getRand();
                for (int j=0; j<numLines; j++) pw.println(MSG);
            } catch (IOException ex) {
                System.err.println(String.format("Failed Writing to file: %s", 
                        array[i]));
            }
        }
        return array;
    }

    /**
     * Deletes all the files who's file paths are specified 
     * in the fileNames array.
     * @param fileNames The files to be deleted.
     */
    public static void deleteFiles(String[] fileNames) {
        for (String fileName : fileNames) {
            File file = new File(fileName);
            if (file.exists()) {
                file.delete();
            }
        }
    }

    /**
     * Creates numFiles amount of files.<br>
     * Counts how many lines are in all the files via Multi-threading.<br>
     * Deletes all the files when finished.
     * @param numFiles The amount of files to be created.
     */
    public static void countLinesThread(int numFiles) {
        Initialize();
        /* Create Files */
        String[] fileNames = createFiles(numFiles);
        Thread[] running = new Thread[numFiles];
        int k=0;
        long start = System.currentTimeMillis();
        /* Start all threads */
        for (String fileName : fileNames) {
            LineCounter thread = new LineCounter(fileName);
            running[k++] = thread;
            thread.start();
        }
        /* Join all threads */
        for (Thread thread : running) {
            try {
                thread.join();
            } catch (InterruptedException e) {
                // Shouldn't happen.
            }
        }
        long end = System.currentTimeMillis();
        System.out.println(String.format("threads time = %d ms, lines = %d",
                end-start,count));
        /* Delete all files */
        deleteFiles(fileNames);
    }

    @SuppressWarnings("CallToThreadRun")
    /**
     * Creates numFiles amount of files.<br>
     * Counts how many lines are in all the files in one process.<br>
     * Deletes all the files when finished.
     * @param numFiles The amount of files to be created. 
     */
    public static void countLinesOneProcess(int numFiles) {
        Initialize();
        /* Create Files */
        String[] fileNames = createFiles(numFiles);
        /* Iterate Files*/
        long start = System.currentTimeMillis();
        LineCounter thread;
        for (String fileName : fileNames) {
            thread = new LineCounter(fileName);
            thread.run(); // same process
        }
        long end = System.currentTimeMillis();
        System.out.println(String.format("linear time = %d ms, lines = %d",
                end-start,count));
        /* Delete all files */
        deleteFiles(fileNames);
    }

    public static void main(String[] args) {
        int num = 1000;
        countLinesThread(num);
        countLinesOneProcess(num);
    }

    /**
     * Auxiliary class designed to count the amount of lines in a text file.
     */// NESTED CLASS LINECOUNTER START:
    private static class LineCounter extends Thread {

        /* Privates: */
        private String fileName;

        /* Constructor: */
        private LineCounter(String fileName) {
            this.fileName=fileName;
        }

        /* Methods: */

        /**
         * Reads a file and counts the amount of lines it has.
         */ @Override
        public void run() {
            int count=0;
            try ( // Try with Resources:
                    FileReader fr = new FileReader(fileName);
                    //Scanner sc = new Scanner(fr);
                    BufferedReader br = new BufferedReader(fr);
                    ) {
                String str;
                for (str=br.readLine(); str!=null; str=br.readLine()) count++;
                //for (; sc.hasNext(); sc.nextLine()) count++;
                incrementCount(count);
            } catch (IOException e) {
                System.err.println(String.format("Failed Reading from file: %s", 
                fileName));            
            }
        }
    } // NESTED CLASS LINECOUNTER END;
} // CLASS MATALA_4A END;

最佳答案

瓶颈是磁盘。

您每次只能使用一个线程访问磁盘,因此使用多个线程无济于事,线程切换所需的超时会降低您的全局性能。

仅当您需要拆分等待不同来源(例如网络和磁盘,或两个不同的磁盘,或许多网络流)上的长时间 I/O 操作的工作,或者如果您有 cpu 密集型操作时,使用多线程才有意义可以在不同的核心之间拆分。

请记住,对于一个好的多线程程序,您需要始终考虑:

  • 在线程之间切换上下文时间
  • 长 I/O 操作可以并行或不并行
  • 是否存在用于计算的密集 CPU 时间
  • cpu 计算是否可以拆分为子问题
  • 线程间共享数据的复杂性(信号量或同步)
  • 与单线程应用程序相比,难以读取、编写和管理多线程代码

关于java - 多线程运行速度比单进程慢,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/34589623/

相关文章:

java - JBOSS 上的此 URL 不支持 HTTP 方法 POST

java - GC和YGC的发生以及下一步的行动当然

java - 具有 Synthetica Aluoxy 外观和感觉的 jxdatepicker 无法正常工作

c - 使用条件变量多线程时出现意外行为

multithreading - 多线程 DBMS?

java - 私有(private)类型..错误

java - 我的代码中的DataSnapshot会因为多线程而被覆盖吗

java - 如何获得从扫描仪输入到阵列的所有排列?

java - 扫描仪在使用 next() 或 nextFoo() 后跳过 nextLine()?

java - 在无法解决和MasterMind