0

特定のファイルを読み取る頻度カウンターを作成して、ファイル内の各文字の頻度(パーセンテージ)を返します。これまでの私のコードはファイルを読み取り、ファイルに出現する各文字の数をリストします。代わりにパーセンテージを生成するために、すべてのカウントを組み合わせる方法がわかりません。以下は私のコードです。コードブロックを正しく使用しなかった場合はご容赦ください。まだこれらすべてを学んでいます。

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

public class FrequencyCounter
{

    public static void main(String[] args )
    {
        char[] capital = { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J','K', 'L', 'M', 'N',
                'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'};

        char[] small =   { 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n',
                'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z' };

        Scanner scan;
        try 
        {
            scan = new Scanner(new File("c://Users//Mikel//Desktop//School Work//CIS407//Week1//mary.txt"));
        } 

        catch (Exception e) 
        {
            System.out.println("File not found");
            return;
        }

        int[] count = new int[26];



        while(scan.hasNextLine()) 
        {
            String line = scan.nextLine();
            System.out.println("Line read: " + line);
            char[] digit = line.toCharArray();


            for(int i = 0; i < digit.length; i++) 
            {
                for(int j = 0; j < 26; j++) 
                {
                    if(digit[i] == capital[j] || digit[i] == small[j]) 
                    {
                        count[j]++;
                        break;
                    }
                }
            }
        }


        for (int i = 0; i < 26; i++)
        {
            System.out.print("  " + capital[i]);
            System.out.println("          " + (count[i]));

        }

    }
}
4

2 に答える 2

1

ch - 'a'小文字のインデックスを返す整数式でありch - 'A'、大文字に対しても同じことを行うことを確認することで、コードを大幅に簡略化できます。したがって、ネストされたループと大文字と小文字の配列は必要ありません。Char.isUpperCase(ch)またはを呼び出すことで、文字が大文字か小文字かを判断できますChar.isLowerCase(ch)

すべてのカウントを合計し、通常の方法でパーセンテージを計算することで、パーセンテージを取得できます。

double pct = (count[i]*100.0) / total;
于 2012-04-08T23:50:35.107 に答える
0

これを行うには多くの方法があります。あなたが指摘したように、次のステップは合計を考え出すことです。これは、2つの配列をループして、合計を別の変数に格納することを意味します。そこから、各カウントを合計で割ってパーセンテージを付けることができます。

つまり、タスクを完了するには、さらに2つのループが必要です。

コードをメソッドに分割することも検討してください。ループごとに1つと言うと、次のようになります。

countLetters();
calculateTotal();
printPercentages();

また、コードを単純化することもできます。0から25にループして等しいかどうかを確認するのではなく、Character.toLowerCase(digit [i])を呼び出して、小文字を取得できます[1つだけ処理する必要があります]。次に、文字から-'a'を引くと、26文字の配列をまったく使用せずにインデックスを取得できます。

于 2012-04-08T23:48:09.643 に答える