質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.09%

二次元配列にnullを格納して、その二次元配列の要素を引数としてエラーなくメソッドを呼び出したい。

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,502

score 23

前提・実現したいこと

二次元配列にnullを格納して、その二次元配列の要素を引数としてエラーなくメソッドを呼び出したい。

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

二次元配列にnullを格納し、メソッドを呼び出すとjava.lang.NullPointerExceptionが返されてしまう。

[createRecordメソッド]
ケースNo1
ケースNo2
Exception in thread "main" java.lang.NullPointerException
    at com.ams.work210.Work210Test.main(Work210Test.java:33)

該当のソースコード

package com.ams.work210;

public class Work210Test {
    public static void main(String[] args) {
        Work210 work210 = new Work210();
        System.out.println("[createRecordメソッド]");

        String[][] record2 = {null, {}, { "X" }, { "X", "X", "X", "X" }, { "X", "X", "X" },
                { "4903110006770", "毎朝のパン", "138" }, { "4903110006770", null, "138" },
                { "4903110006770", "毎朝の食パン", null } };

        System.out.println("ケースNo1");
        String eMessage = "";

        ProductBean productBean = new ProductBean();


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

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

            try {
                productBean = work210.createRecord(record2[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です。");
        }

        if (record.length != 3) {
            throw new IllegalArgumentException("引数の要素数が不正です。");
        }

        try {
            if (record != 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];

        productBean.setCode(code);
        productBean.setName(name);

        if (record[2] == null) {
            productBean.setPrice(null);
            return productBean;

        } else {
            BigDecimal price = new BigDecimal(keepPrice);
            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;
    }

}

試したこと

二次元配列にnullを格納せずに以下のように書いた場合は、エラーなく実行出来ました。けれどなぜエラーなく実行できたのかわかりません。

String[] record1 = null;
String[][] record2 = {{}, { "X" }, { "X", "X", "X", "X" }, { "X", "X", "X" },
{ "4903110006770", "毎朝のパン", "138" }, { "4903110006770", null, "138" },
{ "4903110006770", "毎朝の食パン", null } };

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

なぜエラーなく実行できたのかわかりません。

NullPointerException よりも, null という値のあり方そのものをご理解頂いていないようです.

オブジェクト変数には, オブジェクトを指している状態(!null)と指していない状態(null) があります.
オブジェクトを指していれば, 例えば String a="abc"; ならば String のメソッドである length() が使えます(a.length() は例外にならず 3 が返る)が, オブジェクトを指していなければ(String a=null; なら) NullPointerException が発生します.
null になる可能性のあるオブジェクト変数を用いる場合, その変数でメソッドを呼び出す前に, 変数が null かどうかチェックを行い, null で無ければメソッドを呼び出すようにしなくてはなりません. (例: if(a != null) System.out.println("length="+a.length());)
(メソッドレベルでは, 引数として null が渡された場合は意図的に NullPointerException を発生させる場合はあります.)

オブジェクト(変数)にはこのように常に null かどうかを気にする必要があり, それを怠ると思わぬ時に例外が発生してプログラムが止まる可能性があるため, 最近は言語レベルで「変数が null になる可能性がある/ない」宣言や「null チェックを含むメソッド呼び出しが書ける」ような機能・言語が登場しています.

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/18 14:54

    返信が遅くなり申し訳ありません。メソッドを呼び出す前に、変数がnullかどうかを意識し忘れていました。次回からは、メソッドを呼び出す前にオブジェクトがnullであるかどうかをチェックするようにします。回答ありがとうございました。

    キャンセル

+1

配列内のどの要素でNullPointerExceptionが発生してるのかを確認してください。
(Work210Test.java:33
i == 0の時、record2[i]は nullです。
for文で、nullに対してlengthを実行しているので、NullPointerExceptionが発生します。
コードをそのまま生かすなら、
for (int j = 0; j < record2[i].length; j++) {
の前に、record2[i]がnullか否かチェックし、nullの場合には、continueするといった処理が必要です。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/18 14:57

    返信が遅くなり申し訳ありません。nullに対してlengthを実行していたので、NullPointerExceptionが発生していることに気づいていませんでした。ご指摘ありがとうございます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.09%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • トップ
  • Javaに関する質問
  • 二次元配列にnullを格納して、その二次元配列の要素を引数としてエラーなくメソッドを呼び出したい。