Right click event (with item name)

Hey guys.
I've had some truble with making a right click event.
I can make a event that does something after you right clicked with for example a piece of sugar.
but i can't require it to have a name. This is my code:

Code:

    @EventHandler
    public void druginjection(PlayerInteractEvent event) {
      
      
        Player player = event.getPlayer();
        if(player.getInventory().getItemInHand().getType() == Material.STICK && player.getItemInHand().getItemMeta().getDisplayName() == "Heroin injection") {
          
            if(event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
      
                player.sendMessage(ChatColor.BOLD + "Hmm that feels good!");
              
                player.addPotionEffect(new PotionEffect(PotionEffectType.ABSORPTION, 100, 2));  
              
                player.setItemInHand(null);
              
            }
          
        }
      
      
      
      
      
    }

 

Are you getting any errors in console? Have you registered your events? I would check to see if Action == your action first before checking the item in hand.

 

Show the code where you made the heroin injection item please.

 

I don't get any errors from the console the only thing it sais is that the plugin is enabled

I don't make the item with code, I'm trying to make it possible that i can make it myself ingame or with a anvil

 

Don't set the Item to Null, That will cause a NullPointerException! You are also using == instead of .equals(); , which is a bad habit of getting into. You use player.getInventory().getItemInHand() instead of player.getItemInHand(); , I would suggest you change that. Are you registering Events? does your class implement Listener?

 

What do you mean by "bad habit"? == is actually preferred when comparing enumerators.

 

Change null to "new ItemStack(Material.AIR);

 

For Strings, sure. For enums, nope.

 

@nverdier @SuperOriginal I meant that .equeals() should be used here;

 

@Zombie_Striker
Ah, didn't see that.

 

I used .equals() and it works perfect now!
Thx so much!