纠正和压缩 Java 程序

Question

我想我几乎已经搞定了我的 java 程序。它旨在通过使用 10 个不同的线程读取文本文件并找到最大的整数。我收到了这个错误：

Error:(1, 8) java: class Worker is public, should be declared in a file named Worker.java

我觉得我的代码可能比它需要的更复杂，所以我试图弄清楚如何缩小它的大小，同时修复上面的错误。非常感谢您在此问题上提供的任何帮助，如果我能澄清任何事情，请告诉我。另外，“工人”类是否必须是一个单独的文件？我将它添加到同一个文件中，但出现上述错误。

import java.io.BufferedReader;
import java.io.FileReader;

public class datafile {

    public static void main(String[] args) {
        int[] array = new int[100000];
        int count;
        int index = 0;
        String datafile = "dataset529.txt"; //string which contains datafile
        String line; //current line of text file

        try (BufferedReader br = new BufferedReader(new FileReader(datafile))) { //reads in the datafile
            while ((line = br.readLine()) != null) { //reads through each line
                array[index++] = Integer.parseInt(line); //pulls out the number of each line and puts it in numbers[]
            }
        }



            Thread[] threads = new Thread[10];
            worker[] workers = new worker[10];


            int range = array.length / 10;
            for (count = 0; count < 10; count++) {
                int startAt = count * range;
                int endAt = startAt + range;
                workers[count] = new worker(startAt, endAt, array);

            }

            for (count = 0; count < 10; count++) {
                threads[count] = new Thread(workers[count]);
                threads[count].start();
            }

            boolean isProcessing = false;
            do {
                isProcessing = false;
                for (Thread t : threads) {
                    if (t.isAlive()) {
                        isProcessing = true;
                        break;
                    }
                }
            } while (isProcessing);

            for (worker worker : workers) {
                System.out.println("Max = " + worker.getMax());
            }
        }


    }


     public  class worker implements Runnable {

        private int startAt;
        private int endAt;
        private int randomNumbers[];

        int max = Integer.MIN_VALUE;

        public worker(int startAt, int endAt, int[] randomNumbers) {
            this.startAt = startAt;
            this.endAt = endAt;
            this.randomNumbers = randomNumbers;
        }

        @Override
        public void run() {
            for (int index = startAt; index < endAt; index++) {

                if (randomNumbers != null && randomNumbers[index] > max)
                    max = randomNumbers[index];
            }
        }

        public int getMax() {
            return max;
        }

    }

Answer 1

我已经写了一些 cmets，但我会将它们全部收集到一个答案中，以便将来任何人都可以看到汇总信息：

在 readtextfile 类的源代码末尾（根据 java 命名约定应该是 ReadTextile），你有太多右大括号，

    } while (isProcessing);

    for (Worker worker : workers) {
        System.out.println("Max = " + worker.getMax());
    }
}

}
}
        }

上面的内容应该在碰到最左边一列的第一个大括号上结束。在创建任何 Java 类时，这是一个很好的经验法则，如果您有多个最左大括号，或者您的最后一个大括号不是最左括号，那么您可能在某个地方犯了错误，应该检查您的大括号。

至于您的文件问题您应该将所有类命名为Java conventions，并且每个类都应该存储在一个名为 ClassName.java 的文件中（区分大小写）。例如：

public class ReadTextFile应该存储在ReadTextFile.java中

你也可以让Worker 成为一个内部类。为此，您几乎可以将源代码复制到ReadTextFile 类中（确保它在main 方法之外）。请参阅this 有关内部类的教程以快速了解。

至于你的其他问题Code Review SE 是问这个问题的合适地方，那里的聪明人可能会提供比我更好的答案。不过，我还建议使用 10 个线程可能不是在文本文件中找到最大 int 的最有效方法（无论是在开发时间还是在执行时间上）。