java - Q re Java 程序，用于从文本文件读取、创建数组列表、重新格式化并输出到另一个文件

Question

作为一名 Java 学生(初学者)，我刚刚完成了编写一个程序来执行以下任务的任务:

1. 读取文本文件的内容；

2. 找出文本中的所有单词，将所有大写字母替换为小写字母；

3. 生成文本中出现的单词的字典(即按字母顺序排列的单词列表，每行一个单词，不重复)；
4. 将字典输出到另一个文件。

我们被建议有一个单独的类包含我们的主要方法，并使用 ArrayList、Scanner、PrintWriter 和 FileReader 类。

我成功地完成了下面的工作，但它在我的知识中提出了一些问题/根本性的差距。

我相信基于好的 O.O. 原则。编程实践，我应该将下面分解成一系列具有单一目的的方法，然后调用这些方法。但是，我真的很难做到这一点。任何人都可以详细说明/建议改进该特定点的方法吗？或者如果我错了请纠正我？

public class Main
{

    private static ArrayList<String> dictionary; 
    private static ArrayList<String> tempdictionary;

    public static void main() throws FileNotFoundException
    {
        Scanner inFile = new Scanner(new FileReader("H:\\csc8001\\results.txt"));
        dictionary = new ArrayList <>();
        tempdictionary = new ArrayList <>();

        while (inFile.hasNext()) { 
            dictionary.add(inFile.next().toLowerCase().replaceAll("[^a-zA-Z ]", ""));
        }
        inFile.close();

        PrintWriter outFile = new PrintWriter("H:\\csc8001\\results1.txt");
        int index = 0;
        for (String a : dictionary)
        {
            if (!tempdictionary.contains(a)){ 
                tempdictionary.add(a);
            } else {
                index++;
            }
        }

        Collections.sort(tempdictionary);
        for (int i=0; i<tempdictionary.size(); i++)
            outFile.println(tempdictionary.get(i));
        outFile.flush();
        outFile.close();
    }

}

Answer 1

正如其他人所说，您的代码绝对没有任何问题，因为它代表您被赋予的任务。但是您表达了对 OO 原则的兴趣，所以我有一些可能有用的想法。注意 - 下面是非常开放的批评，它对特定任务采取了过于笼统的方法 - 有一个 perennial debate关于这一点，许多人订阅了 "You ain't gonna need it"学校。

为什么面向对象的方法是合适的？

如果您可能需要同一类的多个副本，但具有不同的属性值，则面向对象的方法将很有用。在您指定的任务中，情况并非如此。

但是，可以想象，您可能想要在后续任务中从(一个)其他文件制作字典，或者在同一程序中使用多个文件编写一个程序。

在这种情况下什么构成对象？

在这个任务中，你被要求创建一个字典。字典可以是一个对象。您可以从这个起点着手完成任务。例如:

public class Dictionary
{
}

字典有什么用？按字母顺序排序的单词列表。正如其他人所说，一个不错的选择是为此使用 SortedSet (习惯于查看 API documentation a.k.a javadocs ，它们是您必不可少的 friend )，但是您被告知使用 ArrayList，这就是我们要做的。

public class Dictionary
{
    ArrayList<String> words;
}

您可以继续添加方法来完成每项任务。就个人而言，我可能有一个采用 String 文件名的构造函数。我已经将文件的读取直接放入其中 - 您同样可以调用一个私有(private)方法(例如 parseFile())来获取文件并提取单词。

public Dictionary(String inputFilename) throws FileNotFoundException 
{
    Scanner inFile = new Scanner(new FileReader(inputFilename));
    words = new ArrayList <String>();

    while (inFile.hasNext()) { 
        words.add(inFile.next().toLowerCase().replaceAll("[^a-zA-Z ]", ""));
    }
    inFile.close();

    removeDuplicates();

    Collections.sort(words);
}

然后我会添加一个 removeDuplicates() 方法(我在上面调用过)——有多种方法可以做到这一点。您可以像您所做的那样使用临时 ArrayList - 我将给出另一个我怀疑更快的示例。

private void removeDuplicates()
{
    HashSet<String> dupSet = new HashSet<String>();
    dupSet.addAll(words);
    words.clear();
    words.addAll(dupSet);
}

最后 - 添加一个 writeToFile(String outFilename) 方法

public void writeToFile(String outFilename) throws FileNotFoundException
{
    PrintWriter outFile = new PrintWriter(outFilename);
    for (int i=0; i<words.size(); i++)
        outFile.println(words.get(i));
    outFile.flush(); // strictly redundant
    outFile.close();
}

你被告知要将 main 放在你的类中，那么就这样做吧。现在可以很短了:

    public static void main(String[] args) throws FileNotFoundException
    {
         d = new Dictionary("H:\\csc8001\\results.txt");
         d.outputToFile("H:\\csc8001\\results1.txt");
    }

你收获了什么？

您现在有一个 Dictionary 类，您可以将其导入任何项目并按上述方式使用。如果下周的作业是“修改你的词典程序，为这个目录中的所有 100 个文件制作一个词典”，你就笑了。您还可以向 Dictionary 类添加函数(例如 mergeWith(Dictionary d2))，同时相当有信心您不会破坏现有函数或任何使用这些函数的程序。

你失去了什么？

阅读此答案需要 15 分钟。是时候编写一些可能永远不会使用的方法了。您的原始程序非常简洁，本质上是一个脚本。

小风格的东西

将您的类称为 Main 是很不寻常的。这没什么问题，但即使您没有采用上面的字典对象方法，我也会重命名它 - 只要其中包含 main 方法，它仍然可以运行。

你真的不需要 tempdictionary - 你可以只“唯一化”并排序 dictionary - 从而避免保留未排序的列表和重复项(以及它使用的相关内存)

初始化您的 PrintWriter，然后进行一些处理(使列表唯一并排序)，然后输出看起来有点不寻常。尽量在输出位置附近打开文件，然后尽快再次关闭它。