Check and see if damage is from arrow, if so, who shot it?

How can I check if a players damage come from an arrow, and if it did who shot the arrow, and if it was the player, who was the player?

 

to get the Shooter:

Code:java

public void onEntityShootBowzBroLol(EntityShootBowEvent event) {
        if(event.isCancelled()) { // We don't want to make any of our plugin-friends mad :D
            return;
        }
        if(!(event.getEntity() instanceof Player)) {
            return;
        }
        Player shooter = (Player) event.getEntity();
        "DO WHAT YOU WANT!"
}
}

to get the damager:

Code:java

public void onBla(EntityDamageByEntityEvent event){
if(event.getCause().equals(DamageCause.PROJECTILE)){
int dmg = event.getDamage();
BLAH
}
 
 

 

You can just use @EventHandler()'s ignore option to skip cancelled events.

However, checking if it's a projectile doesn't guarantee it's an arrow... it could be a snowball for all we know... there has to be a way to get the entity that did the damage (the arrow) and check if instanceof Arrow... for the shooter of the arrow I think it's a event method that gets that, the attacker.

 

Digi you do exactly what you said, check if the projectile is an instance of Arrow

 

Does this work if there's several shooters at once, or if there's multiple dispensers shooting at the same time? It seems like with only one tracking variable being used, the tracking variable could get overwritten before the arrow impact...

 

ShadowDrakken yes that would be the case for manniL 's code, but try what I said by getting attacker....
However, you can't know who shot the arrow in a dispenser, it could be anyone from the guy that placed the dispenser, the guy that placed the arrows inside or the guy that flipped the switch which you can't detect.

 

the dispenser would be the who

Anyway, I tried the code, it doesn't work... I found other code that looks more accurate, but it doesn't work either. I haven't setup any of my debugging to test exactly why though. There's no need to keep track of who shot it from what I can see, the arrow actually keeps track of the shooter itself.

In the EntityDamagedByEntity event you can get the projectile and the projectile itself has getShooter() you just have to cast it to the correct type, the problem I'm running into appears to be (I'll know for sure once I write my debugging) that getShooter() is returning LivingEntity not Player so I'm not sure how to tell if the LivingEntity is a player or a skeleton so that I can ignore or use it properly based on that.

 

entity instanceof Player and then cast.

 

Ok, finally got it all working I haven't tested with magic (not sure if there even is any), but this works for melee (tested), archery (tested) and should work for thrown potions (untested, but follows projectile rules) as well

Code:java

 
public void onEntityDeath (final EntityDeathEvent event) {
  EntityDamageByEntityEvent damageEvent;
 
  try {
    damageEvent = (EntityDamageByEntityEvent) event.getEntity().getLastDamageCause();
  } catch (Exception e) {
    return;
  }
 
  DamageCause damageCause = damageEvent.getCause();
  Entity attacker = null;
 
  switch (damageCause) {
  case MAGIC:
  case ENTITY_ATTACK:
    attacker = damageEvent.getDamager();
    break;
  case PROJECTILE:
    // Only certain projectiles are player originating
    if (damageEvent.getDamager().getType() == EntityType.ARROW) {
      Projectile arrow = (Arrow) damageEvent.getDamager();
      attacker = arrow.getShooter();
    } else if (damageEvent.getDamager().getType() == EntityType.SPLASH_POTION) {
      Projectile potion = (ThrownPotion) damageEvent.getDamager();
      attacker = potion.getShooter();
    } else {
      return;
    }
     break;
  }
 
  // If the attacker isn't a player, there's nothing to do
  if (attacker.getType() != EntityType.PLAYER) return;
 
  Entity target = damageEvent.getEntity();
 
  // Now you have your attacker and target!
}
 

 

Splash Potions return as Magic not Projectile

 

You guys seem to have gotten pretty far off from the OP's original request...

Code:java

@EventHandler(ignoreCancelled = true)
public void onHit(EntityDamageByEntityEvent event)
{
    if(event.getCause() == DamageCause.PROJECTILE)
    {
        Projectile proj = (Projectile)event.getDamager();
        LivingEntity shooter = proj.getShooter();
        String shooterName = getName(shooter);
        String targetName = getName(event.getEntity());
        Bukkit.broadcastMessage(String.format("%1$s shot %2$s for %3$d damage!", shooterName, targetName, event.getDamage()));
    }
}
 
private static String getName(Entity ent)
{
    if(ent instanceof Player)
        return ((Player)ent).getName();
    return ent.getClass().getSimpleName();
}

"Courier shot CraftSkeleton for 8 damage!" etc.
This probably also works with snowballs, but it will say 0 damage.

 

Doesn't Entity class have a getName() method that gets automatically filled with player's name, mob name or whatever the entity's name is ? :} Not sure, you should test.

 

Nope.

 

Google is so unreliable there because the page that they have cached might be outdated xD you'd be better off in-page searching.

Anyway, it would've been better to have a getName() tough.

Still, ent.getType().getName() would print a better mob/entity instead of the class name

 

I did in-page search first, but I can't exactly link to my Ctrl+F .

True... I didn't think of that.