Trying to count items...

Hey guys
What am I doing wrong here?

PHP:

public class Items {
    public static int countItems(Player player, Material material, short damageValue) {
        PlayerInventory inventory = player.getInventory();
        ItemStack[] items = inventory.getContents();
        int amount = 0;
        for (int i=0; i<items.length; i++) {
            if (items[i].getType() == material && items[i].getDurability() == damageValue) {
                amount += items[i].getAmount();
            }
        }
        return amount;
    }
}


NullPointerException at the if-statement.

Thanks!

Also, this is where I call the method.

PHP:

ItemStack items = new ItemStack(Material.OBSIDIAN, 64);
if (Items.countItems(player, items.getType(), items.getDurability()) >= items.getAmount()) {


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

 

empty slots are null

 

Shouldn't items[i].getType() == material just return false then?
If not, what would the best way solving this be?

 

no, you cannot call .getType() on null (thats why it is throwing a NPE )

Code:java

public class Items 
{
    public static int countItems(Player player, Material material, short damageValue)
    {
        PlayerInventory inventory = player.getInventory();
        ItemStack[] items = inventory.getContents();
        int amount = 0;
        for (ItemStack item : items) 
        {
            if (item.getType() == material && item.getDurability() == damageValue) {
                amount += item.getAmount();
            }
        }
        return amount;
    }
}

a foreach loop should not loop the null values

 

Ok, thanks for the help