Solved @EventHandler code happening three times

I am creating a wizard pvp plugin but whenever players click with the item the code runs three times instead of just once like it is supposed to. I really need help fixing this.

Code:java

package Pimp;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.block.Block;
import org.bukkit.entity.EnderPearl;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.entity.Snowball;
import org.bukkit.entity.WitherSkull;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.player.PlayerInteractEvent;
 
public class MyPlayerListener implements Listener {
    public static Pimp plugin;
    long LastRun = 0;
    long LastRunEnderpearl = 0;
 
    @SuppressWarnings("deprecation")
    @EventHandler(priority = EventPriority.MONITOR)
    public void onPlayerInteract(PlayerInteractEvent event) {
        final Player player = event.getPlayer();
        int blockId = player.getItemInHand().getType().getId();
        if (blockId == 280) {
            if (LastRun < System.currentTimeMillis() - 1000) {
                player.throwSnowball();
                LastRun = System.currentTimeMillis();
        }
        } else if (blockId == 294) {
            Block block = player.getTargetBlock(null, 5);
            Location location = block.getLocation();
            World world = player.getWorld();
                if (LastRun < System.currentTimeMillis() - 1000) {
                    world.strikeLightning(location);               
                    LastRun = System.currentTimeMillis();       
            }
        } else if (blockId == 369) {
            if (LastRun < System.currentTimeMillis() - 1000) {
                player.launchProjectile(WitherSkull.class);
                LastRun = System.currentTimeMillis();       
        }
        } else if (blockId == 406) {
            if (LastRunEnderpearl < System.currentTimeMillis() - 30000) {
                player.launchProjectile(EnderPearl.class);
                LastRunEnderpearl = System.currentTimeMillis();
        }
 
        }
    }
 
    @SuppressWarnings("deprecation")
    @EventHandler(priority = EventPriority.HIGHEST)
    public void onEntityDamageByEntityEvent(EntityDamageByEntityEvent event) {
        if (event.getDamager() instanceof Snowball
                && event.getEntity() instanceof Player) {
            Entity ent = (Entity) event.getEntity();
            if (ent instanceof Player) {
                ((Player) ent).damage(5);
            }
        }
    }
    @EventHandler(priority = EventPriority.HIGHEST)
    public void onPlayerDeathEvent(PlayerDeathEvent event) {
        final Player player = (event.getEntity());
        event.setDeathMessage(ChatColor.GOLD + "" + player.getName() + ChatColor.GRAY + " has fallen!");
        return;
        }
 
 
}
 

 

Is the event supposed to happen when you right click? If so, use this:

Code:java

if(event.getAction() == Action.RIGHT_CLICK_AIR ||event.getAction() == Action.RIGHT_CLICK_BLOCK){
      final Player player = event.getPlayer();
        int blockId = player.getItemInHand().getType().getId();
        if (blockId == 280) {
          if(event.getAction() == Action.RIGHT_CLICK_AIR ||event.getAction() == Action.RIGHT_CLICK_BLOCK)  if (LastRun < System.currentTimeMillis() - 1000) {
                player.throwSnowball();
                LastRun = System.currentTimeMillis();
        }
        } else if (blockId == 294) {
            Block block = player.getTargetBlock(null, 5);
            Location location = block.getLocation();
            World world = player.getWorld();
                if (LastRun < System.currentTimeMillis() - 1000) {
                    world.strikeLightning(location);             
                    LastRun = System.currentTimeMillis();     
            }
        } else if (blockId == 369) {
            if (LastRun < System.currentTimeMillis() - 1000) {
                player.launchProjectile(WitherSkull.class);
                LastRun = System.currentTimeMillis();     
        }
        } else if (blockId == 406) {
            if (LastRunEnderpearl < System.currentTimeMillis() - 30000) {
                player.launchProjectile(EnderPearl.class);
                LastRunEnderpearl = System.currentTimeMillis();
        }
        }
        }
}

 

No i want it either or and the way that i am making it its easier. But my problem is that the code from the interactions happens three times instead of one

KingMagnus

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

 

I just tested your code on my server and for left/right clicking anything it only did it once. Are you sure you didn't mess up anywhere?

 

You could have three different instances of your plugin running on your server.

 

I might just die laughing if this is your problem.

 

THANK YOU SO MUCH amhokies! #VirginiaTech

 

Glad I could help