スッキリわかるJava入門 第10章 練習問題 回答でのエラー

Question

スッキリわかるJava入門という書籍の練習問題を解いていて、
Wandクラス、Wizardクラスについては、練習問題の回答をそのまま書き写したのですが、下記のように、値がnullであると、2個エラーがでてしまいます。

なぜこのようなエラーが起きてしまうのでしょうか？

大変恐れ入りますが、ご教授いただけると助かります。。。。。

Exception in thread "main" java.lang.NullPointerException
	at Wizard.heal(Wizard.java from InputFileObject:41)
	at Main.main(Main.java from InputFileObject:13)

1個目のエラー
int recovPoint = (int) (basePoint * this.getWand().getPower());
2個目のエラー
wizard.heal(h);

//Mainクラス
public class Main {
    public static void main(String[] args) {
        Hero h = new Hero();
        System.out.println(h.getHp());
        Wizard wizard = new Wizard();
        wizard.heal(h);
    }
}

//Heroクラス
public class Hero {
    private String name;
    private int hp = 10;

    public String getName(){
        return this.name;
    }

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

    public int getHp(){
        return this.hp;
    }

    public void setHp(int hp){
        this.hp = hp;
    }
}

//Wizardクラス
public class Wizard {
    private int hp;
    private int mp;
    private String name;
    private Wand wand ;

    public String getName(){
        return this.name;
    }

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

    public int getHp(){
        return this.hp;
    }

    public void setHp(int hp){
        this.hp = hp;
    }

    public int getMp(){
        return this.hp;
    }

    public void setMp(int mp){
        this.mp = mp;
    }

    public Wand getWand(){
        return this.wand;
    }

    public void setWand(Wand wand){
        this.wand = wand;
    }

    public void heal(Hero h){
        int basePoint = 10;
        int recovPoint = (int) (basePoint * this.getWand().getPower());
        h.setHp(h.getHp() + recovPoint);
        System.out.println
                (h.getName() + "のHPを" + recovPoint + "回復した！");
    }
}

//Wandクラス
public class Wand {
    private String name;
    private double power;

    public String getName(){
        return this.name;
    }

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

    public double getPower(){
        return this.power;
    }

    public void setPower(double power){
        this.power = power;
    }
}

Answer 1

質問に記されたのはスタックトレースと呼ばれるものですが、

Main.javaの13行目を実行した
↓
Main.javaの13行目によって呼び出されたWizard.javaの41行目を実行した
↓
それによってNullPointerExceptionという例外が発生した

という一連の流れを示しています。「エラーが2箇所発生した」のではありません。ですから、注目すべきはWizard.javaの41行目がなぜNullPointerExceptionとなるのかということです。

当該行を見ると、

        int recovPoint = (int) (basePoint * this.getWand().getPower());

となっていますが、ここでgetWand()というメソッドを呼び出しています。getWand()は単に自分のメンバー変数として持っているwandを返すだけのものですが、このwandという変数に初期値を代入する処理がどこにも無いので、この行に到達した時点でwandはnullのままです。ですからgetWand()の戻り値はnullです。

それなのに、その戻り値を参照してgetPower()というメソッドを呼び出そうとしています。ですから、ここはnull.getPower()としているのと同じことになります。これがNullPointerExceptionの原因でしょう。

メンバー変数wandには、値をセットするためのsetWand()というメソッドが用意されていますが、参考にした書籍にはそれを呼ぶコードがあるのでありませんか？それが抜けているのだと思います。

Comments

[revoiot]

大変わかりやすい回答いただきありがとうございました。
これから、wandに初期値を設定してみたり、setwandで値を変えてみたりしてみようと思います。

大変恐れ入りますが、今回はwandのインスタンス化をすすめてくれたAkashicさんをベストアンサーに選ばさせていただきました。

Answer 2

Exception in thread "main" java.lang.NullPointerException
    at Wizard.heal(Wizard.java from InputFileObject:41)
    at Main.main(Main.java from InputFileObject:13)

2個のエラーというか、
Main.java の wizard.heal(h); から呼び出した先
Wizard.java の int recovPoint = (int) (basePoint * this.getWand().getPower());
でエラーになっている意味かと思います。

直接の原因は、this.getWand()のインスタンスが作られておらずnullになっているので、getPower()をする時にNullPointerExceptionになっているように見えます。

書籍の練習問題の回答とのことですので、setWand()している箇所などWand関連で書き漏れが無いか確認してみてはいかがでしょうか？どこかでnew Wand()してsetWand()するのかと予想されます。

単純にNullPointerException回避だけであれば、Wizard .java内でnewするだけでも回避できます。
結局どこかで設定しないとh.getName()もnullになるかと思いますが。

public class Wizard {
　　…
    private Wand wand = new Wand();
　　…

Comments

[revoiot]

再度書き漏れがないのか確認しましたが、特にありませんでした。

おそらく、書籍のミスで、wandをインスタンス化する処理が書かれていなかったのだと思われます。

大変わかりやすい回答をしていただきありがとうございました。

Answer 3

https://github.com/walkingmask/sukkiri_java/blob/master/10/Wizard.java

によりますと, Wizard クラスのコンストラクタで Wand を設定する(ことになる?)ようです. が, これは古い版かもしれません.

Comments

[revoiot]

ご回答いただきありがとうございます。
私が持っている参考書のバージョンの回答では、Wizard クラスのコンストラクタの記述はありませんでした。
おそらく、参考書の回答の不足によるものだと思われます。