Mushroom soup heal help!

(p.getItemInHand().getTypeId()== Material.MUSHROOM_SOUP.getId())

getTypeId() and getID() are yellow and i dont now how i can fix it ?

 

getId() and getTypeId() are deprecated.

Instead, use their Material enum names:

if (p.getInventory().getItemInHand() == Material.MUSHROOM_SOUP) {}

 

I think you need a getType() in the above poster's code, since getItemInHand returns an ItemStack, and you want it's Material.

 

Maulss
it's p.getItemInHand().getType(), not p.getInventory().getItemInHand()

MrZancrow
All Item Id's and Data values are deprecated (Minecraft is moving away from magic numbers)

EDIT: Ninja'd by NathanWolf

 

MrZancrow NathanWolf Maulss This is what i did.

Code:java

  @EventHandler
  public void onRightClick(PlayerInteractEvent e)
  {
    Player p = e.getPlayer();
    if (p.getHealth() != 20.0D) //ARGUMENT SO IF HEALTH IS 20 THEN IT WILL NOT EAT THE SOUP
    {
      int soup = 7;
      if ((e.getAction() == Action.RIGHT_CLICK_AIR) || ((e.getAction() == Action.RIGHT_CLICK_BLOCK) && (p.getItemInHand().getType() == Material.MUSHROOM_SOUP))) { //CHECKS TO SEE IF ITEMINHAND IS MUSHROOM_SOUP
        p.setHealth(p.getHealth() + soup > p.getMaxHealth() ? p.getMaxHealth() : p.getHealth() + soup); //CHECKS TO SEE IF HEALTH IS GREATER OR LESS THAN 20 AND IF LESS THAN IT SETS IT TO 20
        p.getItemInHand().setType(Material.BOWL); // ONCE TAKEN IT MAKES THE MUSHROOM_SOUP INTO A BOWL
      }
    }
  }
}