Using a value from the player NBT as a string in a GUI

[Flenix]

Hey guys,

 

I'm struggling with this; one of the last things getting the economy system in my mod working.

 

Basically, as a quick overview of the mod to get an idea, players can get both physical and digital currency. They transmute between these using an ATM block, and currently it's hardcoded so when they right-click the ATM with a $10 bill, It will deposit the $10 and tell them they've done so. I also have a debit card, which checks the balance in the NBT, and proves it's 10 (thanks to maxpowa for helping me get that part working )

 

However, I don't want the debit card to display the value; I want the value to be shown inside a GUI. I've tried quite a few things, and I can't seem to get it to work...

 

Here was my latest attempt, trying to set a variable in the ATM block class and then use it in the drawString.

 

Anyone got any suggestions?

 

Gui:

package co.uk.silvania.city.tileentities.containers;

import java.text.DecimalFormat;

import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.client.resources.ResourceLocation;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.item.ItemStack;
import net.minecraft.util.StatCollector;
import net.minecraft.world.World;

import org.lwjgl.opengl.GL11;

import co.uk.silvania.city.tileentities.TileEntityATMEntity;
import co.uk.silvania.city.tileentities.blocks.TileEntityATMBlock;

public class GuiATM extends GuiContainer {

//DecimalFormat decim = new DecimalFormat("#.##");

    private static final ResourceLocation texture = new ResourceLocation("flenixcities", "/textures/gui/atm.png");

        public GuiATM (InventoryPlayer inventoryPlayer, TileEntityATMEntity tileEntity, World world, int x, int y, int z) {
        	super(new ContainerATM(inventoryPlayer, tileEntity));
        }
        
        protected int xSize = 176;
        protected int ySize = 198;
        
        @Override
        protected void drawGuiContainerForegroundLayer(int param1, int param2) {
        	fontRenderer.drawString("ATM", 8, -8, 4210752);
        	fontRenderer.drawString("Balance", 8, 9, 4210752);
        	fontRenderer.drawString("Safe", 132, -11, 4210752);
        	fontRenderer.drawString("Deposit", 124, -3, 4210752);
        	fontRenderer.drawString("Withdraw", 8, 29, 4210752);
        	fontRenderer.drawString("Deposit", 42, 68, 4210752);
        	fontRenderer.drawString("$" + TileEntityATMBlock.playerBalance, 58, 9, 1237000);
        	fontRenderer.drawString("345.67", 58, 29, 1237000);
        	fontRenderer.drawString(StatCollector.translateToLocal("container.inventory"), 8, ySize - 111, 4210752);
        }

        @Override
        protected void drawGuiContainerBackgroundLayer(float par1, int par2,
                        int par3) {
                GL11.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
                this.mc.func_110434_K().func_110577_a(texture);
                int x = (width - xSize) / 2;
                int y = (height - ySize) / 2;
                this.drawTexturedModalRect(x, y, 0, 0, xSize, ySize);
        }
}

 

Block:

package co.uk.silvania.city.tileentities.blocks;

import java.util.List;

import co.uk.silvania.city.FlenixCities;
import co.uk.silvania.city.NBTConfig;
import co.uk.silvania.city.items.econ.ItemNote10;
import co.uk.silvania.city.tileentities.TileEntityATMEntity;
import cpw.mods.fml.common.Mod.Metadata;
import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
import net.minecraft.block.Block;
import net.minecraft.block.BlockContainer;
import net.minecraft.block.material.Material;
import net.minecraft.client.renderer.texture.IconRegister;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.EntityLiving;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.MathHelper;
import net.minecraft.world.IBlockAccess;
import net.minecraft.world.World;

public class TileEntityATMBlock extends BlockContainer {	

public static double playerBalance = 0;

public TileEntityATMBlock(int id) {
	super(id, Material.iron);
	this.setHardness(1.0F);
	this.setCreativeTab(FlenixCities.tabEcon);
	this.setLightOpacity(0);
}

@Override
public TileEntity createNewTileEntity(World world) {
	return new TileEntityATMEntity();
}

@Override
public int getRenderType() {
	return -1;
}

//Huge thanks to "maxpowa" in helping me get this working!!
//Temporary Economy deposit system. Eventually this will happen inside the GUI, you will place in money and it'll get sucked into the bank,
//and the bank will pay you. For now, this is fine, and the code will be effectively the same anyway.
    @Override
    public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer player, int i, float j, float k, float l) {
        if (!world.isRemote) {
            if (player.getHeldItem() != null) {
                if (player.getHeldItem().getItem() == FlenixCities.debitCard) {
                    double balance = 0;
                    NBTTagCompound nbt = NBTConfig.getTagCompoundInFile(NBTConfig.getWorldConfig(world));
                    if (nbt.hasKey(player.username)) {
                        NBTTagCompound playernbt = nbt.getCompoundTag(player.username);
                        if (playernbt.hasKey("Balance")) 
                            balance = playernbt.getDouble("Balance");
                    }
                    player.addChatMessage("Your Balance is: " + balance);
                    player.openGui(FlenixCities.instance, 0, world, x, y, z);
                }
            }
        if (player.getHeldItem() != null) {
            if (player.getHeldItem().getItem() == FlenixCities.note1000) {
                double currentBalance = 0;
                NBTTagCompound nbt = NBTConfig.getTagCompoundInFile(NBTConfig.getWorldConfig(world));
                if (nbt.hasKey(player.username)) {
                    NBTTagCompound playernbt = nbt.getCompoundTag(player.username);
                    if (playernbt.hasKey("Balance"))
                        currentBalance = playernbt.getDouble("Balance");
                    double modifiedBalance = currentBalance + ItemNote10.moneyValue;
                    playernbt.setDouble("Balance", modifiedBalance);
                    nbt.setCompoundTag(player.username, playernbt);
                } else {
                    NBTTagCompound playernbt = new NBTTagCompound();
                    if (playernbt.hasKey("Balance"))
                        currentBalance = playernbt.getDouble("Balance");
                    double modifiedBalance = currentBalance + ItemNote10.moneyValue;
                    playernbt.setDouble("Balance", modifiedBalance);
                    nbt.setCompoundTag(player.username, playernbt);
                }
                NBTTagCompound playernbt = nbt.getCompoundTag(player.username);
            player.addChatMessage(ItemNote10.moneyValue + " Deposited! Your balance is now " + playernbt.getDouble("Balance"));
            //This is where the playerBalance should be changed right? Or am I doing something wrong?
            double playerBalance = playernbt.getDouble("Balance");
            NBTConfig.saveConfig(nbt, NBTConfig.getWorldConfig(world));
            }
        }
    }
	return true;
    }

