java 最大値最小値の表示

Question

前提・実現したいこと

エラーへの対応策が分からないです。

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

Exception in thread "main" 最大値：9  最小値：2
java.lang.ArrayIndexOutOfBoundsException: 3
	at problem5.Problem5_13.main(Problem5_13.java:35)

該当のソースコード

package problem5;

/**
 * 練習問題5-13
 * 
 * ３×３の二次元配列を作成し、それぞれに0から9の乱数を発生させ、実行例のように、その内容と、数値の最大値・最小値を表示させなさい。
 */
public class Problem5_13 {
	/**
	 * メインメソッド メイン処理を実行します。
	 * 
	 * @param args
	 *            プログラム起動引数
	 */
	public static void main(String[] args) {
		// 配列を作成
		int[][] a = new int[3][3];
		// forの二重ループ
		for (int i = 0; i < 3; i++) {
			for (int j = 0; j < 3; j++) {
				// 0~9の乱数を設定
				a[i][j] = (int) Math.floor(Math.random() * 10);
				// 結果の表示
				System.out.print(a[i][j] + "");
			}
			// 改行
			System.out.println();
		}
		int max, min;
		max = a[0][0];
		min = a[0][0];

		for (int i = 1; i < a.length; i++) {
			for (int j = 1; j < a.length; i++) {
				if (a[i][j] > max) {
					max = a[i][j];
					System.out.print("最大値：" + max + "  ");
				}
				if (a[i][j] < min) {
					min = a[i][j];
					System.out.println("最小値：" + min);
				}
			}

		}
	}
}

Answer 1

余談ですが、定数を使用したりするともう少しコードが見やすくなりますよ。

int[][] a = new int[3][3];
// forの二重ループ
for (int i = 0; i < 3; i++) {
    for (int j = 0; j < 3; j++) {
        // 0~9の乱数を設定
        a[i][j] = (int) Math.floor(Math.random() * 10);
        // 結果の表示
        System.out.print(a[i][j] + "");
    }
    // 改行
    System.out.println();
}

↓

final int I_LENGTH = 3;
final int J_LENGTH = 3;
int[][] a = new int[I_LENGTH][J_LENGTH];
// forの二重ループ
for (int i = 0; i < I_LENGTH; i++) {
    for (int j = 0; j < J_LENGTH; j++) {
        // 0~9の乱数を設定
        a[i][j] = (int) Math.floor(Math.random() * 10);
        // 結果の表示
        System.out.print(a[i][j] + "");
    }
    // 改行
    System.out.println();
}

Comments

[aaaajj]

ご回答ありがとうございます！
書き直してみます！

Answer 2

        for (int j = 1; j < a.length; i++) {

j++ しないといけません

で、ほんまは

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

ですね

Comments

[aaaajj]

ご回答ありがとうございます。
エラーは解決できたのですが最大値や最小値が複数出てきたりしてしまうのですが、他の場所に原因があるのでしょうか？

[y_waiwai]

コードを追いかけてみてください。
保持してる値より大きい値が来るとその都度プリントするようになってますよ
最小値も同じ

一度だけにしたいなら、コードの最後にプリントするようにしましょう

[aaaajj]

ありがとうございます！
上手く動きました。