Null Pointer on Player Interact Event

Player interact event throws a null pointer exception on line 31

Code:java

@EventHandler
public void onInteract(PlayerInteractEvent e){
ItemStack s = new ItemStack(Material.JACK_O_LANTERN);
ItemMeta smeta = s.getItemMeta();
smeta.setDisplayName(ChatColor.GOLD + "Show Players");
List<String> slore = new ArrayList<String>();
slore.add("Right cick this to");
slore.add("open the match");
slore.add("joining queue!");
smeta.setLore(slore);
s.setItemMeta(smeta);
 
ItemStack p = new ItemStack(Material.PUMPKIN);
ItemMeta pmeta = p.getItemMeta();
pmeta.setDisplayName(ChatColor.GOLD + "Hide Players");
List<String> plore = new ArrayList<String>();
plore.add("Right cick this to");
plore.add("hide all players!");
pmeta.setLore(plore);
p.setItemMeta(pmeta);
 
ItemStack j = new ItemStack(Material.JACK_O_LANTERN);
ItemMeta jmeta = j.getItemMeta();
jmeta.setDisplayName(ChatColor.GOLD + "Show Players");
List<String> jlore = new ArrayList<String>();
jlore.add("Right cick this to");
jlore.add("show all players!");
jmeta.setLore(jlore);
j.setItemMeta(jmeta);
if (e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK || e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK) {
if (e.getItem().getType().equals(Material.MOB_SPAWNER)) {
if(me.jackson30007.arenas.main.wood.contains(e.getPlayer())){
e.getPlayer().openInventory(league);
}
else if(me.jackson30007.arenas.main.stone.contains(e.getPlayer())){
e.getPlayer().openInventory(league1);
}
else if(me.jackson30007.arenas.main.iron.contains(e.getPlayer())){
e.getPlayer().openInventory(league2);
}
else if(me.jackson30007.arenas.main.gold.contains(e.getPlayer())){
e.getPlayer().openInventory(league3);
}
else if(me.jackson30007.arenas.main.diamond.contains(e.getPlayer())){
e.getPlayer().openInventory(league4);
}else{
e.getPlayer().sendMessage(ChatColor.RED + "You are not in a valid league!");
return;
}
}else
if(e.getItem().getType().equals(Material.PUMPKIN)){
for(Player pl : Bukkit.getOnlinePlayers()){
e.getPlayer().hidePlayer(pl);
}
e.getPlayer().sendMessage("Hid Players");
e.getPlayer().addPotionEffect(new PotionEffect(PotionEffectType.BLINDNESS,30,10,true));
e.getPlayer().getInventory().setItemInHand(j);
me.jackson30007.arenas.main.hiden.add(e.getPlayer());
}else
if(e.getItem().getType().equals(Material.JACK_O_LANTERN)){
for(Player pl : Bukkit.getOnlinePlayers()){
e.getPlayer().showPlayer(pl);
}
e.getPlayer().sendMessage("Showing Players");
e.getPlayer().addPotionEffect(new PotionEffect(PotionEffectType.BLINDNESS,30,10,true));
e.getPlayer().getInventory().setItemInHand(p);
me.jackson30007.arenas.main.hiden.remove(e.getPlayer());
}
 
if (e.getClickedBlock().getType().equals(Material.CHEST)) {
e.setCancelled(true);
}
}
}

 

Primitives need only be compared with (==), not .equals(). Regardless, the item must be null (e.getItem() == null).

 

use if(e.getPlayer().getItemInHand().getType()....) {
and not e.getItem.