[1.11.2] [Solved]Custom GUI error: NullPointerException

[ModMCdl]

Hey, so in my custom GUI, I'm getting a NullPointerException directed at the following override: (according to the console)

@Override
		public boolean onBlockActivated(World worldIn, BlockPos pos, IBlockState state, EntityPlayer playerIn, EnumHand hand, EnumFacing facing, float hitX, float hitY, float hitZ) {
			if(!worldIn.isRemote)
				playerIn.openGui(Reference.MOD_ID, GuiHandler.PESTLE_GUI, worldIn, pos.getX(), pos.getY(), pos.getZ());
			return true;
		}

I just can't figure out why I'm getting this error.

 

Crash report:

https://pastebin.com/NyFi0T5X

My GUI Handler:

https://pastebin.com/b4w8x1Zf

My Container:

https://pastebin.com/YYhks7au

 

Edited November 3, 2017 by ModMCdl
Solved

[ModMCdl]

2 minutes ago, diesieben07 said:

How is the stack "directed at the override"? There is no reference to the code you posted in the stack trace.

 

The issue is caused by a Slot having a null reference for it's inventory. In this case it can only be either the TileEntity or the player inventory. The latter is very unlikely.

Please show your whole Block class.

An error in the console was pointing at it, must have been a fluke.

 

Block Class:

https://pastebin.com/hHA0scMf 

[Draco18s]

Or rather, show the TileEntity class

[ModMCdl]

Just now, Draco18s said:

Or rather, show the TileEntity class

TileEntityPestle.java

https://pastebin.com/x1n5wapC

[ModMCdl]

1 minute ago, diesieben07 said:

Your block does not override hasTileEntity, as such it does not have a TileEntity.

Ah, didn't realize that was a thing I needed to do! What would I put in the override here? Would I just return the tileentity I made?

[ModMCdl]

10 minutes ago, diesieben07 said:

Please read the documentation for the method. It clearly explains what it does.

It is also f***ing self-explanatory.

Alright, no need to get all up in arms. I'm still learning here. And you going off doesn't help anyone who comes to this post in the future trying to solve the same problem.

Edited November 2, 2017 by ModMCdl

[ModMCdl]

33 minutes ago, diesieben07 said:

Your block does not override hasTileEntity, as such it does not have a TileEntity.

Would I also need to override createTileEntity and, because I'm delving into new territory here, are there any thorough examples/explanations for how this is done? (Besides the documentations which helped me very little.

[ModMCdl]

4 minutes ago, diesieben07 said:

You need to both override hasTileEntity and createTileEntity, yes.

The documentation has no information about TileEntities, I'd be curious what little help you got out of it despite that.

I went here: https://takahikokawasaki.github.io/minecraft-resources/javadoc/forge/1.8-11.14.1.1320/net/minecraft/block/Block.html#hasTileEntity(net.minecraft.block.state.IBlockState)

[ModMCdl]

Just now, diesieben07 said:

That is neither official nor up to date in any way, shape or form.

You directed to me "read the documentation for the method" and that was the only documentation I found.

[ModMCdl]

Just now, diesieben07 said:

You have an IDE, do you not?

It only gave me the standard Overrides "x" method

[ModMCdl]

Just now, diesieben07 said:

Yours must be broken then, you should look into fixing it:

I use eclipse. It has never done that. Hovering over the method is what brings up said explanation, and I only get "Overrides hasTileEntity" and the path.

[Draco18s]

*Cough*

This:

[ModMCdl]

Once I overrode hasTileEntity and createTileEntity, I now error in my TileEntityPestle class.

https://pastebin.com/15r7mcCS

[Draco18s]

Your inventory is size 3.

 

        this.addSlotToContainer(new Slot(tileentity, 0, 31, 24));

        this.addSlotToContainer(new Slot(tileentity, 1, 66, 43));

        this.addSlotToContainer(new SlotPestleOutput(player.player, tileentity, 3, 119, 30));

 

0, 1, 3. Yes. That is how we count.

[ModMCdl]

1 minute ago, Draco18s said:

Your inventory is size 3.

 

        this.addSlotToContainer(new Slot(tileentity, 0, 31, 24));

        this.addSlotToContainer(new Slot(tileentity, 1, 66, 43));

        this.addSlotToContainer(new SlotPestleOutput(player.player, tileentity, 3, 119, 30));

 

0, 1, 3. Yes. That is how we count.

Hahahahahaha

NOW I feel stupid. I replaced everything that referenced the mysterious "slot 3" and reset the index to 0, 1, 2. Now the GUI works, it doesn't crash, and now I just have to figure out why the crafting recipes don't actually work!

 

[ModMCdl]

Side note, what's the updated name for transferStackInSlot. It doesn't seem to be working for me in 1.11.2

Please just ignore that, I'm an idiot.

Edited November 2, 2017 by ModMCdl

[ModMCdl]

1 minute ago, diesieben07 said:

transferStackInSlot still exists.

I am well aware. I'm just an idiot.

[ModMCdl]

Now, because I'm very needy, can you also possibly point out what's wrong here with my crafting? I don't get any errors or crashes, it just doesn't work.

Pestle Recipe class:

https://pastebin.com/15huRuYq