I am attempting to repair the sword that the player is holding on a client event.

Here is my code:

PlayerEntity player = Minecraft.getInstance().player;
ItemStack heldItem = player.getHeldItemMainhand();

if(heldItem.getItem() == Items.DIAMOND_SWORD){
    LOGGER.info(heldItem.getDamage());

    //repair item
    heldItem.damageItem(-10, player,PlayerEntity::livingTick);

    LOGGER.info(heldItem.getDamage());
}

 

This outputs:

14

14

into the console, showing that the damage value is not being changed. This doesn't work for positive damage values either. 

 

I assume that I am just using the wrong thing for the 3rd parameter (of damageItem), which wants a Consumer<PlayerEntity> type. I am not sure what  a Consumer<PlayerEntity> is or how to use it. As such, I am not sure if this is a problem of my Java ability, or my understanding of the Forge API.

 

Thanks for any help!

 

Edited August 7, 20205 yr by NathaFred

Okay, I assume there is a way to transfer the task to the server. Or in other words modify the inventory from the server, when a client event runs. Or would I have to use a server event?

1 hour ago, NathaFred said:

Or would I have to use a server event?

Use a server event. Do not trust the client. The client is a lying cheating bastard.

So it is! XD So after a number of iterations I finally fixed part of my problem. I found that any Event not located under the .client namespace is a server event. So I just had to make sure event.getWorld().isRemote was false. The problem I have now is that item.damageItem() does not work whereas item.setDamage does. I am currently using setDamage and just adding the value that way, which is arguably simpler, but I would like to know why damageItem was failing. My updated code is here running on a server event:

PlayerEntity player = event.getPlayer();
if(player.getHeldItemMainhand().isDamageable()) {

    Consumer<PlayerEntity> pl = (p) -> LOGGER.info(p);

    LOGGER.info(player.getHeldItemMainhand().getDamage());

    //this line of code works
    //player.getHeldItemMainhand().setDamage(player.getHeldItemMainhand().getDamage() - 8);
    
    //this one does not
    player.getHeldItemMainhand().damageItem(-8,player,pl);

    LOGGER.info(player.getHeldItemMainhand().getDamage());
}

Edited August 7, 20205 yr by NathaFred

1. Yeah I figured that was wrong. I should get into the habit of searching the vanilla code for usage examples.

2. I am repairing the item, so that was intended.

 

It still does nothing even with the same implementation as vanilla...

if (heldItem.isDamageable()) {

    heldItem.damageItem(8, player,
            (p_220017_1_) -> {p_220017_1_.sendBreakAnimation(Hand.MAIN_HAND);}
            );

    LOGGER.info(heldItem.getDamage());
}

 

I kept the same Consumer name as vanilla just in case but that doesn't change anything. I am not sure what is wrong, as it was working for a few minutes seemingly. However, I now cannot get it to do anything... 

Edited August 8, 20205 yr by NathaFred
Everything I said in this message before was wrong...