我一直在尝试提升我的 Java 技能以更多地使用 Java 5 和 Java 6。我一直在做一些编程练习。我被要求从一个文本文件中读取一个段落并输出一个排序(降序)的单词列表并输出每个单词的计数。

我的代码如下。

我的问题是:

1. 我的文件输入例程是最尊重 JVM 资源的例程吗？

2. 是否可以减少读取文件内容并将内容放入可以生成排序单词列表的集合中的步骤？

3. 我是否以最有效的方式使用了集合类和接口(interface)？

非常感谢任何意见。我只是想找点乐子并提高我的编程技能。

import java.io.*;
import  java.util.*;

public class Sort
{
    public static void main(String[] args)
    {
        String   sUnsorted       = null;
        String[] saSplit         = null;

        int iCurrentWordCount    = 1;
        String currentword       = null;
        String pastword          = "";

        // Read the text file into a string
        sUnsorted = readIn("input1.txt");

        // Parse the String by white space into String array of single words
        saSplit   = sUnsorted.split("\\s+");

        // Sort the String array in descending order
        java.util.Arrays.sort(saSplit, Collections.reverseOrder());


        // Count the occurences of each word in the String array
        for (int i = 0; i < saSplit.length; i++ )
        {

            currentword = saSplit[i];

            // If this word was seen before, increase the count & print the
            // word to stdout
            if ( currentword.equals(pastword) )
            {
                iCurrentWordCount ++;
                System.out.println(currentword);
            }
            // Output the count of the LAST word to stdout,
            // Reset our counter
            else if (!currentword.equals(pastword))
            {

                if ( !pastword.equals("") )
                {

                    System.out.println("Word Count for " + pastword + ": " + iCurrentWordCount);

                }


                System.out.println(currentword );
                iCurrentWordCount = 1;

            }

            pastword = currentword;  
        }// end for loop

       // Print out the count for the last word processed
       System.out.println("Word Count for " + currentword + ": " + iCurrentWordCount);



    }// end funciton main()


    // Read The Input File Into A String      
    public static String readIn(String infile)
    {
        String result = " ";

        try
        {
            FileInputStream file = new FileInputStream (infile);
            DataInputStream in   = new DataInputStream (file);
            byte[] b             = new byte[ in.available() ];

            in.readFully (b);
            in.close ();

            result = new String (b, 0, b.length, "US-ASCII");

        }
        catch ( Exception e )
        {
            e.printStackTrace();
        }

        return result;
    }// end funciton readIn()

}// end class Sort()

/////////////////////////////////////////////////
//  Updated Copy 1, Based On The Useful Comments
//////////////////////////////////////////////////

import java.io.*;
import java.util.*;

public class Sort2
{
    public static void main(String[] args) throws Exception
    {
        // Scanner will tokenize on white space, like we need
        Scanner scanner               = new Scanner(new FileInputStream("input1.txt"));
        ArrayList <String> wordlist   = new  ArrayList<String>();
        String currentword            = null;   
        String pastword               = null;
        int iCurrentWordCount         = 1;       

        while (scanner.hasNext())
            wordlist.add(scanner.next() );

        // Sort in descending natural order
        Collections.sort(wordlist);
        Collections.reverse(wordlist);

        for ( String temp : wordlist )
        {
            currentword = temp;

            // If this word was seen before, increase the count & print the
            // word to stdout
            if ( currentword.equals(pastword) )
            {
                iCurrentWordCount ++;
                System.out.println(currentword);
            }
            // Output the count of the LAST word to stdout,
            // Reset our counter
            else //if (!currentword.equals(pastword))
            {
                if ( pastword != null )
                    System.out.println("Count for " + pastword + ": " +  
                                                            CurrentWordCount);   

                System.out.println(currentword );
                iCurrentWordCount = 1;    
            }

            pastword = currentword;  
        }// end for loop

        System.out.println("Count for " + currentword + ": " + iCurrentWordCount);

    }// end funciton main()


}// end class Sort2

最佳答案

1. 在 Java 中有更多惯用的方式来读取文件中的所有单词。 BreakIterator是一种从输入中读取单词的更好方法。

2. 使用 List<String>而不是 Array在几乎所有情况下。数组在技术上不是 Collection API 的一部分并且不像 List 那样容易替换实现, Set和 Map是。

3. 你应该使用 Map<String,AtomicInteger>做你的字数统计而不是走Array一遍又一遍。 AtomicInteger与 Integer 不同，它是可变的所以你可以incrementAndGet()在恰好是线程安全的单个操作中。 SortedMap实现也会按顺序为您提供单词及其计数。

4. Make as many variables, even local ones final as possible.并在您使用它们之前声明它们，而不是在它们的预期范围会丢失的顶部。

5. 你应该几乎总是使用 BufferedReader或 BufferedStream在执行磁盘 IO 时，缓冲区大小等于磁盘 block 大小的倍数。

就是说，在您拥有“正确”行为之前，不要担心微优化。

关于java - 更高效还是更现代？使用 Java 读取和排序文本文件，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/6268632/