Remove items from an inventory

If you want to perform transactions between players or simply want to restrict certain items, you may have to remove a certain amount of a certain type from an inventory. The standard remove function only removes actual stacks, and only one stack at a time. This function can solve that issue:

Code:

    public static void removeInventoryItems(Inventory inv, ItemStack item) {
        removeInventoryItems(inv, item.getType(), item.getAmount());
    }
    public static void removeInventoryItems(Inventory inv, Material type, int amount) {
        for (ItemStack is : inv.getContents()) {
            if (is != null && is.getType() == type) {
                int newamount = is.getAmount() - amount;
                if (newamount > 0) {
                    is.setAmount(newamount);
                    break;
                } else {
                    inv.remove(is);
                    amount = -newamount;
                    if (amount == 0) break;
                }
            }
        }
    }

You can add an inventory contains call before this function to ensure the player has all items you wish to remove.

 

Is there a list of all the item names in java?

 

Item names consist of a type (Material) and the durability member. I see this function does lack a durability check for colored wool, this could be a problem. For general item types, look in the Material enumeration.

Code:

ItemStack item = new ItemStack(Material.WOOD, 5);

 

@bergerkiller
Hey there, I'm getting an error in eclipse on

Code:

      removeInventoryItems(inv, item.getType(), item.getAmount());

on removeInventoryItems
I'm not exactly sure what I'm doing wrong. My plugin, with the source in the jar:
<Edit by Moderator: Redacted mediafire url>

Mind taking a look?

 

@Specops343You have removeInventoryItems(Inventory, ItemStack) in ExtendedPvp and removeInventoryItems(Inventory, Material, int) in ExtendedPvpPlayerListener. Move one of them into the same class as the other.

 

BK! You can help me with making tutorials bud
Is this what you used on Infected?
Love that your helping others <3

 

OK I don't quite understand? Where do I actually get to specify the item? LOL I'm new to bukkit development and new to Java but I do understand it. Trust me! I am just wandering how could I specify the removal of 20 diamonds?

 

The above code contains an error or Bukkit does altogether. If there are two of the same items with same quantity both will be removed. Example: 3 stacks of 20 diamonds. By removing the first stack all 3 are removed since they are the same item type & quantity. This will not happen if you attempt to remove 3 stacks of diamonds with all different quantities. Why?

 

Fixed.

Code:java

package us.zsugano.itemsave.utils;
 
import org.bukkit.Material;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
 
public class InventoryUtil {
 
    public static void removeInventoryItems(Inventory inv, ItemStack item) {
        removeInventoryItems(inv, item.getType(), item.getAmount());
    }
 
    public static void removeInventoryItems(Inventory inv, Material type, int amount) {
        ItemStack[] items = inv.getContents();
        for (int i = 0; i < items.length; i++) {
            ItemStack is = items[i];
            if (is != null && is.getType() == type) {
                int newamount = is.getAmount() - amount;
                if (newamount > 0) {
                    is.setAmount(newamount);
                    break;
                } else {
                    items[i] = new ItemStack(Material.AIR);
                    amount = -newamount;
                    if (amount == 0) break;
                }
            }
        }
        inv.setContents(items);
    }
 
    public static int getTotalItems(Inventory inv, Material type) {
        int amount = 0;
        for (ItemStack is : inv.getContents()) {
            if (is != null) {
                if (is.getType() == type) {
                    amount += is.getAmount();
                }
            }
        }
        return amount;   
    }
 
}[/i][/i]

Oh and sorry for necro bumping, but I felt like it needed to be updated to work correctly

EDIT by Moderator: merged posts, please use the edit button instead of double posting.