java.lang.NullPointerExceptionの対処

引数の数だけ乱数（0～99）を生成する、int[] makeArray(int n)、戻り値はint型の配列
引数が０または負の値の場合は、「引数が不正です」と表示し、nullを返却する。
引数として受け取った配列の中身を出力する、void printArray(int[] a)、戻り値void
引数の配列がnullだった場合は、何も表示しない。

上記２つのメソッドを定義する。

main()メソッドで、画面から数値を入力させて、
入力した値をmakeArrayメソッドに渡し、
その戻り値をprintArrayメソッドに渡して実行確認をする。

package sample;

import java.util.Scanner;
import java.util.Random;

public class Sample {
	
	public static int[] makeArray(int n) {
		Random rand = new Random(System.currentTimeMillis());
		
		if(n <= 0) {
			System.out.println("引数が不正です");
			return null;
		}
		
		int[] array = new int[n];
		for(int i=0;i<array.length;i++) {
			array[i] = rand.nextInt(100);
		}
		return array;
	}
	
	public static void printArray(int[] a) {
		if(a == null) {
		}
		
		for(int i=0;i<a.length;i++) {
			System.out.print(a[i]+",");
		}
	}
	
	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		
		System.out.print("乱数の数：");
		int n = sc.nextInt();
		makeArray(n);
		printArray(makeArray(n));
	}
}

上記のコードを実行すると、正の数は実行されますが、0と負の数はエラーが起こります。
どう対処すれば正しく実行されるか協力お願いします。
エラー

Exception in thread "main" java.lang.NullPointerException: Cannot read the array length because "a" is null
at sample.Sample.printArray(Sample.java:27)
at sample.Sample.main(Sample.java:38)

行動規範の内容に同意します

回答1件

ベストアンサー

エラーはその続きもきちんと読みましょう。

Exception in thread "main" java.lang.NullPointerException: Cannot read the array length because "a" is null

"a"という変数がnullだったので、配列が読み込めなかった、と言っています。

at sample.Sample.printArray(Sample.java:27)

これは、Sampleというファイルの27行目のprintArrayというメソッドで起こったエラーだという事を示しています。数えてみると、

java
1    public static void printArray(int[] a) {
2        if(a == null) {
3        }
4
5        for(int i=0;i<a.length;i++) {  // この行
6            System.out.print(a[i]+",");
7        }
8    }

を指しています。
これは、aがnullでも、何もしないまま通過してしまうため、結局nullに対して配列の長さを参照しようとしているため、エラーが起きているのです。
対処法の1つとしては、aがnullだった時にメソッドを終了する方法があります。

diff
1    public static void printArray(int[] a) {
2        if(a == null) {
3+           return;
4        }
5
6        for(int i=0;i<a.length;i++) {
7            System.out.print(a[i]+",");
8        }
9    }