Adding +1 Enchant to a Item

Hello,
So I am making a enchantment plugin, And I need to add +1 to the current enchant. For example if you have Sharpness 1 on a sword and you type /sharpness it will add +1 to the sharpness of the Sword in your hand, How would I add the +1 enchantment?

Thanks

 

Can you show the current code that you have, to maybe give us a better idea at what you're trying to achieve?

 

FerusGrim Alright So I am giving the players a Pickaxe and I when they click the Item in the GUI I want it to add +1 Efficiency to the Pick (So if the pick(Item in hand) has Efficiency 1 when they click the item in the GUI it will be Efficiency 2)

Code:java

 if (event.getCurrentItem() != null) {
if (event.getCurrentItem().hasItemMeta()) {
if (event.getCurrentItem().getItemMeta().getDisplayName().equals("+1 Efficiency")) {
event.setCancelled(true);
player.getItemInHand().addEnchantment(Enchantment.DIG_SPEED, 1); //Just for now

 

int level = 0;
if (item.getEnchantments().containsKey(Enchantment.DIG_SPEED)) level = item.getEnchantmentLevel(Enchantment.DIG_SPEED);
item.addUnsafeEnchantment(Enchantment.DIG_SPEED, level + 1);

 

mrCookieSlime I tried that and it did not seem to work. Here is what I have,

Code:java

 
ItemStack Pick = new ItemStack(Material.DIAMOND_PICKAXE);
ItemMeta PickMeta = Pick.getItemMeta();
 PickMeta.setDisplayName("§cImmortal§4Factions§8: §7§o(Right Click To Upgrade)");
Pick.setItemMeta(PickMeta);
 
 
@EventHandler
public void invClick(InventoryClickEvent event) {
Player player = (Player) event.getWhoClicked();
int level = 0;
 
if (event.getCurrentItem() != null) {
if (event.getCurrentItem().hasItemMeta()) {
if (event.getCurrentItem().getItemMeta().getDisplayName().equals("+1 Efficiency")) {
event.setCancelled(true);
if (Pick.getEnchantments().containsKey(Enchantment.DIG_SPEED))
level = Pick.getEnchantmentLevel(Enchantment.DIG_SPEED);
Pick.addUnsafeEnchantment(Enchantment.DIG_SPEED, level + 1);

 

diamondcodes
You forgot a null-check.
if (event.getCurrentItem().getItemMeta().hasDisplayName()
Also did you make sure that your Item is called +1 Efficiency without any Colors.
Otherwise you have to do this:

Code:java

if (ChatColor.stripColor(event.getCurrentItem().getItemMeta().getDisplayName()).equals("+1 Efficiency")) {

 

mrCookieSlime Heres my whole code now Still does not seem to work

Code:java

package me.dc.main;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.java.JavaPlugin;
 
/**
* Created by Matthew on 2014-10-19.
*/
public class Main extends JavaPlugin implements Listener {
 
    public static Inventory PickUpgrade = Bukkit.createInventory(null, 18, "§bPick Upgrade");
 
    public static ItemStack Pick = new ItemStack(Material.DIAMOND_PICKAXE);
    public static ItemMeta PickMeta = Pick.getItemMeta();
 
    public static ItemStack Eff = new ItemStack(Material.DIAMOND);
    public static ItemMeta EffMeta = Eff.getItemMeta();
 
    @Override
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
    }
 
 
    @EventHandler
    public void onJoin(PlayerJoinEvent event) {
        Player player = event.getPlayer();
 
        PickMeta.setDisplayName("§cImmortal§4Factions§8: §7§o(Right Click To Upgrade)");
        Pick.setItemMeta(PickMeta);
 
        player.getInventory().addItem(Pick);
    }
 
    @EventHandler
    public void onClick(PlayerInteractEvent event) {
        Player player = event.getPlayer();
        Action action = event.getAction();
 
 
        EffMeta.setDisplayName("+1 Efficiency");
        Eff.setItemMeta(EffMeta);
 
        if (player.getItemInHand() == Pick && action == Action.RIGHT_CLICK_BLOCK || action == Action.RIGHT_CLICK_AIR || action == Action.RIGHT_CLICK_BLOCK) {
 
            PickUpgrade.setItem(0, Eff);
            player.openInventory(PickUpgrade);
        }
    }
 
    @EventHandler
    public void invClick(InventoryClickEvent event) {
        Player player = (Player) event.getWhoClicked();
        int level = 0;
 
        if (event.getCurrentItem() != null) {
            if (event.getCurrentItem().hasItemMeta()) {
                if (ChatColor.stripColor(event.getCurrentItem().getItemMeta().getDisplayName()).equals("+1 Efficiency")) {
                    event.setCancelled(true);
                    if (Pick.getEnchantments().containsKey(Enchantment.DIG_SPEED))
                        level = Pick.getEnchantmentLevel(Enchantment.DIG_SPEED);
                    Pick.addUnsafeEnchantment(Enchantment.DIG_SPEED, level + 1);
                }
            }
        }
    }
}
 

 

diamondcodes
What exactly is not working?
Any Errors?
Also, there are a couple of mistakes.

Code:java

if (player.getItemInHand() == Pick && action == Action.RIGHT_CLICK_BLOCK || action == Action.RIGHT_CLICK_AIR || action == Action.RIGHT_CLICK_BLOCK) {

should better be

Code:java

if (player.getItemInHand().isSimiliar(Pick) && (action == Action.RIGHT_CLICK_BLOCK || action == Action.RIGHT_CLICK_AIR)) {

An Item will never be == another Item. They can only be similiar, never the same.
And the () are important...
Just like you learned it in Math, * before - except there are ()

Also, you wanna put in this null-check as well to prevent NPEs:
if (event.getCurrentItem().getItemMeta().hasDisplayName()

 

mrCookieSlime Thanks, It opens the GUI when I click the pick, But when I click the diamond(+1 Efficiency) It does nothing
No errors

 

Because you are adding the Enchantment to an ItemStack, not to the Players Pickaxe.

It is like having 2 apples and eat 1 One. The other one is still not eaten.

You wanna set the Item in the Players Hand to the Enchanted Pick, but first check if he is actually holding the Pick, otherwise people will be able to abuse it for duplication purposes.

Also, please remove the public static in front of your Variables...
It causes memory leaks if you are not nullifying them onDisable. Also since you are accessing them from the same Class, it is unnecessary.