int型の2次元配列に入っている数字の発生頻度が知りたい

Question

int型の2次元配列に入っている数字の発生頻度が知りたい

ここに質問の内容を詳しく書いてください。
int型の2次元配列に入っている数字の発生頻度についてウェブサイトで調べてみてもよくわからないし、お手上げ状態です。わかりやすく教えていただけないでしょうか。こうすればよいなどのヒントが欲しいです。

発生している問題・エラーメッセージ

個数の位置がずれたり，0の頻度などすべてが反映されていない。

該当のソースコード

/*
課題053
int型の2次元配列aには，0～4の整数値が格納されている．
各整数値の頻度をint型の配列frequencyに格納せよ．
先頭から，0の頻度，1の頻度，…，4の頻度とすること．
 */
public class Kadai053 {

	public static void printArray(int[] array) {
		for(int i=0; i<array.length; i++)
			System.out.printf("%d : %s\n", i, array[i]);
		System.out.println();
		return;
	}

	public static void main(String[] args) {
		int[][] a = {{2, 1, 1, 0, 2, 2, 3, 0},
				{1, 3, 0, 2, 3, 4, 3, 1},
				{4, 1, 3, 3, 3, 4, 0, 3},
				{0, 4, 4, 4, 2, 0, 4, 3},
				{3, 4, 1, 2, 0, 4, 4, 4},
				{3, 1, 1, 1, 0, 0, 3, 3},
				{3, 1, 4, 1, 4, 1, 4, 1},
				{3, 3, 4, 2, 2, 1, 4, 3}};

		int[] b=new int[5] ;
		int d=0;
		int e=0;
		int s=0;
		int h=0;

		for(int i=0;i<a.length*b.length;i++) {//40

			if(i%8==0) {
				s++;
				e++;
				h=0;
			}
			h=s-1;
			if(i>=8) {
				i=i-8*s;
			}
			if(s>b.length) {
				break;
			}
			h++;
			for(int j=0;j<8;j++) {
				System.out.println((e-1)+"\t"+j);
				if(a[s-1][j]==(0)) {
				d+=1;
				b[s-1]=d/8;
			}
				//if(a[h-1][j]==e-1) {
					//d+=1;
					//b[s-1]=d;
				//}
				}
			if(s>b.length) {
				break;
			}

		}

		int[] frequency=new int[b.length] ;

		for(int i=0;i<frequency.length;i++) {
			frequency[i]=b[i];
		}

		printArray(frequency);

	}

}

試したこと

for文2個で納めれるようにして、出力できるようにした。

Answer 1

正しい答えが出ているのであれば、それでいいと思います。

より短く（単純に）するのであれば、
0から4を順にチェックしていますが、よく考えるとa[i][j]の値がそのままfrequencyのインデックスに対応しています。
なのでこれで十分です。

public class Kadai {
    public static void printArray(int[] array) {
        for (int i = 0; i < array.length; i++)
            System.out.printf("%d : %d\n", i, array[i]);
        System.out.println();
    }

    public static void main(String[] args) {
        int[][] a = { { 2, 1, 1, 0, 2, 2, 3, 0 },
                { 1, 3, 0, 2, 3, 4, 3, 1 },
                { 4, 1, 3, 3, 3, 4, 0, 3 },
                { 0, 4, 4, 4, 2, 0, 4, 3 },
                { 3, 4, 1, 2, 0, 4, 4, 4 },
                { 3, 1, 1, 1, 0, 0, 3, 3 },
                { 3, 1, 4, 1, 4, 1, 4, 1 },
                { 3, 3, 4, 2, 2, 1, 4, 3 } };

        int[] frequency = new int[5];

        for (int i = 0; i < a.length; i++) {
            for (int j = 0; j < a[i].length; j++) {
                frequency[a[i][j]] += 1;
            }
        }

        printArray(frequency);
    }
}

---

コーディング中はそれまでの考えに引きずられて、遠回りなコードになることがありますね。
わたしは、後で見返して「なんでこんな冗長なこと書いていたんだろう？？」となることが、よくあります^^;

Answer 2

public class Kadai {
public static void printArray(int[] array) {
for(int i=0; i<array.length; i++)
System.out.printf("%d : %s\n", i, array[i]);
System.out.println();
return;
}

	public static void main(String[] args) {
		int[][] a = {{2, 1, 1, 0, 2, 2, 3, 0},
				{1, 3, 0, 2, 3, 4, 3, 1},
				{4, 1, 3, 3, 3, 4, 0, 3},
				{0, 4, 4, 4, 2, 0, 4, 3},
				{3, 4, 1, 2, 0, 4, 4, 4},
				{3, 1, 1, 1, 0, 0, 3, 3},
				{3, 1, 4, 1, 4, 1, 4, 1},
				{3, 3, 4, 2, 2, 1, 4, 3}};

		int[] frequency=new int[5] ;
		int k=0;

		for(int fre=0;fre<frequency.length;fre++) {
			k=0;
			for(int i=0;i<a.length;i++) {//tate
				for(int j=0;j<a[i].length;j++) {//yoko
					if(a[i][j]==fre) {
						k+=1;
						frequency[fre]=k;
				}

				}
			}

		}



		printArray(frequency);

	}

}

出力結果
0 : 9
1 : 14
2 : 8
3 : 17
4 : 16

自己解決することができました。自分に甘えてしまい、丸投げしてしまいました。大変申し訳ございません。
自分で考えて理解しないと自分のためにもならないので回答してくださらなかったみなさんありがとうございます。もし、このようにするともっと短い文で表せるなどがありましたらお願します。