Jump to content

[Forge 1.11.2] "Dynamic" item lore that shows a (custom) NBT tag?


SnakeBlock

Recommended Posts

I have an item with a custom NBT tag called "ConsumedAmount". This works, but I also want some form of a 'dynamic' lore, that shows the NBT tag. Like "Amount: >ConsumedAmount<". You can increase the NBT tag (integer) by 1 every time you right click using an overridden onItemRightClick. I've tried to convert the integer into a string and concatenate it with "Amount: " in my addInformation, but it stays at 0.

 

I've tried to change the NBT tag manually to 1000 from the code as a debug inside of the addInformation, and it appears to 'flicker' between 1000 and 0 rapidly. But it doesn't show any changes from outside the addInformation and in the onItemRightClick at all.

 

I've been trying to get this to work for a while now, but no dice. It stays at 0. Any help?

 

    @Override
    public void addInformation(ItemStack par1ItemStack, EntityPlayer par2EntityPlayer, List par3List, boolean par4)
    {
    	int ConsumedAmountNBT = (SelfStack.getTagCompound().getInteger("ConsumedAmount"));
    	
        par3List.add("\u00A77Hold right click to consume \u00A7fIron Ingots\u00A77, and gain buffs.");
        par3List.add("");
        par3List.add("\u00A7eAmount consumed: \u00A76" + Integer.toString(ConsumedAmountNBT));
    }

 

I can send the entire code including the onItemRightClick if necessary (just ask).

Link to comment
Share on other sites

6 minutes ago, SnakeBlock said:

int ConsumedAmountNBT = (SelfStack.getTagCompound().getInteger("ConsumedAmount"));

What is SelfStack, where are you getting that from? If you want to add information based on the NBT of the stack in question, you need to get the NBT from the stack in question - the one the method gives you.

  • Like 1
Link to comment
Share on other sites

Not to mention, Items are Singletons.

There exists only 1 of each item. Any data stored in the Item class, will be global for each ItemStack containing said Item.

The way you intend to work using this SelfStack will thus not work.

 

You are given the correct ItemStack in almost all possible method parameters (here, it's par1ItemStack¹), either directly, or if need ever be, through the EntityPlayer.

 

¹Please edit your parameter names as well. par1, par2...par27 looks horrible and are easily mistaken for each other by the human eye.

  • Like 1

Also previously known as eAndPi.

"Pi, is there a station coming up where we can board your train of thought?" -Kronnn

Published Mods: Underworld

Handy links: Vic_'s Forge events Own WIP Tutorials.

Link to comment
Share on other sites

2 hours ago, Matryoshika said:

Not to mention, Items are Singletons.

There exists only 1 of each item. Any data stored in the Item class, will be global for each ItemStack containing said Item.

The way you intend to work using this SelfStack will thus not work.

 

You are given the correct ItemStack in almost all possible method parameters (here, it's par1ItemStack¹), either directly, or if need ever be, through the EntityPlayer.

 

¹Please edit your parameter names as well. par1, par2...par27 looks horrible and are easily mistaken for each other by the human eye.

So. I've been playing around with your suggestion, along with some extra help from Google.

https://pastebin.com/Xfh8fp3F

 

I've removed the onUpdate thing and did this instead. Although, Eclipse gives me an error...

"The method onItemRightClick(ItemStack, World, EntityPlayer) of type Item_IronInfusedGemstone must override or implement a supertype method"

 

I know why this is happening - it's because the onItemRightClick line should be "public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer p, EnumHand h)", like it is in the method I'm actually overriding.

 

But this raises the question for me, how would I use a method parameter for the right click along with the addInformation? Just doing "World worldIn, EntityPlayer p, EnumHand h, ItemStack stack" gives me an error.

 

I'd imagine I'm probably talking nonsense right now - apologies, seriously. I just want to get this working. :P

Link to comment
Share on other sites

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



×
×
  • Create New...

Important Information

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