OMG This events .

Code

Code:

@EventHandler
    public void onDeath(PlayerDeathEvent e){
        if(e.getEntity().getKiller() != null && e.getEntity() instanceof Player && e.getEntity().getKiller() instanceof Witch){
            Player death = (Player) e.getEntity();
            PacketPlayInClientCommand ppicc = new PacketPlayInClientCommand(EnumClientCommand.PERFORM_RESPAWN);
            ((CraftPlayer)death).getHandle().playerConnection.a(ppicc);
            Apollon.g().players.remove(death.getName());
            Apollon.g().players.put(death.getName(), new PlayerData("Czarodziej", new Date()));
            SkinChanger.removeSkin(death);
            SkinChanger.changeSkin(death, czarodziejSkin);
            TagChanger.changeTag(death, death.getName());
            Commands.addPotionsCzarodziej(death);
        }
       
        if(e.getEntity().getKiller() != null){
            if(e.getEntity().getKiller() instanceof Player){
                Bukkit.broadcastMessage("KUPA GRACZ");
                Player killer = (Player) e.getEntity().getKiller();
                Player death = (Player) e.getEntity();
                PacketPlayInClientCommand ppicc = new PacketPlayInClientCommand(EnumClientCommand.PERFORM_RESPAWN);
                ((CraftPlayer)death).getHandle().playerConnection.a(ppicc);
                boolean cK = Apollon.g().players.containsKey(killer.getName());
                if(cK){
                    String rasaK = Apollon.g().players.get(killer.getName()).getRasa();
                        Apollon.g().players.remove(death.getName());
                        Apollon.g().players.put(death.getName(), new PlayerData(rasaK, null));
                    if(rasaK.equalsIgnoreCase("Wilkolak")){
                        SkinChanger.removeSkin(death);
                        SkinChanger.changeSkin(death, wilkolakSkin);
                        TagChanger.changeTag(death, death.getName());
                    }else if(rasaK.equalsIgnoreCase("Gracz")){
                       
                    }else if(rasaK.equalsIgnoreCase("Wampir")){
                        SkinChanger.removeSkin(death);
                        SkinChanger.changeSkin(death, wampirSkin);
                        TagChanger.changeTag(death, death.getName());
                    }else if(rasaK.equalsIgnoreCase("Czarodziej")){
                        SkinChanger.removeSkin(death);
                        SkinChanger.changeSkin(death, czarodziejSkin);
                        TagChanger.changeTag(death, death.getName());
                        Commands.addPotionsCzarodziej(death);
                    }
                }
            }
        }
    }

Errors : 0

Working only with Player

 

Plugers11
What are you trying to exactly do?

 

Yep! That's code!

My best answer is this. Seeing as you have this.....

Code:

if(e.getEntity().getKiller() != null){
            if(e.getEntity().getKiller() instanceof Player)

I can assume that you don't know that getKiller() only returns a Player or null. Does that answer your concern?

 

All of your checks to ensure that the killer is an instanceof a Player are pointless. LivingEntity#getKiller is always a Player object.

EDIT: Ninja'd by teej107

Assuming you're attempting to get the killer if they're not a Player.

1) Listen to EntityDamageByEntityEvent.
2) Check if EntityDamageByEntityEvent#getEntity#getHealth is less-than-or-equal-to 0.
3) If the above check is true, the killer is EntityDamageByEntityEvent#getDamager.

Note, I say 'less-than-or-equal-to 0' because I can't recall if health can go into negatives if the damage is higher than existing health.

EDIT: Though, I suppose 'less-than 1' would also suffice.

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

 

FerusGrim
Hmm
Never mind but
How u want to check health from damageevent ?
I can use InteractEntityEvent but not damage

 

You could try something like this:

Code:java

@EventHandler
public void onDamage(EntityDamageByEntityEvent e) {
Damageable damaged = e.getEntity();
Damageable damager = e.getDamager();
 
double damagedHealth = damaged.getHealth();
double damagerHealth = damager.getHealth();
}

 

Health has been doubles for a while now, less than 1 doesn't suffice