@Override
public boolean isOpaqueCube() {
	return false;
}

public boolean renderAsNormalBlock() {
	return false;
}

@Override
public void onBlockPlacedBy(World world, int x, int y, int z, EntityLivingBase entityliving, ItemStack itemStack)
{
    int blockSet = world.getBlockMetadata(x, y, z) / 4;
    int direction = MathHelper.floor_double((entityliving.rotationYaw * 4F) / 360F + 0.5D) & 3;
    int newMeta = (blockSet * 4) + direction;
    world.setBlockMetadataWithNotify(x, y, z, newMeta, 0);
}
    
@SideOnly(Side.CLIENT)
public void getSubBlocks(int par1, CreativeTabs creativeTabs, List list) {
	for (int var4 = 0; var4 < 16; ++var4) {
		list.add(new ItemStack(par1, 1, 0));
	}
}
}

 

Let me know if you need other classes.

[Chibill]

Use the TE of your block to send it from server side where the activate block is Also called.

[Flenix]

Since GUIs are Client-Side only you won't get around sending packets. The way I'd do it immediately after you open the gui send the client a custom packet that will tell the (now open) GUI which value to display.

 

Alright, I'll have a play. I don't mind that; I'll be adding packet stuff later anyway so it's not a big deal

I don't suppose you have a link to a relatively up-to-date tutorial on packet handling?

[Guff]

If I may, this may clean up your file a bit as well as help you with your question, without packets:

 

//If you're not using the world and coord params, you can remove them
	public GuiATM (InventoryPlayer inventoryPlayer, TileEntityATMEntity tileEntity, World world, int x, int y, int z) {
    	super(new ContainerATM(inventoryPlayer, tileEntity));
    	this.xSize = 176;
    	this.ySize = 198;
    	
    	this.player = inventoryPlayer.player;
    }
    //these are inherited, no need to make new ones.
//	    protected int xSize = 176;
//	    protected int ySize = 198;
    
	private EntityPlayer player;
    @Override
    protected void drawGuiContainerForegroundLayer(int param1, int param2)
    {
    	fontRenderer.drawString("ATM", 8, -8, 4210752);
    	fontRenderer.drawString("Balance", 8, 9, 4210752);
    	fontRenderer.drawString("Safe", 132, -11, 4210752);
    	fontRenderer.drawString("Deposit", 124, -3, 4210752);
    	fontRenderer.drawString("Withdraw", 8, 29, 4210752);
    	fontRenderer.drawString("Deposit", 42, 68, 4210752);
    	
    	String playerBalance = this.player.getEntityData().getInteger("balanceKey") + "";
    	//playerBalance shouldn't be static.
    	fontRenderer.drawString("$" + TileEntityATMBlock.playerBalance, 58, 9, 1237000);
    	fontRenderer.drawString("345.67", 58, 29, 1237000);
    	fontRenderer.drawString(StatCollector.translateToLocal("container.inventory"), 8, ySize - 111, 4210752);
    }

    @Override
    protected void drawGuiContainerBackgroundLayer(float par1, int par2,
                    int par3) {
            GL11.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
            this.mc.func_110434_K().func_110577_a(texture);
            int x = (width - xSize) / 2;
            int y = (height - ySize) / 2;
            this.drawTexturedModalRect(x, y, 0, 0, xSize, ySize);
    }

 

Obviously balanceKey is probably not where you actually stored the player balance, and I don't even know if you stored the balance inside of the player's NBT. If you did, however, that's how you would get it, but just changing the key to your key for balance. Then you can draw whatever string you want to by appending that, I know you know what you're doing.

 

Since every Gui that comes up is a unique instance of the type, this will work even when two different people are using the block at the same time.

[Guff]

The Gui is Client-side and all he wants to do is display the value to the player, and as far as I can tell he wants nothing to do with sending it to the server and doing anything with it there.

[Flenix]

I'll eventually need packets with it anyway so I'll probably use the packet method. Eventually I plan to add bukkit support (for people using MCPC+ and BukkitForge), so that players can transfer their balance from this to a bukkit econ and vice versa via a hidden server command. So, at any rate, both the client and server will need it in my setup