Jump to content

Making an item which is both stackable and damageable


Jay Avery

Recommended Posts

I want to make food items which can spoil over time while in a stack. It seems logical to use the vanilla item damage/durability for the spoilage, but damaged items can't be stacked. My first instinct would be to make a subclass of ItemStack and implement my own functionality for combining damage and stack sizes, but since ItemStack is final I can't do that. Does anyone have any ideas for how I could achieve something like this? Or will it be painfully difficult/unachieveable?

 

Edit: update in post #5

Link to comment
Share on other sites

A couple things:

 

1) any item stored in a chest (or other non-entityliving inventory) will not tick and thus not rot.

2) any item in an unloaded chunk (or in a player that is not connected to the server's inventory) will not tick and thus not rot.

3) how do you plan to handle combining a stack of "damage:1" meat with "damage:10" meat?

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

1) I'm planning to create custom chest TileEntities, so I will be able to add ticking for that anyway.

2) I understand that, and I'll either introduce workarounds (instant rot when a chunk becomes re-loaded, perhaps) or just let it go.

3) Some sort of averaging process - something like how spoilage levels are combined in the game Don't Starve.

 

I know it's a big job and I might end up dropping it, but I'd like to give it a try if I can.

Link to comment
Share on other sites

Hi

 

I think it shouldn't be too difficult.  Just add a few custom methods to your Item class (not the ItemStack) and Item renderer to implement the effects of rotting.  For example, instead of using the ItemStack durability bar to indicate rotting, draw a durability bar yourself when rendering the item; or alternatively change the item's appearance another way (eg its colour or a degraded texture).

 

You can work around the ticking problem by storing a timestamp in the Item NBT (or perhaps - as a Capability).

I.e. you have a master clock for the world, when an item is created, you copy the value of your master clock as "creation time".  Later, whenever you render the item or do something with it, check the clock against the saved "creation time" and calculate the amount of rotting.

 

-TGG

Link to comment
Share on other sites

Since I posted, I've found a way to hijack the vanilla durability rendering without needing to use actual item durability - and made a capability to handle my decay process. The problem I'm stuck with now is how to enable combining stacks of the same item that have different levels of decay - I'd really like to be able to make that possible by some kind of averaging process. But the ability to combine stacks seems to mostly rely on

ItemStack.areItemStackTagsEqual

which checks the capabilities. So if I'm storing my decay capability then vanilla will automatically prevent stacks with different decay levels from being combined.

 

I could override all the various stack-combining methods in my inventory containers (I've already got custom containers anyway so that's easily possible), to manually check whether stacks can combine, and then update the stack capability accordingly, but that would be a lot of hassle and probably not totally reliable.

 

Another option is to somehow cheat the NBT check by making my own subclass of

NBTTagCompound

to override the

equals

method and prevent it from checking the values which I want to be irrelevant. But I'm pretty sure that overriding an

equals

method so that it tells an outright lie about whether two objects are equal is some kind of cardinal sin and will probably cause all kinds of problems I haven't thought of.

 

And even if I use one of those workarounds to make sure combining stacks is allowed, I still have the problem of how and where I combine the information from their capabilities to make the new one. It looks like

gatherCapabilities

in the

ForgeEventFactory

is the method that takes the capabilities of two stacks and combines them into one set. But that method uses the

CapabilityDispatcher

constructor which seems to always add the capabilities separately and doesn't have any ability to combine them in some other way.

 

Maybe the ability to combine differently-decayed stacks is just a lost cause. :( It's still pretty good that I've got damageable stacks at all, so perhaps I should just let it go. If anyone else has any ideas or suggestions though, please tell me!

Link to comment
Share on other sites

Since I posted, I've found a way to hijack the vanilla durability rendering without needing to use actual item durability - and made a capability to handle my decay process. The problem I'm stuck with now is how to enable combining stacks of the same item that have different levels of decay - I'd really like to be able to make that possible by some kind of averaging process. But the ability to combine stacks seems to mostly rely on

ItemStack.areItemStackTagsEqual

which checks the capabilities. So if I'm storing my decay capability then vanilla will automatically prevent stacks with different decay levels from being combined.

 

I could override all the various stack-combining methods in my inventory containers (I've already got custom containers anyway so that's easily possible), to manually check whether stacks can combine, and then update the stack capability accordingly, but that would be a lot of hassle and probably not totally reliable.

 

Another option is to somehow cheat the NBT check by making my own subclass of

NBTTagCompound

to override the

equals

method and prevent it from checking the values which I want to be irrelevant. But I'm pretty sure that overriding an

equals

method so that it tells an outright lie about whether two objects are equal is some kind of cardinal sin and will probably cause all kinds of problems I haven't thought of.

 

And even if I use one of those workarounds to make sure combining stacks is allowed, I still have the problem of how and where I combine the information from their capabilities to make the new one. It looks like

gatherCapabilities

in the

ForgeEventFactory

is the method that takes the capabilities of two stacks and combines them into one set. But that method uses the

CapabilityDispatcher

constructor which seems to always add the capabilities separately and doesn't have any ability to combine them in some other way.

 

Maybe the ability to combine differently-decayed stacks is just a lost cause. :( It's still pretty good that I've got damageable stacks at all, so perhaps I should just let it go. If anyone else has any ideas or suggestions though, please tell me!

You could put in a PR for slot click? Then when a slot is clicked handle averaging.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

PR?

Pull Request

 

I don't understand what you mean by pull request here, isn't that a github thing? :S

Forge uses github and accepts pull requests, when they deem them useful.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

Little warning: You cannot by any means extend the ItemStack class as it is final!

 

Yes, I acknowledged this in my original post!

 

I want to make food items which can spoil over time while in a stack. It seems logical to use the vanilla item damage/durability for the spoilage, but damaged items can't be stacked. My first instinct would be to make a subclass of ItemStack and implement my own functionality for combining damage and stack sizes, but since ItemStack is final I can't do that. Does anyone have any ideas for how I could achieve something like this? Or will it be painfully difficult/unachieveable?
Link to comment
Share on other sites

I don't understand what you mean by pull request here, isn't that a github thing? :S

Forge uses github and accepts pull requests, when they deem them useful.

 

Ohh, so you mean suggest a change to Forge that would make it easier for me to do this?

Not a change an addition, you will suggest the event OnSlotClickEvent or some name similar.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

Or you could reduce the number of stages of rot to the point that you don't mind having separate stacks. For instance, have only fresh and tainted before something turns completely into rotting flesh or its vegetable equivalent (dirt?).

 

Along the way, decide if/what will be durable for long journeys, and whether an enchantment (e.g. refrigeration) on a chest (or new device) might stop the rot.

 

And after all that, figure out if the game would still be playable. What if you go on vacation for two weeks while your baked chicken sits in a multiplayer world. Will your character starve to death the next time he logs in?

The debugger is a powerful and necessary tool in any IDE, so learn how to use it. You'll be able to tell us more and get better help here if you investigate your runtime problems in the debugger before posting.

Link to comment
Share on other sites

This is actually the decision I've come to! I'm going to have only three stages of spoiling, so that stacks can be combined if they're in the same stage but not if they're in different stages. As for the rate of spoiling, that'll need plenty of testing to get it calibrated right, but the aim of my mod is a harder survival game anyway. :)

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.