Дублирование предметов

[fedortik]

@EventHandler
    public void onCraftPrepare(PrepareItemCraftEvent event) {
        CraftingInventory inventory = event.getInventory();
        ItemStack[] matrix = inventory.getMatrix();

        if (matrix.length != 9) {return;}
        ItemStack centerItem = matrix[4];
        if (centerItem != null && centerItem.getType().toString().endsWith("_PICKAXE")) {
            boolean validUpgrade = true;
            
            for (int i = 0; i < matrix.length; i++) {
                if (i == 4) continue;
                ItemStack slotItem = matrix[i];

                if (slotItem == null || slotItem.getType() != Material.COBBLESTONE) {
                    validUpgrade = false;
                    break;
                }
            }

            if (validUpgrade) {
                ItemStack upgradedPickaxe = centerItem.clone();
                ItemMeta meta = upgradedPickaxe.getItemMeta();
                if (meta != null) {
                    int enchantLevel = meta.getEnchantLevel(Enchantment.DIG_SPEED) + 1;
                    meta.addEnchant(Enchantment.DIG_SPEED, enchantLevel, true);
                    upgradedPickaxe.setItemMeta(meta);

                    inventory.setResult(upgradedPickaxe);
                }
            }
        }
    }

Имеется такой обработчик события. Казалось бы элементарно просто. Если в центре кирка а вокруг неё 8 булыжника то ставит в результат крафта кирку с эффективностью +1 . Но если один баг. Если положить в верстак
то после крафта оставшийся булыжник становится на 1 меньше(как и должно быть), а затем умножается на 2(Так не должно быть)
На всякий случай уточню, что это единственный слушатель в плагине

 

[fedortik]

Я конечно добавил условие

if (slotItem == null || slotItem.getType() != Material.COBBLESTONE || slotItem.getAmount() != 1) {
                    validUpgrade = false;
                    break;
                }

что бы в слоте верстака был только 1 булыжник. Так после крафта 0 x 2 = 0
но всё же хотелось бы понимать почему так происходит

 

[FeniksDEV]

Это баг, скорее всего единственным верным решением здесь будет использовать ShapedRecipe.
https://github.com/PaperMC/Paper/issues/1822

 

[alexandrage]

Тут явно не весь код, ибо не вижу ничего, что умножает стак.

 

[imDaniX]

Тут явно не весь код, ибо не вижу ничего, что умножает стак.

Как написали выше - баг. Возникает, когда устанавливаешь результат крафта во время PrepareItemCraftEvent.

 

[imDaniX]

Не правда, вон собери мой плагин, он ровно таки же заменяет предмет в PrepareItemCraftEvent.

Выше приведен запрос с гитхаба Paper, проблема присутствует. Сам имел дело, но уже не помню всего контекста. Возможно, связано с отсутствием соответствующего Recipe.
В твоём примеры ты в целом наоборот устанавливаешь результат на воздух - и у ТС, и на гитхабе, случай обратный.

 

[alexandrage]

Выше приведен запрос с гитхаба Paper, проблема присутствует. Сам имел дело, но уже не помню всего контекста. Возможно, связано с отсутствием соответствующего Recipe.
В твоём примеры ты в целом наоборот устанавливаешь результат на воздух - и у ТС, и на гитхабе, случай обратный.

У меня существующий рецепт и предметом в результате да, ну значит пусть регистрирует рецепт как положено, если баг при отсутствие рецепта.

 

[fedortik]

У меня существующий рецепт и предметом в результате да, ну значит пусть регистрирует рецепт как положено, если баг при отсутствие рецепта.

Спасибо вам за ваше компетентное мнение, но задумка в том что разные предметы в том числе и кастомные будут прокачивать разные зачарования. Три зачанования по 10 уровней это уже 1000 рецептов, поэтому в моём случае эффективнее запретить класть больше 1 предмета в слот верстака

 

[alexandrage]

Спасибо вам за ваше компетентное мнение, но задумка в том что разные предметы в том числе и кастомные будут прокачивать разные зачарования. Три зачанования по 10 уровней это уже 1000 рецептов, поэтому в моём случае эффективнее запретить класть больше 1 предмета в слот верстака

Зачем тебе 1000 рецептов? Тебе одного хватит, что бы обойти баг. Закастомишь ивентом и все.