Jump to content

Recommended Posts

Posted

Hello,

 

I am writing huge combat/weapon/armour API, I've managed to do interfaces that include a bunch of nice wearing/equipping trackers and now trying to add effects whether player is holding something or not.

 

Where can I find a method which tell us which of 9 HotBar slots is a "Held Item" one? Also I am not sure - is this a server side data or local client? (which would be a shame, but i don't think it is).

 

T.Hanks in advance :)

1.7.10 is no longer supported by forge, you are on your own.

Posted

Retarded me :)

 

Found answer just a few topics below mine.

 

ticker + player.getCurrentEquippedItem()

 

Shame on me :P

1.7.10 is no longer supported by forge, you are on your own.

Posted

Well, I wasn't aware of some of the methods in MC code (it has changed a bit over versions and I really don't track every change). Since you may know something more and I'd like to optimalize my code, please tell me what should I rewrite (connect to base MC code).

Interface (only showing 3 that use playerTick())

/**
 * This method is called when player is holding an item.
 */
public void onHeld(ItemStack stack, EntityLivingBase player);

/**
 * This method is called when player is wearing an item in equipment slot.
 */
public void onWorn(ItemStack stack, EntityLivingBase player);

/**
* This method is called when player has an item inside inventory.
*/
public void onCarried(ItemStack stack, EntityLivingBase player);

Handling

@SubscribeEvent
public void playerTick(PlayerEvent.LivingUpdateEvent event) 
//Tracks:
onHeld() //Checks if player.getCurrentEquippedItem() instanceof Interface and does stuff
onWorn() //Scans 12 slots (extended equipment) and checks instanceof
onCarried() //Scans 36 EQ slots + if backpack (custom bp) is present -> scans it too

 

Now: Is thare are way to keep tracking players EQ without scanning up to 70 slots (backpack) every tick?

I was thinking about tracking adding/removing items from inventory, but that will be too pain in the ass (a lot of nulls and unexpected situations).

1.7.10 is no longer supported by forge, you are on your own.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • You can check mod compatibility remove new mods and test them one by one.
    • @Tsuk1 Also, new note, you can use blockbench to make the custom item model for when it is not on the head.   EDIT: Funny story, I am making a mod similar to yours! Mine is called NorseMC.
    • @Nood_dev Could you send a screenshot of your weapon code? Here is the one I made (for a dagger): The specific UUID does not matter, just that it is the same every time, which is why UUID#randomUUID does not work public class DaggerItem extends TieredItem implements Vanishable { protected static final double REACH_MODIFIER = -1.5D; protected final Multimap<Attribute, AttributeModifier> defaultModifiers; protected final UUID BASE_ATTACK_REACH_UUID = UUID.fromString("6fe75b5c-9d1b-4e83-9eea-a1d5a94e8dd5") public DaggerItem(Tier pTier, int pAttackDamageModifier, float pAttackSpeedModifier, Properties pProperties) { super(pTier, pAttackDamageModifier, pAttackSpeedModifier, pProperties); this.attackDamage = (float) pAttackDamageModifier + pTier.getAttackDamageBonus(); ImmutableMultimap.Builder<Attribute, AttributeModifier> builder = ImmutableMultimap.builder(); builder.put(Attributes.ATTACK_DAMAGE, new AttributeModifier(BASE_ATTACK_DAMAGE_UUID, "Weapon modifier", this.attackDamage, AttributeModifier.Operation.ADDITION)); builder.put(Attributes.ATTACK_SPEED, new AttributeModifier(BASE_ATTACK_SPEED_UUID, "Weapon modifier", pAttackSpeedModifier, AttributeModifier.Operation.ADDITION)); // THE ONE YOU WANT: builder.put(ForgeMod.ENTITY_REACH.get(), new AttributeModifier(BASE_ATTACK_REACH_UUID, "Weapon modifier", REACH_MODIFIER, AttributeModifier.Operation.ADDITION)); this.defaultModifiers = builder.build(); } @Override public Multimap<Attribute, AttributeModifier> getDefaultAttributeModifiers(EquipmentSlot pEquipmentSlot) { return pEquipmentSlot == EquipmentSlot.MAINHAND ? this.defaultModifiers : super.getDefaultAttributeModifiers(pEquipmentSlot); } }
    • https://images.app.goo.gl/1PxFKdxByTgkxvSu6
    • That's what we'll try out. I could never figure out how to recreate the crash, so I'll just have to wait and see.
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.