Food Item with changing hunger values

[meee39]

I am making an Electronic Lunchbox mod that uses RF to feed you. I have this code:

package electroniclunchbox.items;

import java.util.List;

import org.lwjgl.input.Keyboard;

import cofh.redstoneflux.api.IEnergyContainerItem;
import electroniclunchbox.ElectronicLunchbox;
import electroniclunchbox.gui.ModGuiHandler;
import net.minecraft.advancements.CriteriaTriggers;
import net.minecraft.client.util.ITooltipFlag;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.EntityPlayerMP;
import net.minecraft.init.SoundEvents;
import net.minecraft.item.ItemFood;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.potion.PotionEffect;
import net.minecraft.stats.StatList;
import net.minecraft.util.ActionResult;
import net.minecraft.util.EnumHand;
import net.minecraft.util.SoundCategory;
import net.minecraft.world.World;

public class ItemElectronicLunchbox extends ItemFood implements IEnergyContainerItem {
	public static final int capacity = 1000000;
	public static final int maxReceive = 10000;
	
	public ItemElectronicLunchbox(int amount, float saturation, boolean isWolfFood) {
		super(amount, saturation, isWolfFood);
	}

	@Override
	public ItemStack onItemUseFinish(ItemStack stack, World worldIn, EntityLivingBase entityLiving) {
		if (entityLiving instanceof EntityPlayer)
        {
            EntityPlayer entityplayer = (EntityPlayer)entityLiving;
            entityplayer.getFoodStats().addStats(this, stack);
            worldIn.playSound((EntityPlayer)null, entityplayer.posX, entityplayer.posY, entityplayer.posZ, SoundEvents.ENTITY_PLAYER_BURP, SoundCategory.PLAYERS, 0.5F, worldIn.rand.nextFloat() * 0.1F + 0.9F);
            this.onFoodEaten(stack, worldIn, entityplayer);
            entityplayer.addStat(StatList.getObjectUseStats(this));

            if (entityplayer instanceof EntityPlayerMP)
            {
                CriteriaTriggers.CONSUME_ITEM.trigger((EntityPlayerMP)entityplayer, stack);
            }
        }

        return stack;
	}

	@Override
	public int getMaxItemUseDuration(ItemStack stack) {
		return super.getMaxItemUseDuration(stack);
	}

	@Override
	public int getHealAmount(ItemStack stack) {
		NBTTagCompound nbt = stack.getTagCompound();
		
		return nbt.getInteger("Hunger");
	}

	@Override
	public float getSaturationModifier(ItemStack stack) {
		NBTTagCompound nbt = stack.getTagCompound();
		
		return nbt.getInteger("Saturation");
	}

	@Override
	public boolean isWolfsFavoriteMeat() {
		return super.isWolfsFavoriteMeat();
	}

	@Override
	public ItemFood setPotionEffect(PotionEffect effect, float probability) {
		return super.setPotionEffect(effect, probability);
	}

	@Override
	public ItemFood setAlwaysEdible() {
		return super.setAlwaysEdible();
	}

	@Override
	public void addInformation(ItemStack stack, World worldIn, List<String> tooltip, ITooltipFlag flagIn) {
		super.addInformation(stack, worldIn, tooltip, flagIn);
		
		if (!stack.hasTagCompound()) {
			return;
		}
		
		NBTTagCompound nbt = stack.getTagCompound();
		
		tooltip.add("Shift-Right Click Me To Edit My Settings");
		tooltip.add("Hunger: " + nbt.getInteger("Hunger"));
		tooltip.add("Saturation: " + nbt.getInteger("Saturation"));
		tooltip.add("RF Cost: ");
	}

	@Override
	public double getDurabilityForDisplay(ItemStack stack) {
		if (!stack.hasTagCompound()) {
			NBTTagCompound nbt = new NBTTagCompound();
			nbt.setInteger("Energy", 0);
			nbt.setInteger("Hunger", 0);
			nbt.setInteger("Saturation", 0);
			stack.setTagCompound(nbt);
		}
		
		NBTTagCompound nbt = stack.getTagCompound();
		
		return (double)capacity / (double)nbt.getInteger("Energy");
	}

	@Override
	public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand handIn) {
		if (Keyboard.isKeyDown(Keyboard.KEY_LSHIFT)) {
			playerIn.openGui(ElectronicLunchbox.instance, ModGuiHandler.electronicLunchboxGuiID, worldIn, (int)playerIn.posX, (int)playerIn.posY, (int)playerIn.posZ);
		}
		
		return super.onItemRightClick(worldIn, playerIn, handIn);
	}

	@Override
	public boolean showDurabilityBar(ItemStack stack) {
		return true;
	}

	@Override
	public int extractEnergy(ItemStack arg0, int arg1, boolean arg2) {
		return arg1;
	}

	@Override
	public int getEnergyStored(ItemStack arg0) {
		return 0;
	}

	@Override
	public int getMaxEnergyStored(ItemStack arg0) {
		return 1000000;
	}

	@Override
	public int receiveEnergy(ItemStack arg0, int arg1, boolean arg2) {
		if (!arg0.hasTagCompound()) {
			arg0.setTagCompound(new NBTTagCompound());
		}
		int energy = arg0.getTagCompound().getInteger("Energy");
		int energyReceived = Math.min(capacity - energy, Math.min(maxReceive, maxReceive));

		if (!arg2) {
			energy += energyReceived;
			arg0.getTagCompound().setInteger("Energy", energy);
		}
		return energyReceived;
	}	
}

The NBT on it is set in a Gui, but that works perfectly.

 

The problem is that no matter what the NBT says the hunger and saturation will heal, it will heal the whole hunger bar full of hunger and saturation.

[larsgerrits]

Are you sure you aren't just setting the NBT values way too high? What are the NBT values? For example, cooked beef/pork has 8 "food" + 0.8 saturation and cooked chicken gives 6 "food" + 0.6 saturation.

 

Also this code will crash:

if (Keyboard.isKeyDown(Keyboard.KEY_LSHIFT)) {
			playerIn.openGui(ElectronicLunchbox.instance, ModGuiHandler.electronicLunchboxGuiID, worldIn, (int)playerIn.posX, (int)playerIn.posY, (int)playerIn.posZ);
		}

Keyboard is a class only available on the client side, so this will crash on the server. I assume you want the player sneaking to be abe to open the GUI, so you can use Entity::isSneaking instead.

[meee39]

The NBT values are set by a slider in the Gui. The maximum value for both is 20. I think that is fine for hunger, but not saturation.

[meee39]

Also whenever you eat with it or reenter the world, the hunger is set back to 10 and the saturation to 20.

[Draco18s]

  On 11/26/2017 at 1:44 PM, meee39 said:

The NBT values are set by a slider in the Gui. The maximum value for both is 20. I think that is fine for hunger, but not saturation.

Expand  

Are you using a packet to send the slider value from the GUI back to the server?

[meee39]

No. How do I do that?

 

EDIT: I've done that in 1.11.2. I think I'll be able to figure it out

Edited November 26, 2017 by meee39