This method to take Gold away not working..

Hey guys. Another problem, someone mind telling me why this doesn't work or fixing it?

Code:

public void rmoney(Player player, int xam) {
        int rem = 0;
        for (ItemStack i : player.getInventory().getContents()) {
            if (i != null) {
                if (i.getType() == Material.GOLD_INGOT) {
                    while (i.getAmount() > 0  || rem < xam) {
                        i.setAmount(i.getAmount() - 1);
                        rem++;
                    }
                }
            }
        }
    }

Thank you in advanced.

 

Well you're not removing the item after it gets to amount 0, you must set it to null.

And instead of while() you should do a basic math calculation instead... something like i.getAmount() - xam and if that is below or equal to 0 then subtract i.getAmount() from rem and set item to null, otherwise set the amount and return because you've removed the amount you needed.

 

Okay, thank you for this.

 

Also if you have multiple stacks of gold this will take 1 from each stack. Shove break; in after rem++; and then after that bracket } put return;

 

Frazz86
No it won't, mind the while()

 

This.
And thanks, I got it fixed up.