分からないエラーが出てしまいます

Question

初心者です。　よろしくお願いします。
以下のソースを実行すると、
クラス数:3

1組の人数3
1組1番の国語の点数22
Exception in thread "main" java.lang.NullPointerException
at M6_19.main(M6_19.java:25)

と出力されます。
自分としましては、
クラス数:3

1組の人数3
1組1番の国語の点数22
1組1番の数学の点数68
1組1番の社会の点数57
…
2組1番の国語の点数41
としたいのですが、どこでエラーとなってしまうのか、わかりません。

import java.util.Scanner;
class M6_19{
  public static void main(String[] args){
    Scanner stdIn = new Scanner(System.in);
    int n,i,z,p,q;
    int u=0;
    int j;
    int sum=0;
    int total=0;
    int ninzu=0;
    String[] s = {"国語","数学","社会","理科","英語"};
    
    System.out.print("クラス数:");
    n=stdIn.nextInt();
    int[][][] a = new int[n][][];
    
    for(i=0;i<n;i++){
      System.out.printf("\n%d組の人数",i+1);
      u = stdIn.nextInt();
      a[i] = new int[u][];//a[0][]は一組の点数の箱
      ninzu += u;
      for(z=0;z<a[i].length;z++){
        for(q=0;q<s.length;q++){
          System.out.printf("%d組%d番の"+s[q]+"の点数",i+1,z+1);
          a[i][z][q]=stdIn.nextInt();
          a[i][z] = new int[q];
        }
      }
    }
    
    /*
    System.out.println("  組 |    合計　　平均");
    System.out.println("-----+---------------");
    //合計
    for(i=0;i<a.length;i++){
    for(z=0;z<a[i].length;z++){
    for(q=0;q<s.length;q++){
    sum += a[i][z][q];
  }
}
total += sum;
System.out.printf("%2d組|%7d%7.1f\n",i+1,sum,(double)sum/u);
sum=0;
}
System.out.println("-----+---------------");
System.out.printf("  計 |%7d%7.1f\n",total,(double)total/ninzu);
*/
}
}
```ここに言語を入力
コード

なお、コメントアウトしてあるところは改良前のものですので、気になさらないでください。

Answer 1

とりあえず、エラーメッセージから25行目でエラーを吐いているのが分かるはずです。
その行の前後に着目しましょう。

a[i][z][q]=stdIn.nextInt(); // 25行目
a[i][z] = new int[q];

ちらと見て、『なんで領域確保する前に参照してるの？』と慣れている人なら感じます。
まずは順序を逆にしてみましょう。

a[i][z] = new int[q];         // 先に確保してから
a[i][z][q]=stdIn.nextInt();   // 値を入れる 

少しマシになりましたが、これでもまだおかしいです。
a[i][z]はq要素の配列ですが、それならq番目にアクセスできないからです。(0~q-1まで有効)

改めて、どのような配列を確保すべきなのか再考しましょう。

ここでは、ある生徒の各教科の点数を保持すれば良いのですね。
それならば、教科数(s.length)だけ要素を持てる配列を前以て確保すべきなのです。

for(z=0;z<a[i].length;z++){
    a[i][z] = new int[s.length];
    for(q=0;q<s.length;q++){
        System.out.printf("%d組%d番の"+s[q]+"の点数",i+1,z+1);
        a[i][z][q]=stdIn.nextInt();
    }
}

これで動くはず。

見たことが無いエラーでも、どの行で発生しているか分かるだけでここまで絞れます。
もちろん解決に至るためには勘と経験が必要ではありますが。

エラーの再現

できるだけミニマムなコードでエラーを再現するのは、良い学びになると思います。

class Main {
    public static void main(String[] args) {
        int arr[][] = new int[10][];
        
        System.out.println(arr[0]);
        System.out.println(arr[0][0]);
    }
}

実行結果 Wandbox

null
Exception in thread "main" java.lang.NullPointerException
	at Main.main(prog.java:6)

ついでに

生徒の点数を保持するためのクラスを書いた方がコーディングが楽です。
三次元配列なんて扱うのは面倒で仕方がありませんので。

Comments

[rink]

解答ありがとうございます！いつも丁寧にご回答してくださりとても助かります。
領域を確保する前に値を代入してしまったことになんで気が付かなかったのか、自分でも不思議なのですが、
今回は３次元配列のお勉強のためにわざとそうさせてもらってます。わざわざご意見有難うございます！

Answer 2

Exception in thread "main" java.lang.NullPointerException

at M6_19.main(M6_19.java:25)

M6_19.javaの、25行目で、NullPointerExceptionが発生した、ということですね。
「java NullPointerException」でぐぐってみるといろいろ解説が出てくると思います

Comments

[rink]

解答ありがとうございます！自分で調べてみたのですが、Nullとなっているところがよくわからなくて戸惑ってしまって…　
もっと勉強します！