I've got a block with a tile entity. The block takes the player that placed it and passes it to the tile entity, where the tile entity saves it to NBT. When it tries to do this, though, getName() for EntityPlayer is throwing a null pointer. I know that the EntityPlayer is not null, as the tile entity was functioning correctly before I added the nbt methods. Any help would be appreciated, thanks!

 

BlockGristGatherer.java

Spoiler

https://pastebin.com/xDD4iSE3

 

TileEntityGristGatherer.java

Spoiler

https://pastebin.com/TQatCguY

The crash report

Spoiler

https://pastebin.com/mi0VbSEN

 

Edited July 20, 20187 yr by naturaGodhead
1.12

2 minutes ago, naturaGodhead said:

TileEntityGristGatherer.java

Problematic code #11

Also you shouldn't be calling writeToNBT in your update method.

1 minute ago, Animefan8888 said:

Problematic code #11

Also you shouldn't be calling writeToNBT in your update method.

Okay but is there an explanation for why I shouldn't have any arguments in my TE constructor? How would I get the player entity that placed the block into the tileentity otherwise?

Just now, naturaGodhead said:

Okay but is there an explanation for why I shouldn't have any arguments in my TE constructor? How would I get the player entity that placed the block into the tileentity otherwise?

You can keep the one with the EntityPlayer parameter, but there needs to be a constructor with no arguments. This is because when the world is loaded and the TE needs to be placed it doesn't have the EntityPlayer that placed it, or anything else.

3 minutes ago, Animefan8888 said:

You can keep the one with the EntityPlayer parameter, but there needs to be a constructor with no arguments. This is because when the world is loaded and the TE needs to be placed it doesn't have the EntityPlayer that placed it, or anything else.

Okay, that makes sense. Thank you

One of those changes seems to have fixed my problem

I have not dealt with tile entities in a long time, but iirc you can have a field for the EntityPlayer in your TileEntity class and set it to the player that placed the block by overriding

    public void onBlockPlacedBy(World worldIn, BlockPos pos, IBlockState state, EntityLivingBase placer, ItemStack stack)
    {
    }

in your block's class.

Quote

    private int counter = 0;
    private EntityPlayer player;

 

Problematic code #9