EnchantmentUtil

So its a bit difficult to work with enchantments and user input because all the enum values are a different name than what is used in game. For example, feather falling is referred to as PROTECTION_FALL.

Here is a utility class that will make it simpler to get enchantments from a string.

What the getEnchantment(String) method does is it accepts user input, like "feather-falling" and replaces the hypen with an underscore and capitalizes it. It first tries to see if you entered the enum value, like "protection-fall" and if not, it will check against the hashmap to see if it is one of the ingame names. If so, it will return the enum enchantment.

In your own code, do a null check against the return value. If its null, then there was user error. If not, then you have the enchantment ready.

There is also a method, getEnchantments(), which returns a list of strings, with values like "feather-falling" and "fire-protection". You can do what you want with that list, like combine it into a string and send it to the player.

Code:java

package pack.pack;
 
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
 
import org.bukkit.enchantments.Enchantment;
 
public class ItemUtil {
 
    private static HashMap<String, Enchantment> enchants = new HashMap<String, Enchantment>();
 
    static {
        enchants.put("SHARPNESS", Enchantment.DAMAGE_ALL);
        enchants.put("POWER", Enchantment.ARROW_DAMAGE);
        enchants.put("FIRE_PROTECTION", Enchantment.PROTECTION_FIRE);
        enchants.put("FEATHER_FALLING", Enchantment.PROTECTION_FALL);
        enchants.put("PROTECTION", Enchantment.PROTECTION_ENVIRONMENTAL);
        enchants.put("BLAST_PROTECTION", Enchantment.PROTECTION_EXPLOSIONS);
        enchants.put("PROJECTILE_PROTECTION", Enchantment.PROTECTION_PROJECTILE);
        enchants.put("RESPIRATION", Enchantment.OXYGEN);
        enchants.put("INFINITY", Enchantment.ARROW_INFINITE);
        enchants.put("AQUA_AFFINITY", Enchantment.WATER_WORKER);
        enchants.put("UNBREAKING", Enchantment.DURABILITY);
        enchants.put("SMITE", Enchantment.DAMAGE_UNDEAD);
        enchants.put("BANE_OF_ANTHROPODS", Enchantment.DAMAGE_ARTHROPODS);
        enchants.put("EFFICIENCY", Enchantment.DIG_SPEED);
        enchants.put("FIRE_ASPECT", Enchantment.FIRE_ASPECT);
        enchants.put("SILK_TOUCH", Enchantment.SILK_TOUCH);
        enchants.put("FORTUNE", Enchantment.LOOT_BONUS_BLOCKS);
        enchants.put("LOOTING", Enchantment.LOOT_BONUS_MOBS);
        enchants.put("PUNCH", Enchantment.ARROW_KNOCKBACK);
        enchants.put("FLAME", Enchantment.ARROW_FIRE);
    }
 
    public static Enchantment getEnchantment(String ench) {
        ench = ench.toUpperCase().replace('-', '_');
        Enchantment e = Enchantment.getByName(ench);
        if(e == null) {
            if(enchants.containsKey(ench)) {
                e = enchants.get(ench);
            } else {
                if(e == null) {
                    return null;
                }
            }
        }
        return e;
    }
 
    public static List<String> getEnchantments() {
        List<String> list = new ArrayList<String>();
        for(String key:enchants.keySet()) {
            list.add(key.toLowerCase().replace('_', '-'));
        }
        return list;
    }
 
}

 

Thanks for this! I ended up making something like this a while back in one of my plugins because it got really annoying ._.

 

In the else after you check if enchants.containsKey, is there any reason why you're using == null again, after you already use it higher up in your code?

 

macguy8
I had some more code after that in my own application, but had forgotten to remove it for this, I'll fix that