Healing items

So I am coding a kitpvp server and I want no items/armor to take durability damage. So I used this code:

Code:

@EventHandler
    public void onDamage(EntityDamageByEntityEvent e) {
        if (e.getDamager() instanceof Player) {
            Player killer = (Player) e.getDamager();
                killer.getItemInHand().setDurability((short) 1); 
        }else if(e.getEntity() instanceof Player){
            ItemStack[] armor = ((Player) e.getEntity()).getInventory().getArmorContents();
            for(ItemStack i : armor){
                i.setDurability((short)0);
            }
        } 
    }   
    @EventHandler
    public void onBowDamage(EntityShootBowEvent e) {
        if (e.getEntity() instanceof Player) {
            e.getBow().setDurability((short) 1);
               
            }
           
        } 

But for some reason the armor heals but the bow and sword don't.

 

I notice you put '(short) 1); on the bow and then you put '(short) 0); on the armor, could that be why?

 

@ChucknorisSR
You probably just want to use if, not else if.

In your code if a player is being attacked by a zombie, his armor will heal, but not if he is being attacked by another player.

 

Like @MnMaxon already said Java checks first if the damager is an instance of a player. If it's true it returns and doesn't even check your else if statement. Just remove the "else", so it checks both.