Help with Player move event

OK so I'm trying to check if a player is standing on Ender_Stone and if they are, are they wearing a Diamond helmet and if there not //do nothing

else if there wearing anything other it adds a bunch of potion effects and a message

but it seems i cant get it to work at all heres my code

Code:java

 
package com.arnie.space.listeners;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.block.BlockFace;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
 
public class PlayerListener implements Listener{
 
 
public void SpaceCheck(PlayerMoveEvent event){
Player player = event.getPlayer();
Block block = player.getLocation().getBlock().getRelative(BlockFace.DOWN);
 
if (block.getType() == Material.ENDER_STONE){
if (player.getInventory().getHelmet().getType() == Material.DIAMOND_HELMET){
// do nothing
} else {
if (player.getInventory().getHelmet().getType() == Material.LEATHER_HELMET){
 
player.sendMessage(ChatColor.RED + "WARNING:" + ChatColor.YELLOW + "Helmet malfunction Oxygen levels depleting!");
player.addPotionEffect(new PotionEffect(PotionEffectType.SLOW, 20, 10));
player.addPotionEffect(new PotionEffect(PotionEffectType.CONFUSION, 20, 10));
player.addPotionEffect(new PotionEffect(PotionEffectType.HARM, 20, 10));
player.addPotionEffect(new PotionEffect(PotionEffectType.JUMP, 20, 10));
 
[syntax][/syntax]

 

you forgot the @EventHandler annotation.

 

I'm a Retard ¬_¬

Thanks

Sorry i have another issue now

I get a null point on line 25 :/

Which is

Code:java

 
 
            if (player.getInventory().getHelmet().getType() == Material.DIAMOND_HELMET){[syntax]
 
EDIT by Moderator: merged posts, please use the edit button instead of double posting.[/syntax]

 

use a null-check: if(player == null){ return; }

 

fixed it by checking if the helmet slot was empty first not if it was a diamond one

thanks for help guys

I have another Question if someone could help I dont seem to understand how to Delay the tasks so it doesnt spam the user but i cant seem to get my head round it

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

 

you should put this or its equivalent as the first bit of code in your SpaceCheck event. this way you only do your code if the player as changed block possition not every little movement.


int fromX=(int)event.getFrom().getX();
int fromY=(int)event.getFrom().getY();
int fromZ=(int)event.getFrom().getZ();
int toX=(int)event.getTo().getX();
int toY=(int)event.getTo().getY();
int toZ=(int)event.getTo().getZ();

if(fromX!=toX||fromZ!=toZ||fromY!=toY){
PUT YOU CODE INSIDE HERE
}