ItemInHand [Solved]

[Ernio]

Hello,

I have this question:

How can I get the item ID the player is holding in hand and save it into variable with sub-blocks (itemDmg)?

Whats the function for it and how to use it?

Maybe eg. code?

 

Best regards,

Ernio

[BlueSpud]

You can get the currently equipped item of the player by using

Minecraft.getMinecraft().thePlayer.getCurrentlyEquippedItem()

I believe. This will return an ItemStack, and you can do what you wish with that.

[Ernio]

I found it (yesterday) before your answer, but thanks anyway.

Everything works fine, but one thing - what does this returns when you have nothing in your hand?

private int ScrollID = 0;
public boolean onBlockActivated(World var1, int var2, int var3, int var4, EntityPlayer player, int var6, float var7, float var8, float var9)
{
	if (!player.isSneaking())
	{
		Item item = Minecraft.getMinecraft().thePlayer.getCurrentEquippedItem().getItem();
		if(item == Main.Plan) this.ScrollID = 11;
		player.openGui(Main.instance, this.ScrollID, var1, var2, var3, var4);
		this.ScrollID = 0;
		return true;
	}
	else
	{
		return false;
	}
}

When I rightclick with fist (empty hand) it crashes and says that ,,Item item = Minecraft.getMinecraft().thePlayer.getCurrentEquippedItem().getItem();" is a wrong call - NullPointer

Also note: When this.ScrollID = 0; GuiHandler calls null.

 

Thanks in advance,

Ernio

[SanAndreaP]

1. Don't use the Minecraft class within methods which are called (also) serverside! THere's a player instance given in the onBlockActivated method parameters

2. check in your if statement if player.getCurrentEquippedItem() is not null