Why does it not work? (Arrow hit damage)

Hello,
does anybody see the fould I made in my void?

Code:java

@EventHandler
    public void onEntityDamageByEntity(EntityDamageByEntityEvent e){
        Player d = (Player) e.getDamager(); //Damager
        Player p = (Player) e.getEntity(); //Damaged
        if(e.getCause() == DamageCause.PROJECTILE){
            if (((String)teams.get(d.getName())).equalsIgnoreCase("red")) {
                if((((String)teams.get(p.getName())).equalsIgnoreCase("red"))){
                    e.setCancelled(true);
                    d.sendMessage(prefix + ChatColor.RED + "It's not allowed to hit your own team!");
                }
            }
            else if (((String)teams.get(d.getName())).equalsIgnoreCase("blue")) {
                if((((String)teams.get(p.getName())).equalsIgnoreCase("blue"))){
                    e.setCancelled(true);
                    d.sendMessage(prefix + ChatColor.RED + "It's not allowed to hit your own team!");
                }
            }
        }
    }

 

HungerCraftNL I believe (I could be wrong) that the e.getDamager() returns any entity that damaged the player, e.g.
player, arrow, etc

 

Pizza371

But I checked of it was an arrow by doing

Code:java

if(e.getCause() == DamageCause.PROJECTILE){

right?

 

HungerCraftNL but you're casting it to a player
if(e.getDamager() instanceof Arrow) {
Arrow a = (Arrow) e.getDamager();
Player damager = a.getShooter();
//stuff
}

 

You have to check that if the entity is a Player.

Example:

Object o="test example";
Integer i=(Integer)o; //this will throw a ClassCastException because String is not an subclass of Integer

Similarly,
e.getDamager(); may not be an instance of a Player.

You need to check it by using
if(e.getDamager() instanceof Player)

Check e.getEntity() with this too.

 

Thank you

 

No worries

 

Louis1234567890987654321 I've another problem too, I've messages dubble do you know where it can come from?

 

Do you mean you recieve the message "dubble"?