Solved How do you check if a player is holding x item?

Code:

@EventHandler
    public void onSwitch(PlayerItemHeldEvent e)    {
        final Player p = e.getPlayer();
        if (p.getItemInHand().getItemMeta().getLore().contains(ChatColor.GREEN + "Miner")){
            p.sendMessage("k");
             p.addPotionEffect(new PotionEffect(PotionEffectType.FAST_DIGGING, 2000000000, 1));
        }
        else{
            p.removePotionEffect(PotionEffectType.FAST_DIGGING);
        }
           
    }

It gives the user the potion effect when they move off of the item but not when they are on the item

 

Instead of getting the lore, get the name. Then scan for a right click action. This would just, in my opinion, make it better.

 

Agreed. Also, I don't mind if they hold or mine. And this is a matter of opinions but I like to use "equals" or more perferablly (with strings), "equalIgnoreCase" when I check for matching variables.

 

No need to declare your player final. Also, check first if the item has meta and has lore before using them. It is gonna spam your console to death.