Solved [Help] I hate these... Null Pointer Exception

Hello everyone!
I am currently working on a Zombie Survive plugin and have run into an issue with my Survival Crates!

Error:

Code:

Caused by: java.lang.NullPointerException
    at me.XvPROTECTEDvX.SurvivalCrate.SurvivalCrate.onSurvivalCrate(SurvivalCrate.java:29)

Code:

Code:java

@SuppressWarnings("deprecation")
@EventHandler
public void onSurvivalCrate(PlayerInteractEvent event){
Player player = event.getPlayer();
Block block = event.getClickedBlock();
Material blockType = block.getType(); //Line where the error is
Inventory inv = Bukkit.createInventory(player, 36, "Survival Crate");
Random ran = new Random();
int i = ran.nextInt(1);
i +=1;
if(blockType == Material.PISTON_BASE){
if(i == 1){
inv.setItem(20, new ItemStack(Material.BREAD));
inv.setItem(9, new ItemStack(Material.IRON_INGOT, 2));
inv.setItem(1, new ItemStack(Material.APPLE));
player.openInventory(inv);
}
//I have more randoms but I didnt want to create a huge Code entry
//I have a cooldown method but that isn't apart of the problem

Gotta love those Null Pointer Exceptions! <3
It happens when I interact with a mob. (Left/Right click them)

Thank you for reading!
All help is much appreciated!

EDIT: The Exception also occurs when I Right/Left click air.

 

XvBaseballkidvX
Make sure the block you clicked is not null.

 

You rightclick a entity and not a solid block. But you try to get the block that was clicked. since you did not clicked a block you get a NPE

probably you would need the PlayerInteractEntityEvent

 

Because you may not be clicking/interacting with any block. Do a proper check.

 

I did:

Code:

if(blockType == null) return;

And I still got the error :/ any other suggestions?

 

L33m4n123 I need see what Block the Player clicked though, or do you think I should add the PlayerInteractEntityEvent with the PlayerInteractEvent?

 

Oh Wait. So you mean with

that it works with the crates but gives you NPE's if you click an entity or what?

 

Where did you put that check in your code? Where is your updated code?

 

You want to check if the block is null, not the type. Besides, if it's a block, it has a type.

 

L33m4n123
It gives me an NPE when I Right/Left Click something other than a Block. The Crates work fine though (They Open up when I click them).

Skye thank you, trying that right now!

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Ohh ok. Yea my bad. I missunderstood you then there.. thought somehow you only want to interact with mobs.. d'oh

 

Skye Thank you! That fixed my problem!