Checking if there is room in a players inventory for an item

My method for getting whether a players inventory has room for an item.
It throws a NPE on line 7.

(reason I'm checking display names is incase I have a custom item)

Code:java

    public static boolean isItemTakeable(Player player, ItemStack item){
 
        ItemStack[] contents = player.getInventory().getContents();
 
        for (int i = 0; i < contents.length; i++){
 
            if (contents[i].getType().equals(item.getType())){
                if (!contents[i].getType().equals(Material.AIR)){
                    if (contents[i].getItemMeta().getDisplayName().equals(item.getItemMeta().getDisplayName())){
                        if (!(contents[i].getAmount() == contents[i].getMaxStackSize())){
                            return true;
                        }
                    }
                }
 
            }
 
            if (contents[i].getType().equals(Material.AIR)){
                return true;
            }
        }
        return false;
    }[/i][/i][/i][/i][/i][/i]

please ignore the tags which are being inserted by all the calls in the code...
Thanks

 

Scullyking Instead of "if (!contents.getType().equals(Material.AIR))" try "if (!contents == null))"

 

JRL1004

My bad the error was on line 7 not 9. Your suggestion gives same error.

 

A better way to do this is

Code:java

if(p.getInventory().firstEmpty() == (-1)){
// No Room
}else{
// There Is Room
}

If the inventory's full, .firstEmpty() will return -1 as, obviously, the inventory's full.
Otherwise, it'll return the inventory's first available slot.

As for your code, there will obviously be errors. ItemStack[] will have null slots in them.
You cannot do null.getType() as, well... you figure that out.
Your first check should be

Code:

if (contents[i] != null) {
}

Hope this has helped. Tag me if you have any questions.

 

It returns null because contents is null meaning there is nothing in that slot. Check before hand if contents is null because when you try to use getType() you will get a NPE.

edit: got sniped xD

 

stormneo7 full as in no room at all? what if I want to add a torch, there are no empty slots but there are some torches already.

 

That's different. .firstEmpty() only refers to the first slot available that has no items in it.
Say that he had 1 torch and you want to give him 63 while the rest of the inventory is full.
Using .firstEmpty() here will not benefit well.