[Solved][1.11] Remove drop from the leaves

[WildHeart]

Hello, how to remove drop from the leaves?

@SubscribeEvent
    public void onDrop(LivingDropsEvent e)
    {
        if(e.getEntity().worldObj.isRemote) return;

        for(EntityItem entityItem : e.getDrops())
        {
            if(entityItem.getEntityItem().getItem() instanceof ItemFood)
            {
                if(entityItem.getEntityItem().getItem() == Items.APPLE)
                {
                }
            }
        }
    }

 

Edited May 8, 2017 by WildHeart

[Jay Avery]

LivingDropsEvent is for living entities that die and drop items (e.g. animals and monsters). For blocks, you can use HarvestDropsEvent.  Check for your chosen block, then edit the list of drops however you want.

[WildHeart]

Hmm, now how to remove specifically apples?

@SubscribeEvent
    public void onDrop(BlockEvent.HarvestDropsEvent e)
    {
        if(e.getWorld().isRemote) return;

        if(e.getWorld().getBlockState(e.getPos()).getBlock() instanceof BlockLeaves)
        {
            e.getDrops()
        }
    }

 

[Draco18s]

Try iterating.

[WildHeart]

13 minutes ago, Draco18s said:

Try iterating.

Maybe?:

        if(e.getWorld().getBlockState(e.getPos()).getBlock() instanceof BlockLeaves)
        {
            Iterator apples = e.getDrops().iterator();

            while (apples.hasNext())
            {
                if(e.getDrops() instanceof ItemFood)
                {
                    e.setCanceled(true);
                    break;
                }
            }
        }

 

[Draco18s]

No.

Try again.

Look at the documentation.

[WildHeart]

            while (apples.hasNext())
            {
                if(apples.next() instanceof ItemFood)
                {
                    apples.remove();
                }
            }

?

[WildHeart]

Solved!

    @SubscribeEvent
    public void onDrop(BlockEvent.HarvestDropsEvent e)
    {
        if(!e.getWorld().isRemote)
        {
            if(e.getWorld().getBlockState(e.getPos()).getBlock() instanceof BlockLeaves)
            {
                e.getDrops().removeIf(itemStack -> itemStack == null || itemStack.getItem() == Items.APPLE);
            }
        }
    }

 

[loordgek]

itemstack cannot be null check for itemstack.isempty

[Draco18s]

There also shouldn't be empty stacks in the drops array.