Solved Changing Bow damage

Hello there guys!
So I just came back from a LONG break of programming in java.
Now, I am having a problem here. (honestly, the code might even work) however, I'm looking at it and it doesn't seem properly and I cannot figure out what I'm going to do to make it right.
So what I want: When shoot a bow and the arrow its either a player or mob then do damage that is on the bow's meta lore with something like "Arrow Damage: 1-3" so it will deal damage randomly 1-3.

My code so far:

Code:java

public void onEntityDamageByEntity(EntityDamageByEntityEvent event) {
        Entity damager = event.getDamager();
 
 
        if (damager instanceof Arrow) {
            final Arrow arrow = (Arrow) damager;
            ItemStack i = ((Player) damager).getPlayer().getItemInHand();
            Player p = (Player) damager;
 
            if (i != null && i.hasItemMeta()) {
                for (String s : i.getItemMeta().getLore()) {
 
                    if (s.contains("Arrow Damage:")) {
                        String dmgStr = s.substring(16);
                        String dmgAmnt[] = dmgStr.split("-");
                        Random rand = new Random();
                        int damageAmount = rand.nextInt(Integer.parseInt(dmgAmnt[1]) - Integer.parseInt(dmgAmnt[0]) + 1) + Integer.parseInt(dmgAmnt[0]);
                        event.setDamage(damageAmount);
                    }
 
                }
            }
        }

Yup, hit a cow and gave errors:

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

 

1) listen for EntityShootBowEvent. Set some metadata on arrows that were fired from specified bow.
2) Listen for EntityDamageByEntityEvent. If damager is arrow, then check if there is a metadata and if there is, then change damage.
And your error is because you're checking if damager is arrow
if (damager instanceof Arrow) {
and immediately casting damager to player:
ItemStack i = ((Player) damager)

 

Fixed it thanks!