NullPointerExceptionの原因が分からないです。

Question

主人公(Heroクラス)の体力を回復するアイテム(RipoDクラス)を使おうとすると、NullPointerExceptionが出ます。エラーコードの意味はわかりますが、なぜ参照先がnullになっているのか分かりません。
アイテムを使うとき、リスト(ItemListクラス)からアイテムの在庫を減らすために、リストを取得するメソッド(getItemListを呼び出しています。エラーメッセージを見ると、このgetItemListの参照先がnullのようです。

コード中に※印を振りました。
※1が最初に呼び出され、その中で※2を呼び、さらにその中で※3を読んだときに参照先、つまりitemListがnullになっているようです。
mainクラスでItemListもRipoDもインスタンス化してるので、コンストラクタで設定されてるaddItemListが呼び出されてnullにはならないと思うのですが。
なぜnullなのか分かりません。よろしくお願いします。
なぜこのようなめんどくさいコードになってるかには触れないでください、、、、。

エラーメッセージ
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at Item.RipoD.getStock(RipoD.java:40)//※3
at Item.RipoD.addStock(RipoD.java:31)//※2
at Item.RipoD.use(RipoD.java:25)//※1
at Main.Main.actionPerformed(Main.java:388)//Mainクラスのrd.use(h);の部分

public class ItemList {
	private String itemList;

	public void ItemList(){
		itemList = "";
	}
	
	public String getItemList() {
		return itemList;
	}

　　public void addItemList(int itemNumber, String itemName, int stock) {
		if(itemNumber == 1){
			itemList = "(" + itemNumber +")" + itemName + ":" + stock + "個\n";
		}
		else{
			itemList += "(" + itemNumber +")" + itemName + ":" + stock + "個\n";
		}
	}

public class RipoD implements Item{
	private int stock;
	private String name;
	private int itemNumber;
	ItemList item;
	
	public RipoD() {
		
	}
	
	public RipoD(ItemList item){
		stock = 0;
		name = "リポD";
		itemNumber = 1;
		item.addItemList(itemNumber, name, stock);
		item.addAllItemList(itemNumber, name);
		item.addIList(name);
	}

	public void use(Main.Hero h) {
		System.out.println("ごく、、、ごく、、、ぷはぁっ！\n体力が少し回復した");
		h.addHP(30);
		addStock(-1);   //※1
	}
	
	public void addStock(int num) {
		this.stock += num;
		String after = this.name + ":" + this.getStock() + num;   //※2
		/*item.changStock(this.name + ":" + this.getStock(), after);*/
	}

    public int getStock() {
		String text = item.getItemList();   //※3
		/*int number2 = text.indexOf("(" + itemNumber + ")");
		int number3 = number2 + 2 + this.name.length() + 2;
		int stock = Integer.parseInt(text.substring(number3, number3 + 1));
		this.stock = stock;*/
		return stock;
	}

public class Main extends JFrame implements MouseListener, ActionListener {

	public JTextArea mainArea;
	Scanner input = new Scanner(System.in);
	JMenuItem menuitem1;
	JMenuItem menuitem2;
	JMenuItem menuitem3;
	String userName = "";
	JTextField tf;
	JButton bt;
	Hero h;
	int year = 2;
	int mon = 9;
    int week = 1;
    Item.ItemList item = new Item.ItemList();
	Item.RipoD rd = new Item.RipoD(item);
	Item.RipoDX rdx = new Item.RipoDX(item);
	Item.PowerSyo powsyo = new Item.PowerSyo(item);

	public static void main(String args[]){
        rd.use(h);
	}

字数制限により、関係ないところはコードを消しています。

Comments

[LouiS0616]

エラーについて考察があるのは素晴らしいと思いますが、肝心のエラーメッセージを貼ることをお忘れなく。

[TA-KEY]

分かりました編集します。ご指摘ありがとうございます。

Answer 1

Main：

Hero h; // <- メンバ変数hが初期化されてないからNull

RipoD:

ItemList item; // <- メンバ変数itemが初期化されてないからNull

public RipoD(ItemList item){ // <- 引数itemはメンバ変数itemと別物
    stock = 0;
    name = "リポD";
    itemNumber = 1;
    item.addItemList(itemNumber, name, stock); // <- これは引数item
    item.addAllItemList(itemNumber, name);
    item.addIList(name);
}

public int getStock() {
    String text = item.getItemList();   //※3 上記の理由でこのitemはnull

Comments

[退会済みユーザー]

初期化されてないなら利用時でなくコンパイルエラーでは

[TA-KEY]

解答ありがとうございます！メンバ変数itemを初期化しないと、ItemListのコンストラクタは呼び出されないということであってますか？

[tkturbo]

おお、確かに＞asahina1979さん

[tkturbo]

RipoDクラスのコンストラクタで「this.item = new ItemList()」とかを書いておく必要があるってことで＞TA-KEYさん