I created a custom item that extends TieredItem. I want it to be enchantable through the enchantment table with Sharpnes, Looting etc. I can enchant but only the unbreaking enchantment shows up.

There are functions that define this. The default checks what kind of item things are and allows or denies various enchantments.

TieredItem only allows Unbreaking (because that's the default).

@Override
    public boolean canApplyAtEnchantingTable(ItemStack stack, Enchantment enchantment) {
        enchantment = Enchantments.SHARPNESS;
        return enchantment.canApply(stack.getItem().getDefaultInstance());
    }

I tried to override this method. Whenever i put the item in the enchantment table game crashes with java.lang.StackOverflowError.

@Override
    public boolean canApplyAtEnchantingTable(ItemStack stack, Enchantment enchantment) {
        if(enchantment == Enchantments.SHARPNESS) {
            enchantment.canApply(stack);
        }
        return true;
    }

I changed with this but same error.

55 minutes ago, AzizD said:

@Override
    public boolean canApplyAtEnchantingTable(ItemStack stack, Enchantment enchantment) {
        if(enchantment == Enchantments.SHARPNESS) {
            enchantment.canApply(stack);
        }
        return true;
    }

I changed with this but same error.

well obviously, Enchantment#canApply(), calls ItemStack#CanApplyAtEnchantingTable(), which calls Item#canApplyAtEnchantingTable. Therefore, you got yourself an infinite loop...

    @Override
    public boolean canApplyAtEnchantingTable(ItemStack stack, Enchantment enchantment) {
        Enchantment[] enchantments = {
                Enchantments.SHARPNESS,
                Enchantments.LOOTING,
                Enchantments.FIRE_ASPECT,
                Enchantments.UNBREAKING,
                Enchantments.KNOCKBACK,
                Enchantments.MENDING,
        };
        for(int i=0;i<enchantments.length;i++){
            if(enchantments[i] == enchantment){
                return true;
            }
        }
        return false;
    }

 

I wrote this and it worked. Thx.

23 minutes ago, diesieben07 said:

A better way would be to use a Set (probably ImmutableSet in this case).

It looks way better. Thanks for the advice.

@Override
    public boolean canApplyAtEnchantingTable(ItemStack stack, Enchantment enchantment) {
        Set e = ImmutableSet.of(
                Enchantments.SHARPNESS,
                Enchantments.LOOTING,
                Enchantments.FIRE_ASPECT,
                Enchantments.UNBREAKING,
                Enchantments.KNOCKBACK
                );
        return e.contains(enchantment);
    }

 

5 minutes ago, diesieben07 said:

• Please don't use raw types.
• Store the Set in a field, don't create it every time.

I got it.