[Solved] How to add damage to tools!

Hey. i want to allow players to break iron, lapis, and gold ore with a gold pick, and i want this to happen instantly. That works. What doesn't is the durability doesn't go down. Ideally i would like double damage since its instant mine, and with this particular plugin you can convert wood tools into gold tools. Here the code for the Interact listener (super pick):

Code:java

public class MyGoldListener implements Listener{
    @EventHandler(priority = EventPriority.HIGH)
public void onLeftClick(PlayerInteractEvent e) {
        Player p = (Player)e.getPlayer();
        if(e.getPlayer().getItemInHand().getType() == Material.GOLD_PICKAXE) {
            if(e.getAction() == Action.LEFT_CLICK_BLOCK) {
                if(e.getClickedBlock().getType() == Material.IRON_ORE) {
                    e.getClickedBlock().breakNaturally();
                    p.getItemInHand().setDurability((short) (p.getItemInHand().getDurability() - 2));
                    return;
                    }
                    if(e.getClickedBlock().getType() == Material.GOLD_ORE) {
                        e.getClickedBlock().breakNaturally();
                        p.getItemInHand().setDurability((short) (p.getItemInHand().getDurability() - 2));
                            return;
                        }
                        if(e.getClickedBlock().getType() == Material.LAPIS_ORE) {
                            e.getClickedBlock().breakNaturally();
                            p.getItemInHand().setDurability((short) (p.getItemInHand().getDurability() - 2));
                            return;   
                }               
            }
    }
}}
 

The Block Listener:

Code:java

public class MyBlockListener implements Listener{
    public void onBlockBreak(BlockBreakEvent ev) {
        Block block = ev.getBlock();
        World world = block.getWorld();
        Location loc = block.getLocation();
        BlockState bs = block.getState();
        ItemStack goldpick = new ItemStack(285);
        if(ev.getPlayer().getItemInHand() == goldpick) {
            if(block.getType() == Material.IRON_ORE) {
                block.setType(Material.AIR);
                world.dropItemNaturally(loc, new ItemStack(bs.getType(), 1, bs.getRawData(), bs.getRawData()));
                return;
            }
            if(block.getType() == Material.GOLD_ORE) {
                block.setType(Material.AIR);
                world.dropItemNaturally(loc, new ItemStack(bs.getType(), 1, bs.getRawData(), bs.getRawData()));
                return;
            }
            if(block.getType() == Material.LAPIS_ORE) {
                block.setType(Material.AIR);
                world.dropItemNaturally(loc, new ItemStack(bs.getType(), 1, bs.getRawData(), bs.getRawData()));
                return;
        }
        }
    }
}

 

ItemStack.setDurability(int_here)

 

That will just set the durability to the same thing every time someone breaks a block with the super pick. I need to damage the tool once every time they "left click" or destroy a block.

 

Really...
ItemStack.setDurability(ItemStack.getDurability() - 1)

 

That does not work. Put it in eclipse or netbeans, you will see. It will tell you to cast -1 to short, then its the same thing as what i had.

Could anyone help please?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Why do you cast the entire value to short? Why dont you only cast the 2 to short? I doubt that will fix it, but just saying.

 

You need to add durability, not subtract durability.

 

This is what I'm trying now, what happens is ,it damages 1, but the data++ doesn't add more durability.

PHP:

public class MyGoldListener implements Listener{
    @EventHandler(priority = EventPriority.HIGH)
public void onLeftClick(PlayerInteractEvent e) {
        short data = 1;
        if(e.getPlayer().getItemInHand().getType() == Material.GOLD_PICKAXE) {
            if(e.getAction() == Action.LEFT_CLICK_BLOCK) {
                if(e.getClickedBlock().getType() == Material.IRON_ORE) {               
                    e.getClickedBlock().breakNaturally();               
                        e.getPlayer().getInventory().getItemInHand().setDurability(data) ;
                        data++;
                        return;
                    }                   
                    }           
            else if(e.getClickedBlock().getType() == Material.GOLD_ORE) {
                        e.getClickedBlock().breakNaturally();
                            e.getPlayer().getInventory().getItemInHand().setDurability(data) ;
                            data++;
                            return;
                        }   
            else if(e.getClickedBlock().getType() == Material.LAPIS_ORE) {
                            e.getClickedBlock().breakNaturally();
                                e.getPlayer().getInventory().getItemInHand().setDurability(data) ;
                                data++;
                                return;
                        }
        } else {
            e.setCancelled(false);
        }
    }
}


 

You need to increment the data before setting it to the item. Primitives do not use references.

Edit: Just use the code you had originally, but change the minuses to pluses.

 

EDIT: As simple as it was, your epic. Thanks so much man.