[SOLVED]Removing 1 item from ItemStack

Ok, so I'm needing to remove items from a player inventory(on click), but the problem is, whenever I click, it removes the entirety of the ItemStack, not just 1 entity. How would I correct this?

EDIT: If code will help, I can give you small snippets of how I'm currently doing it.

Code:

protected boolean ****(Player p, ItemStack inHand, Block clicked)

int blockamnt = inHand.getAmount();
inHand.setAmount(blockamnt--);

 

Try refeering to the Inventory instead of the hand, so like

Code:

protected boolean ****(Player p, ItemStack inHand, Block clicked) {
Item item = p.getIteminHand();
p.remove(new ItemStack(item.getID,1));
p.updateInventory
}

I didn't test it but that is the basic idea behind doing it

 

That is a good idea, but getIteminHand returns an ItemStack, not an item. Also, p.remove doesn't work, pity, that.

 

Like I said, didnt test this xP and my bad, use p.removeItem instead

 

removeItem isn't an available class.

 

You have to put the itemStack stuff in it xD replace remove with removeItem

 

Iv done this couple of weeks ago and it works quite fine.
https://github.com/Gugap/Guga_SERVER_MOD/blob/master/me/Guga/Guga_SERVER_MOD/GugaAuctionHandler.java
This is the line you are are looking for:

Code:

player.getInventory().setItem(i, new ItemStack(itemID, stack.getAmount() - amount));

Just use setItem and decrease the amount here by creating new ItemStack with amount--

 

try sifting through this. this does it!

Code:

if(label.equalsIgnoreCase("item") && (args[0].equalsIgnoreCase("exchange")))
        {
            if(player.getItemInHand().getTypeId() == ItemID)
            {
                if(args.length < 2)
                {
                    player.sendMessage("Not enough arguments");
                    return true;
                }
                else if(args.length > 2)
                {
                    player.sendMessage("Too many arguments");
                    return true;
                }
                int amountSold = Integer.parseInt(args[1]);
                int handAmount = player.getItemInHand().getAmount();
                int giveAmount = handAmount - amountSold;
                Material item = player.getItemInHand().getType();

                if(handAmount < amountSold)
                {
                    player.sendMessage("not enough diamonds!");
                    return true;
                }
                else if(handAmount >= amountSold)
                {
                    if(handAmount == amountSold)
                    {
                        player.setItemInHand(null);
                        return true;
                    }
                    else
                    {
                        player.setItemInHand(new ItemStack(item, giveAmount));
                        return true;
                    }
                }
            }
            else
            {
                player.sendMessage("Please hold "+ ItemID);
            }
        }
        return false;
    }

 

problem is realy simple.

Code:

blockamnt--

tells java to FIRST execute

Code:

inHand.setAmmount(blockamnt)

and THEN decrement blockamnt.
Correct way:

Code:

inHand.setAmmount(--blockamnt)

Java will now FIRST decrement blockamnt and THEN set it

Hope i helped you
Greetings
Djuady

Ps: Sorry... My english seems to be fucked up today... been awake for 48 hours
PPs: And of course you should chek if blockamnt is greater 1, because you don't want to have a stack with ammount 0. Remove it from if the ammount equals 1

 

set it null if its 0 or you crash their client

 

I fixed the issue, thanks for all the help.