java equalsメソッドによる文字列比較

Question

初めまして、javaで文字列クラスの配列をlist構造に変換し、Collections.sortで並び換えをを行い、配列に戻して、equalsメソッドで比較し、文字列を数えるプログラムを作成していますが、うまくいきません。プログラムは、まだ日が経っておらず、見苦しいですので、部分的にしか見せる事ができませんが、宜しくお願いします。

import java.util.List;
import java.util.Arrays;
import java.util.ArrayList;
import java.util.Collections;

public class Sample9 {

    public static void main(String args[]) {
	String[] middlestr;
	String[] middlestr2;
	int[] strnum;
	List<String> list = new ArrayList<String>()
try {

            FileReader fr = new FileReader("test1.txt");
            BufferedReader br = new BufferedReader(fr);

            String oldline;
	    String readline;
	    String[] newline;
	    newline = new String[200];
            middlestr = new String[100];
	    middlestr2 = new String[100];
	    strnum = new int[100];
	    readline = "私";
	    int i = 0;
	    int count2 = 0;
	    int count3 =0;
	    int c = 0;
	    int h = 0;
            while ((oldline = br.readLine()) != null) {
            newline[i] = oldline;
	    i++
            }
            for(int l = 0; l<i; l++){
                System.out.println(newline[l]);

	    for(int a = 0; a<i; a++){
	    if(readline.equals(newline[a])){
	        count2++;
	    }
	}
     for(int b = 0; b<i; b++){
	if(readline.equals(newline[b])){
		middlestr[ c ] = newline[ b + 1 ];
		c++;
		}
	}
	for(int e = 0; e<count2; e++)
	        list.add(middlestr[e]);

	Collections.sort(list);
	String[] array = (String[])list.toArray(new String[0]);

//この部分から怪しい。
	for(int g = 1; g<count2; g++){
	if(array[g - 1].equals(array[g])){
		count3++;
	}else if(!array[g - 1].equals(array[g])){
		count3++;
	middlestr2[h] = array[g - 1];
	strnum[h] = count3;
		count3 = 0;
		h++;
	}else if (g<=count2){
	h++;
	middlestr2[h] = array[count2];
	strnum[h] = count3;
}
}
	for(int j = 0; j<=h; j++){
	System.out.println(middlestr2[j]);
	System.out.println(strnum[j]);
	}

この、プログラムの場合、配列arrayの各要素に、「が」が一つ、「は」が５つ、「はだれですか」が２つの場合、出力結果は、
が
１
は
５
null
0
となります。私は、「はだれですか」が2個というところまで出力したいです。
宜しくお願いします。

Comments

[swordone]

......ファイルの読み込みと、newlineに文字列を代入。......←これは，コード化はしていないけどこういう処理が入る，という意味ですか？

[hikarikaikun]

swordoneさんの三番目のご指摘通り、if文の条件を修正したところ、middlestr2[j]がnullになることは無くなりました。しかし、新しい問題が出現したので、もう一度自分で考えてみます。皆さん有難うございました。

Answer 1

明らかにおかしいのは，

    for(int a = 0; a<i; a++){
        if(readline.equals(newline[a])){
            count2++;
        }
    }
    for(int b = 0; b<i; b++){
        if(readline.equals(newline[b])){
            middlestr[ c ] = newline[ b + 1 ];
            c++;
        }
    }

この2つのfor文中のifは使っている変数が違うだけで条件は全く同じ．2回やる意味が分からない．

    for(int e = 0; e<count2; e++)
            list.add(middlestr[e]);

一旦配列にしたものをArrayListにするなら，最初からArrayListに入れれば良いのでは？

    for(int g = 1; g<count2; g++){

        //分岐１
        if(array[g - 1].equals(array[g])){
            count3++;

        //分岐２
        }else if(!array[g - 1].equals(array[g])){
            count3++;
            middlestr2[h] = array[g - 1];
            strnum[h] = count3;
            count3 = 0;
            h++;

        //分岐３
        }else if (g<=count2){
            h++;
            middlestr2[h] = array[count2];
            strnum[h] = count3;
        }
    }

分岐１に入れなかった場合(array[g - 1].equals(array[g])がfalseの場合)，必ず分岐２に入るため，分岐３が無意味なデッドコードになっている．

Answer 2

swordoneさんも書かれていますが、配列とListをわざわざ入れ替えなくても良いのでは？
配列はjava.util.Arrays.sortというメソッドでソートすることができます。
どちらかに揃えたほうが簡潔になって全体を把握しやすくなると思いますよ。

Comments

[hikarikaikun]

Arrays.sortメソッドは以前試してみましたが、上手くいかず使用を断念しました。ご指摘ありがとうございます。

Answer 3

※ザックリでしか拝見していないのですが、【 h++; 】の位置が不自然のように感じました。

//分岐３
}else if (g<=count2){
// h++; //ここでｈを加算してしまうと、一つ後ろの配列に入れる事になるのでは？
middlestr2[h] = array[count2];
strnum[h] = count3;
h++; //ここでｈを加算するのが正しいと思われます。
}