ItemInHand not detecting?

.getItemInHand() only detects items at full durability.

I am trying to change the damage value of a custom weapon.
I don't know if there are any better ways to change damage values, but I'm guessing not.

By using EntityDamageByEntityEvent, I did .setDamage(4); after detecting that what the player had in his/her
hand was that custom weapon. But it only damaged once when it was full durabilty, and not do any damage afterwards.

I've tried setting the durability of the weapon to full after every hit, but it works occasionally. I've also done up a BukkitTask that runs 1 second after the hit, to see if it's actually regenning the durability.

All I need is to set this exact weapon's damage to this exact damage value. If possible, cancelling the damage event of any other item, so that the player can only damage mobs when the item in his/her hand is a weapon. (Wood-Diamond Sword, Wood-Diamond Axe);

help plox

 

xigsag
how do you check the item in hand?

 

Tarestudio

Code:java

player.getItemInHand()
 
//or
 
player.getInventory().getItemInHand()

 

xigsag I think its this one.

Code:java

player.getInventory().getItemInHand()

 

historio
inb4, I already know what the method is to get the item in the player's hand. The thing is, it only detects it if its at full durability.

Code:java

p.getInventory.getItemInHand();
// and
p.getItemInHand();
 
// work pretty much the same btw.
 
// let's say i want to check if the item in the player's
// hand is the item "Excalibur()".
 
if(p.getItemInHand.equals(Excalibur())){
e.setDamage(4);
}else{
e.setCancelled(true);
}

That only detects when Excalibur() is at full durability.

 

xigsag
Well i think a ItemsStack only equals if its data equals as well. Durability is saved as data, so your Exkalibur() is always at full durability, the item in hand not.
Change your check to equal Type/Material and needed metadata.

 

Tarestudio
Well shit, I forgot I could check for

Code:java

p.getItemInHand().getItemMeta().equals(Excalibur().getItemMeta());

Thanks!

Alright, now that I've got that check working, how do I check if the item in hand is not empty? Because it gives me an error every time I try to check for an empty hand's meta data, which obviously isn't going to work.

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

 

!p.getiteminhand null bla bla

 

Samthelord1
Lol ok ty