Cannot cast an arrow's shooter to a player?

Hey all, I'm trying to make a plugin for where a player's health reaches to 0 by a bow, an event will occur. I've come across trying to check if the damager's instance was an arrow (this is under the EntityDamageByEntityEvent btw.) Then after that, I would cast the damager to an arrow and get the shooter from there, then casting the shooter to a player, yet it would not work, and it would give me an error.

Here is my code:

Code:java

if(event.getDamager() instanceof Arrow) {
                        Arrow arrow = (Arrow) event.getDamager();
                        Player damager = (Player) arrow.getShooter();
                        //do stuff to the shooter
}

Error:
http://pastebin.com/B7DrCFNQ

Line 207 is this:

Code:java

 Player damager = (Player) arrow.getShooter();

Thanks for the help

 

Try to use the ProjectileHitEvent and get the shooter with this:

PHP:

e.getEntity().getShooter();

 

HungerCraftNL AnOrangeGamer
event.getDamager()*

 

I don't get what you're saying about "event.getDamager()"
I'm trying to check if the damager in EntityDamageByEntityEvent is an arrow, if it is, get the arrow's shooter and do something to him (send a message, teleport him, etc)
If I don't get the shooter of the arrow, and get the damager, then it'll try to send a message to the arrow because it's the damager, which then would throw an error.

 

Code:java

 
Arrow arrow = (Arrow) e.getDamager();
Player shooter = (Player) arrow.getShooter();
 

Be sure to check if the instance of getDamager() is an arrow and the instance of getShooter() is a player or you will get errors.

 

I had that code to begin with and it threw an error, I also put that code in the thread as well.

 

AnOrangeGamer

Code:java

if(arrow.getShooter() instanceof Player){
    Player shooter = (Player) arrow.getShooter();
}

 

Wolfey
I did that and it threw the exact same error on the line where I checked if the shooter's instance was a player.

Bump?

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

 

Use his method it should work AnOrangeGamer

 

I was thinking about that at first, but I need to get the player that is getting hit by the arrow, then check if the player's health reaches 0, it does an event.

 

Code:java

    @EventHandler
    public void onHit(EntityDamageByEntityEvent e) {
        if (!(e.getDamager() instanceof Arrow))
            return;
        if(!(e.getEntity() instanceof Player)) return;
        Arrow r = (Arrow) e.getDamager();
        Player p = (Player) e.getEntity();
        if(!(r.getShooter() instanceof Player)) return;
        Player shooter = (Player) r.getShooter();
    }

AnOrangeGamer That doesn't work?

 

I did something similar to that, but that I'm pretty sure would not work. The error I have is on the line where I cast the shooter to a player. If you need the error check the pastebin link I put in the thread.

 

AnOrangeGamer then I do not know..

 

Bump?

 

Still no solution... bump?

 

You said you did something similar to that example above? What exactly did you do? Just the code you provided?

 

Yes. I initially had more to the code, like I would add a score to the shooter.

 

I mean, in the above code he did a couple of extra checks, you replied saying you did something basically similar. Often times I find similar does not imply correct. So I was just curious as to what you meant y similar.

 

Yes, I did what he did. All he did was check the instances. I did the same thing then returned if it wasn't true. I did the same thing.

 

Ya, but checking the instances is extremely important.

 

There is a different thing that is used to get the shooter.

Code:java

ProjectileSource ps = ((ProjectileSource) e.getEntity().getShooter());

Eclipse still says it is deprecated because if it returns a player it is deprecated, but if you set it equal to a projectilesource and cast it, it should work... haven't tested though. You can then check if(ps instanceof Player) and cast the ps to Player to get the player who shot it. Try that

 

Oops, sorry, worded it wrong. Yes I did check the instances as well.
unon1100
Thank you, I will try that.