[решено]Удаление дропа из блока

[doodka]

Добрый день!
Задача: удалить дроп, который содержится в блоке из мода. Проблема в том, что через getDrops().clear() удаляется только дроп самого блока, а то, что содержалось в нем-нет.
Если просто изменить тип, то из него вылетают вещи.
Сделал такую очистку:

    @EventHandler()
    public void onBlockBreak(BlockBreakEvent e)
    {
        if(e.isCancelled()) { return; }
        e.getBlock().breakNaturally();

        List<Entity> entity = e.getBlock().getWorld().getEntities();
        for(Entity ent : entity) {
            if(ent instanceof Item) {
                if (ent.getLocation().distance(e.getBlock().getLocation()) < 2) {
                    ent.remove();
                }
            }
        }
    }

Есть ли более хороший способ это реализовать?

 

[saharin94]

        List<Entity> entity = e.getBlock().getWorld().getEntities();
        for(Entity ent : entity) {
            if(ent instanceof Item) {
                if (ent.getLocation().distance(e.getBlock().getLocation()) < 2) {
                    ent.remove();
                }
            }

С таким подходом лагов только прибавится.
Есть такой вариант: очищать инвентарь блока перед его разрушением.

 

[Hilgert]

e.setDrop() вроде

 

[doodka]

В BlockBreakEvent нет такого. А block.getDrops().clear() -удаляет только дроп с самого блока.
Мне нужно удалить вещи, которые хранятся в инвентаре блока из мода (дюп).

 

[saharin94]

> Мне нужно удалить вещи, которые хранятся в инвентаре блока из мода (дюп).
Ну так "залезь" в инвентарь блока и удали всё, тогда ничего не выпадет.

 

[doodka]

Так?

    @EventHandler()
    public void onBlockBreak(BlockBreakEvent e)
    {
        if(e.isCancelled()) { return; }

        Block block = e.getBlock();
        if(block.getState() instanceof InventoryHolder) {
            e.setCancelled(true);
            InventoryHolder inventoryHolder = (InventoryHolder) block.getState();
            inventoryHolder.getInventory().clear();
            block.breakNaturally();
        }
    }

 

[saharin94]

Так?

    @EventHandler()
    public void onBlockBreak(BlockBreakEvent e)
    {
        if(e.isCancelled()) { return; }

        Block block = e.getBlock();
        if(block.getState() instanceof InventoryHolder) {
            e.setCancelled(true);
            InventoryHolder inventoryHolder = (InventoryHolder) block.getState();
            inventoryHolder.getInventory().clear();
            block.breakNaturally();
        }
    }

Как бы так. Только не понимаю, зачем отменять эвент и искусственно ломать блок?
p.s. Смотри, чтобы из этого не вылился гриф, так как у тебя ни приоритета, ни игнора отменённых эвентов.

 

[doodka]

Убрал отмену. Приоритеты и проверки блоков напишу.
Спасибо.

 

[NaPsTeR]

Убрал отмену. Приоритеты и проверки блоков напишу.
Спасибо.

А не проще изменить мод и оффнуть в нем дроп или исправить дюп?

 

[Den_Abr]

А не проще изменить мод и оффнуть в нем дроп или исправить дюп?

Не проще

 

[NaPsTeR]

Не проще

Почему?

 

[Den_Abr]

Почему?

1. Работа с говнокодом мода
2. Лишняя работа по декомпиляции/компиляции/обфускации
   
3. Неуниверсальность метода
4. Необходимость переделки каждой новой выходящей версии мода

 

[Jampire]

При разрушении блока для каждого выпавшего предмета ItemSpawnEvent вызывается. Что делать дальше, думаю, объяснять не стоит?