eating glistering melons

I got it all set up how I want it so that it will give you some hearts and none of your hunger bar but my one problem is that it does not consume the glistering melon heres the code

Code:java

    @EventHandler
    public void OnPlayerSoup(PlayerInteractEvent event){
        Player player = event.getPlayer();
        if(player.getHealth() == 20){
        }else{
            int smelon = +4;
            if((event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() ==Action.RIGHT_CLICK_BLOCK) && player.getItemInHand().getType() == Material.SPECKLED_MELON){
                player.setHealth(player.getHealth() + smelon > player.getMaxHealth() ? player.getMaxHealth() : player.getHealth() + smelon);
                event.getPlayer().getItemInHand().setType(Material.AIR);

 

monkeymanboy

Does anything happen at all? Do you get the hearts?

 

yes

 

monkeymanboy

what if you do:

player.setItemInHand(null);

Does that remove the melon from your hand?

 

Replace

Code:

event.getPlayer().getItemInHand().setType(Material.AIR);

with

Code:

ItemStack melon = event.getPlayer().getItemInHand();
        melon.setAmount(1);
        event.getPlayer().getInventory().removeItem(melon);
        event.getPlayer().updateInventory();

 

yes but if it is a stack at all then it won't get rid of it or even restore the hearts

 

monkeymanboy

you should replace player.getItemInHand().setType(Material.AIR) with player.setItemInHand(null)

That will get rid of it, and it's after you've given the player the hearts.

And later, when you want to handle stacked glistering melons, you decrement the itemstack amount, and if the number prior to the decrement is 1, then do player.setItemInHand(null)

 

ok its all working but one more problem it will consume all stacked in my hand

 

monkeymanboy

Using that code:

ItemStack melon = event.getPlayer().getItemInHand();
if(melon.getAmount() > 1)
{

melon.setAmount(melon.getAmount()--);​

event.getPlayer().updateInventory();​

}
else
{

event.getPlayer().getInventory().removeItem(melon);​

event.getPlayer().updateInventory();​

}

I still think making an ItemStack is unnecessary. Could potentially cause problems later on...

 

ok its fine now just one small thing and if theres no easy solution i don't mind much if its a stack of 1 and you have another stack more than one it will consume one from the multiple stack one

 

Code:

ItemStack melon = event.getPlayer().getItemInHand().clone();
        melon.setAmount(1);
        event.getPlayer().getInventory().removeItem(melon);
        event.getPlayer().updateInventory();

That should work fine.