teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

2

修正

2019/05/14 14:22

投稿

m.ts10806
m.ts10806

スコア80888

answer CHANGED
@@ -4,6 +4,6 @@
4
4
  Wizard.wandがそもそもnullのままです。
5
5
  nullのオブジェクトからメソッド(getPower())呼び出そうとしてエラーになっているように思います。
6
6
 
7
- 現状のコードではWizard内ではWandが型が宣言されただけでインスタンス化されたオブジェクトがセットされてないのでgetしようにもnullだと思います。
7
+ 現状のコードではWizard内ではwandが型が宣言されただけでインスタンス化されたオブジェクトがセットされてないのでgetしようにもnullだと思います。
8
8
 
9
9
  「"魔法の杖"」でnewしたwandを使いたければWizardに渡す必要があるでしょうし、wand自体が「杖」という意味があるので、「武器としてキャラクターに持たせる」のであればもう少しクラス設計を見直したほうが良さそうに思います。

1

修正

2019/05/14 14:22

投稿

m.ts10806
m.ts10806

スコア80888

answer CHANGED
@@ -1,8 +1,9 @@
1
1
  ちゃんとエラー行数も出てますよ。
2
2
  ` at Wizard.heal(Wizard.java:13)`
3
3
 
4
- this.getWand()がnullので
4
+ Wizard.wandそもそもnullのまま
5
- nullのオブジェクトからメソッド呼び出そうとしてエラーになっているように思います。
5
+ nullのオブジェクトからメソッド(getPower())呼び出そうとしてエラーになっているように思います。
6
6
 
7
- Wizard内ではWandがインスタンス化されてないのでgetしようにもnullだと思います。
7
+ 現状のコードではWizard内ではWandが型が宣言されただけでインスタンス化されたオブジェクトがセットされてないのでgetしようにもnullだと思います。
8
+
8
9
  「"魔法の杖"」でnewしたwandを使いたければWizardに渡す必要があるでしょうし、wand自体が「杖」という意味があるので、「武器としてキャラクターに持たせる」のであればもう少しクラス設計を見直したほうが良さそうに思います。