配列の中身がnullだった場合にエラーメッセージをmainメソッドに返したい

Question

前提・実現したいこと

配列がnullだった場合にIllegalArgumentExceptionのエラーメッセージをmainメソッドに投げて、「引数はNULLです。」と表示させたい。

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

本来配列の中身がnullだった場合には、「引数はNULLです。」とエラーメッセージを返したいが要素数が0の時に返されるエラーメッセージ「引数の要素数が不正です。」が返されてしまう。

package com.ams.work210;

public class Work210Test {
	public static void main(String[] args) {
		Work210 work210 = new Work210();
		System.out.println("[createRecordメソッド]");
		String[][] record = { { null }, {}, { "X" }, { "X", "X", "X", "X" }, { "X", "X", "X" },
				{ "4903110006770", "毎朝のパン", "138" }, { "4903110006770", null, "138" },
				{ "4903110006770", "毎朝の食パン", null } };

		for (int i = 0; i < record.length; i++) {
			String eMessage = "";
			System.out.println("ケースNo" + (i + 1));

			for (int j = 0; j < record[i].length; j++) {
				System.out.println("入力値：" + record[i][j]);
			}

			ProductBean productBean = new ProductBean();

			try {
				productBean = work210.createRecord(record[i]);
			} catch (IllegalArgumentException e) {
				eMessage = e.getMessage();
			}

			if (eMessage.length() != 0) {
				System.out.println("結果:" + eMessage + "\n");
			} else {
				System.out.println("結果:" + productBean + "\n");
			}

		}
	}
}

package com.ams.work210;

import java.math.BigDecimal;

public class Work210 {

	public ProductBean createRecord(String[] record) throws IllegalArgumentException {


		if (record == null) {
			throw new IllegalArgumentException("引数がNULLです。");
		}

		boolean hantei = record.length >= 4 || record.length == 0 || record.length < 3;
		if (hantei == true) {
			throw new IllegalArgumentException("引数の要素数が不正です。");
		}

		try {
			if (record[2] != null) {
				Integer.parseInt(record[2]);
			}
		} catch (IllegalArgumentException e) {
			throw new IllegalArgumentException("引数の第3要素が数値に変換できません");
		}

		ProductBean productBean = new ProductBean();
		String code = record[0];
		String name = record[1];
		String keepPrice = record[2];

		if (record[2] == null) {
			productBean.setCode(code);
			productBean.setName(name);
			productBean.setPrice(null);

			return productBean;

		} else {
			BigDecimal price = new BigDecimal(keepPrice);
			productBean.setCode(code);
			productBean.setName(name);
			productBean.setPrice(price);

			return productBean;
		}

	}
}

package com.ams.work210;

import java.math.BigDecimal;

public class ProductBean {
	private String code = "";
	private String name = "";
	private BigDecimal price = new BigDecimal(0);

	public ProductBean() {
	};

	public String getCode() {
		return code;
	}

	public void setCode(String code) {
		this.code = code;
	}

	public String getName() {
		return name;
	}

	public void setName(String name) {
		this.name = name;
	}

	public BigDecimal getPrice() {
		return price;
	}

	public void setPrice(BigDecimal price) {
		this.price = price;
	}

	@Override
	public String toString() {
		return "code=\"" + code + "\"\n" + "name=\"" + name + "\"\n" + "price=" + price;
	}

}

試したこと

条件式を"record[0] == null"のように変えてみたが、ArrayIndexOutOfBoundsExceptionエラーが返されてしまう。

Comments

[退会済みユーザー]

> productBean = work210.createRecord(record[i]);

この行で渡してるrecord[i]は、コードを見る限り全て配列で、Nullは渡されてないです。
Nullを渡したときのコードをお願いします。

[退会済みユーザー]

配列サイズが０なのであって配列の中身が null ではありません。

Answer 1

「配列の中身がnull」ということと「配列自体がnull」は意味が異なります。

        if (record == null) {
            throw new IllegalArgumentException("引数がNULLです。");
        }

この書き方は後者であることを判定するだけで、前者の判定はできません。
中身の判定をしたいのなら、要素を取り出して個別にnullかどうかを判定する必要があります。

Comments

[Takuan2740]

個別に要素を取り出そうと思い、record[0],record[1],record[2]の要素を取り出しnullかどうかを判定しようとするとArrayIndexOutOfBoundsExceptionが返されてエラーになります。

[swordone]

そこは要素数でループを回さないといけません。要素数も不定のようですので。

[Takuan2740]

for文で回して要素数を取り出しnull判定をしたとろこ上手く判定でき、表示させたいエラーメッセージが表示されました。ありがとうございます。要素数が不定だった為エラーメッセージが出ていたと理解しました。

Answer 2

 String[][] record = { { null }, …

を

 String[][] record = { null, …

にしては如何でしょう.

Comments

[Takuan2740]

nullの{}を外したら上手く表示させたいエラーメッセージが出力されました、ありがとうございます。ただmainメソッドの入力値を出力させる箇所で、新たにjava.lang.NullPointerExceptionが返されエラーが表示されるようになりました。入力値を表示させる為に二重for文を使うのはやめた方がいいでしょうか？