Remove item on right click with other item

So I have an event set in which if the player clicks a wooden hoe and he has at least one snowball in his inventory, one snowball gets removed. So if a player has 30 snowballs and he right clicks the hoe, he will only have 29 left.
I was messing around with this for a while but couldn't find a solution.

My Code:

Code:java

if (player.getItemInHand().getType() == Material.WOOD_HOE){
            if(player.getInventory().contains(Material.SNOW_BALL, 1)){
                            player.getInventory.removeItem(new ItemStack(Material.SNOW_BALL, 1));
}else{
player.sendMessage("You don't have enough snowballs");
}
}

 

What is the error/StackTrace?

I notice immediately that you aren't checking if the Action is a right click.

 

yupie_123
You need parentheses after getInventory, because it is a method. If you use a smart IDE, (ex. eclipse) it would point this out.
player.getInventory()...

 

You're right. I didn't immediately catch that, lol.

 

Oh woops, I had it set correctly in my code, I think I messed up copying and pasting.

But what if it doesn't matter if it's a right click or a left click? I want both ways to work. Do I still need to do that?

About the error, well I simply don't get any, it just doesn't remove the snowballs from the inventory.

 

yupie_123
I think I got it.

Code:

player.getInventory.removeItem(new ItemStack(Material.SNOW_BALL, 1));

It removes the ItemStack if it matches with precision, ie both material and count.
Try

Code:

ItemStack item = player.getInventory.first (Material.SNOW_BALL);
item.setCount(item.getCount()-1);

Also, maybe

Code:

player.getInventory().containsAtLeast()

may be better.

Checkout this link.
http://forums.bukkit.org/threads/remove-one-item-from-stack-custom-item.252982/

 

I thought you only wanted right click. In that case, try Hartorn's suggestion.

 

That doesn't seem to work, it says that I need to change the type from ItemStack to int but when I do that I can't use item.setCount

 

yupie_123 : Yeah sorry,

Code:

ItemStack item =  player.getInventory().getItem(player.getInventory().first (Material.SNOW_BALL));

First only returns the index.
Also, better to add a test like that.

Code:

if(item.getCount() ==1)
{
player.getInventory.remove(item)
}
else
{
item.setCount(item.getCount()-1)
}

 

Ok yeah, it all works now, thanks!