Problem with EntityDeathEvent

I'm creatting a small PVP rewarding plugin. And it doesn't work.
This is my code.

Code:JAVA

 
public class entityListener extends EntityListener
{
    @SuppressWarnings("unused")
    private final Kill4Reward plugin;
 
 
    public entityListener(Kill4Reward plugin)
    {
        this.plugin = plugin;
    }
 
    @Override
    public void onEntityDeath (EntityDeathEvent event)
    {
        if (event instanceof PlayerDeathEvent) {
            Player attacker = (Player) ((EntityDamageByEntityEvent) event.getEntity().getLastDamageCause()).getDamager();
            Player victim = (Player) event.getEntity();
            Kill4Reward.economy.depositPlayer(attacker.getName(), 10);
            attacker.sendMessage(ChatColor.RED + "You killed" + victim + "and obtained 10");
 
        }
 
    }
}
 

Any solution?
Thanks for help.

//EDIT
no errors, at console.

 

For precautions do this:

Code:

public void onEntityDeath (EntityDeathEvent event)
{
if (event instanceof PlayerDeathEvent) {
Player attacker = ((EntityDamageByEntityEvent) event.getEntity().getLastDamageCause()).getDamager();
Player victim = event.getEntity();
if((attacker instanceof Player) && (victim instanceof Player))
{
Kill4Reward.economy.depositPlayer(attacker.getName(), 10);
attacker.sendMessage(ChatColor.RED + "You killed" + victim + "and obtained 10");
}
}

That will guard against trying to cast a creeper, spider, ghast, zombie, etc... as a Player.

 

but you "cast"(Not really since you just try and convert Entity to Player without a cast) before the if statement...

 

Something like this ?

Code:JAVA

 
    @Override
    public void onEntityDeath (EntityDeathEvent event)
    {
        if (event instanceof PlayerDeathEvent) {
            Entity eventEntity = ((EntityDamageByEntityEvent) event.getEntity().getLastDamageCause()).getDamager();
            Player victim = (Player) event.getEntity();
                if(eventEntity instanceof Player){
                    Kill4Reward.economy.depositPlayer(((Player) eventEntity).getName(), 10);
                    ((Player) eventEntity).sendMessage(ChatColor.RED + "You killed" + victim + "and obtained 10");
                }
 
        }
 
    }
 

 

Yes, that's better, but you're also casting into an EntityDamageByEntityEvent when you don't know if the last damage someone took was from an Entity

 

Yup I figured that out but forgot to post

Working code:

Code:JAVA

 
    @Override
    public void onEntityDeath(EntityDeathEvent event){
        Entity e = event.getEntity();
        if(event.getEntity().getLastDamageCause() instanceof EntityDamageByEntityEvent){
        Entity dmgr = ((EntityDamageByEntityEvent) event.getEntity().getLastDamageCause()).getDamager();
        if(e instanceof Player){
            if(dmgr instanceof Player){
 
                ((Player) dmgr).sendMessage("You killed " + ((Player) e).getName() + " and obtained " + plugin.getConfig_reward());
                Server server = plugin.getServer();
                server.broadcastMessage(((Player) dmgr).getName() + " killed " + ((Player) e).getName());
                plugin.economy.depositPlayer(((Player) dmgr).getName(), plugin.getConfig_reward());
 
 
            }
        }
        }
 
    }