Solved Check for every single content in Inventory

I've made a little piece of code to be able to shoot projectiles with the diamond shovel:

Code:java

 
    @EventHandler
    public void onShoot(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        ItemStack i = p.getItemInHand();
        if (i.getType() == Material.DIAMOND_SPADE) {
            if (e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK) {
                if (i.getAmount() != 1) {
                    Projectile pj = p.launchProjectile(Egg.class);
                    pj.setVelocity(p.getLocation().getDirection().normalize().multiply(2.5));
                    i.setAmount(i.getAmount() - 1);
                } else if (i.getAmount() == 1) {
                    ActionBarAPI.sendActionBar(p, "§cYour §3AK-47 §cis empty, reload it!");
                    return;
                }
            } else if (e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK) {
                if (p.getInventory().contains(Material.CLAY_BALL)) {
                    for (ItemStack item : p.getInventory().getContents()) {
                        if (i.getAmount() == 31) {
                            return;
                        } else if (i.getAmount() < 31) {
                            ItemStack clay = new ItemStack(Material.CLAY_BALL, 30);
                            if (item.getAmount() == clay.getAmount()) {
                                p.getInventory().remove(item);
                                i.setAmount(31);
                                p.updateInventory();
                            } else if (item.getAmount() > clay.getAmount()) {
                                if (i.getAmount() != 1) {
                                    item.setAmount(item.getAmount() - (30 - i.getAmount()));
                                    i.setAmount(31);
                                } else if (i.getAmount() == 1) {
                                    item.setAmount(item.getAmount() - 30);
                                    i.setAmount(31);
                                }
                            } else if (item.getAmount() >= 0 && item.getAmount() < 30) {
                                i.setAmount(i.getAmount() + item.getAmount());
                                p.getInventory().remove(item);
                                p.updateInventory();
                            }
                        }
                    }
                } else {
                    ActionBarAPI.sendActionBar(p, "§cYou have not enough ammo!");
                }
            }
        }
    }
 

It should reload, when i'm left-clicking.

But it only reloads when the "bullets" (Clay balls) are in the first slot of my inventory.


Does anybody has some suggestions for me?

 

Use the first(Material) method to get the first instance of your bullet in the inventory. (first returns an int, the slot where it was found. Use Inventory#get(int) to get the itemstack.)