Solved Check if there is enough space in inventory

I currently used Barinade 's code. However, i have empty space, but giving me the error message i don't..

This is what i Got

Code:java

public static void inventoryEmpty(Player player, Material item, int num) {
        ItemStack itemToAdd = new ItemStack(item, num);
        int freeSpace = 0;
        for (ItemStack i : player.getInventory()) {
            if (i == null) {
                freeSpace += itemToAdd.getType().getMaxStackSize();
            } else if (i.getType() == itemToAdd.getType()) {
                freeSpace += i.getType().getMaxStackSize() - i.getAmount();
            }
        }
        if (itemToAdd.getAmount() >= freeSpace) {
            // add items
            player.getInventory().addItem(new ItemStack(item, num));
        } else {
            // not enough space, tell the player and abort mission
            player.sendMessage(Lang.TITLE.toString() + Lang.noSpaceinv);
        }
    }

Code:java

if(clicked.getType() == Material.LOG){
                    int pri = 50;
                    if(pri <= currentPoint){
                        Points.removePoint(player, pri);
                        emptySpaceChecker.inventoryEmpty(player, Material.LOG, 64);
                    }else{
                        player.sendMessage(Lang.TITLE.toString() + Lang.noPoint);
                    }
                }

My inventory is almost empty, but it still gives me the Error

Code:java

 player.sendMessage(Lang.TITLE.toString() + Lang.noSpaceinv);

And I found the bug that it remove the point first.. So is there any way that i can return the method boolean?

 

gidonyou I'd look at http://jd.bukkit.org/rb/doxygen/d3/...entory.html#a4ccbe106e5e95df86cecfa5c6f46750b
Use this and if it doesn't return -1, then it has space. This might be easier for you to do that, and also use a few other methods in that (such as the Inventory#first(ItemStack item) and a few others)

 

Code:java

if( player.getInventory.firstEmpty() != -1)
{
//inv is not full
}
else
{
//inv is full
}

 

Try this.

Code:

    @EventHandler
    public void onBlockBreak(BlockBreakEvent event) {
        event.getBlock().getDrops().clear();
        for(ItemStack is : event.getBlock().getDrops()) {
            HashMap<Integer, ItemStack> rest = event.getPlayer().getInventory().addItem(is);
            if(!rest.isEmpty()) {
                for(int i = 0; i <= rest.size(); i++) {
                    event.getBlock().getDrops().add(rest.get(i));
                }
            }
        }
    }

This was actually for another thread, but the basis is the following: event.getPLayer().getInventory().addItem(is) will try to add the ItemStack wherever it can on the inventory, finding every single space available, and will return what it doesn't find space to add.