Events Firing Mutliple Times

Hello, I Have A Walking Dead Server and I am trying to make a Zombie Bites System. My Problem Is That The Entity Damage Events Are Firing Multiple Times I've Tried These 2 Methods But Both Fire Multiple Times:

Code:java

@EventHandler
public void onEvent1(EntityDamageEvent e){
 
  if(e instanceof EntityDamageEvent){
  //Fires Multiple Times For 1 Event.
  }
}
 
@EventHandler
public void onEvent1(EntityDamageByEntityEvent e){
//Fires Multiple Times!
}

That's Not My Exact Code As I check If The Entity Is A Player And Damager Is A Zombie Buy That's The Main Part.
Is There any Other Events To Try Or any Way Around This That You Guys Know? Thank You.

 

TehVoyager Check your imports, sometimes if you use multiple classes and import it twice it fires multiple times

Also make sure you register your event properly

 

jthort
This Is My Code:

Import:

Code:java

        pm.registerEvents(new BiteManager(), this);

Listener Class:

Code:java

package com.mcadventure.nick.walking;
 
import java.util.ArrayList;
 
import org.bukkit.entity.Player;
import org.bukkit.entity.Zombie;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
 
public class BiteManager implements Listener{
 
    public ArrayList<String> recentbit = new ArrayList<String>();
    public ArrayList<String> screwed = new ArrayList<String>();
    public ArrayList<String> amputated = new ArrayList<String>();
 
    public static Main plugin;
 
    public BiteManager(Main m){
        plugin = m;
    }
 
    public BiteManager(){
 
    }
 
    static BiteManager bm = new BiteManager();
 
    public static BiteManager getManager(){
        return bm;
    }
 
    @EventHandler
    public void onZombieBite(EntityDamageByEntityEvent e){
        if(e.getDamager() instanceof Zombie && e.getEntity() instanceof Player){
            Player p = (Player) e.getEntity();
            //Fires Multiple Times
        }
    }
 
}
 

Don't Think I've Imported Anything Wrong. Thanks.

 

TehVoyager Why do you have two constructors?

 

That's How I've Learned to make "Manager" Classes, I Have One Constructor That I Run On My onEnable In The Main Class Where I Initiate the plugin Variable and Another to Create A BitesManager Without Passing In Main, I'm not amazing at Java but that's how I learned. Right Now Am I Making 2 Instances That Both Fire The Event?

I Kind Of Found A Solution, See For YourSelf:

Code:java

package com.mcadventure.nick.walking;
 
import org.bukkit.entity.Player;
import org.bukkit.entity.Zombie;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
 
public class BiteListener implements Listener {
 
    long l = 0;
 
    @EventHandler
    public void onZombieBite(EntityDamageByEntityEvent e){
        if(e.getDamager() instanceof Zombie && e.getEntity() instanceof Player){
            if(System.currentTimeMillis() - l < 1500){
                e.setCancelled(true);
            }else{
                e.setDamage(e.getDamage() * 2);
                l = System.currentTimeMillis();
 
            }
            //10th digit cant be same
        }
    }
 
}

jthort

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