So I am creating an event where if a player is holding a certain item and is behind a player, they can perform a "backstab." I figured out everything except how to get the item from the source of damage to make sure they are holding the correct item...
My Code:
 

	@SubscribeEvent
    public void onLivingHurtEvent(LivingHurtEvent event) {
		if (event.getSource().getImmediateSource() != null) {
			if (event.getSource().getImmediateSource().getHeldEquipment() == _Items.itemAquaDagger) {
			    System.out.println("Player With Aqua Dagger Detected");
				double targetYaw = event.getEntityLiving().getRotationYawHead();
				double playerYaw = event.getSource().getTrueSource().getRotationYawHead();
				if(Math.abs(targetYaw - playerYaw) < 80 || Math.abs(targetYaw - playerYaw) > 280) {
					event.setAmount(event.getAmount() * 5);
					event.getEntity().playSound(SoundEvents.BLOCK_ANVIL_HIT, 1.0F, 1F);
				}
			}
		}
    }

I did find .getHeldEquipment() but I am very certain that is not what I want for this case. Btw the system.out.println was just a test to make sure it was the if statement that was the problem.

.getHeldEquipment() returns an ItemStack, it will never be equal to an Item.

Here is some example code that checks if the player is holding a carrot-on-a-stick (found in the EntityPig class).

[Mojang Code Removed]

Notice that this code checks first if the entity is a player and then checks both main hand and off hand. 

 

Edit: Oops, didn't realize I shouldn't be pasting Mojang code here. Totally makes sense though. For anyone seeing this later, just check the EntityPig method "canBeSteered"

Edited September 2, 20187 yr by Icedice9
Removed Mojang Code

29 minutes ago, Draco18s said:

.getHeldEquipment() returns an ItemStack, it will never be equal to an Item.

Well, do you have any other ideas? I used to work on this on 1.10.2 until I had school stuff and came back to this a long while later. In 1.10.2, you could have used .getHeldItemMainhand().getItem() and work from there. However, I guess in 1.12.2 they changed it so you can't use that anymore. Btw, .getHeldEquipment() returns an Iterable<ItemStack> to my knowledge, the code I put up there was just as far as I gotten to.

Edit: IceDice9, grabbing the information from the player would work since I can use .getHeldItemMainhand() from there, but I need to grab it from event to make sure I am getting the source of the damage, not just a random player (I don't even know how that would go on a multiplayer server)

Edited September 2, 20187 yr by Inforno
Answering another reply in edit without having to double post

3 minutes ago, Inforno said:

Btw, .getHeldEquipment() returns an Iterable<ItemStack> to my knowledge, the code I put up there was just as far as I gotten to.

Your comparison will still not be true.

3 minutes ago, Inforno said:

In 1.10.2, you could have used .getHeldItemMainhand().getItem() and work from there.

This would still work...that method still exists.

4 minutes ago, Animefan8888 said:

Your comparison will still not be true.

This would still work...that method still exists.

The method still exists yes, but I spent more time than needed trying to find a way to find and use it. I mean that the method was removed from .getImmediateSource to my knowledge (unless there is some obvious right in front of me way to still use it and wasted an entire hour cause I am blind)

Edited September 2, 20187 yr by Inforno
Grammar error

14 minutes ago, Inforno said:

getImmediateSource

Returns an Entity, getHeldItemMainHand is a method in EntityLivingBase.

10 minutes ago, Animefan8888 said:

Returns an Entity, getHeldItemMainHand is a method in EntityLivingBase.

I wished I was that blind and that worked. Unfortunately, since I am using LivingHurtEvent, EntityLivingBase, in this case, would be the victim. So basically this line:
 

if (event.getEntityLiving().getHeldItemMainhand().getItem() == _Items.itemAquaDagger) {

would mean the victim has to hold the dagger, not the player.
Unless of course there is something other than LivingHurtEvent I can use..

Edited September 2, 20187 yr by Inforno

1 minute ago, Inforno said:

Unless of course there is something other than LivingHurtEvent I can use..

Entity can be cast to EntityLivingBase or EntityPlayer, which has those methods. Also you should use an instanceof check. 

17 minutes ago, Animefan8888 said:

Entity can be cast to EntityLivingBase or EntityPlayer, which has those methods. Also you should use an instanceof check. 

Thank you! Tbh this is my first time using events, instanceof, or casting as somewhat new to java and forge
Im posting my code incase anyone wants it in the future or you double check incase I did something wrong (but it works so eh):
 

	@SubscribeEvent
    public void onLivingHurtEvent(LivingHurtEvent event) {
		if (event.getSource().getImmediateSource() != null) {
			if (event.getSource().getImmediateSource() instanceof EntityPlayer) {
				EntityPlayer player = (EntityPlayer) event.getSource().getImmediateSource();
				if (player.getHeldItemMainhand().getItem() == _Items.itemAquaDagger) {
					double targetYaw = event.getEntityLiving().getRotationYawHead();
					double playerYaw = event.getSource().getTrueSource().getRotationYawHead();
					if(Math.abs(targetYaw - playerYaw) < 80 || Math.abs(targetYaw - playerYaw) > 280) {
						event.setAmount(event.getAmount() * 5);
						event.getEntity().playSound(SoundEvents.BLOCK_ANVIL_HIT, 1.0F, 1.0F);
					}
				}
			}
		}
    }

 

Edited September 2, 20187 yr by Inforno