Jump to content

Recommended Posts

Posted

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 :D)

 

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.

width=463 height=200

http://s13.postimg.org/z9mlly2av/siglogo.png[/img]

My mods (Links coming soon)

Cities | Roads | Remula | SilvaniaMod | MoreStats

Posted

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 :P

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

width=463 height=200

http://s13.postimg.org/z9mlly2av/siglogo.png[/img]

My mods (Links coming soon)

Cities | Roads | Remula | SilvaniaMod | MoreStats

Posted

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.

Posted

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.

Posted

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 :P

width=463 height=200

http://s13.postimg.org/z9mlly2av/siglogo.png[/img]

My mods (Links coming soon)

Cities | Roads | Remula | SilvaniaMod | MoreStats

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

    • When I first heard about Bitcoin back in 2018, I was skeptical. The idea of a decentralized, digital currency seemed too good to be true. But I was intrigued as I learned more about the technology behind it and its potential. I started small, investing just a few hundred dollars, dipping my toes into the cryptocurrency waters. At first, it was exhilarating to watch the value of my investment grow exponentially. I felt like I was part of the future, an early adopter of this revolutionary new asset. But that euphoria was short-lived. One day, I logged into my digital wallet only to find it empty - my Bitcoin had vanished without a trace. It turned out that the online exchange I had trusted had been hacked, and my funds were stolen. I was devastated, both financially and emotionally. All the potential I had seen in Bitcoin was tainted by the harsh reality that with decentralization came a lack of regulation and oversight. My hard-earned money was gone, lost to the ether of the digital world. This experience taught me a painful lesson about the price of trust in the uncharted territory of cryptocurrency. While the technology holds incredible promise, the risks can be catastrophic if you don't approach it with extreme caution. My Bitcoin investment gamble had failed, and I was left to pick up the pieces, wiser but poorer for having placed my faith in the wrong hands. My sincere appreciation goes to MUYERN TRUST HACKER. You are my hero in recovering my lost funds. Send a direct m a i l ( muyerntrusted ( @ ) mail-me ( . )c o m ) or message on whats app : + 1 ( 4-4-0 ) ( 3 -3 -5 ) ( 0-2-0-5 )
    • You could try posting a log (if there is no log at all, it may be the launcher you are using, the FAQ may have info on how to enable the log) as described in the FAQ, however this will probably need to be reported to/remedied by the mod author.
    • So me and a couple of friends are playing with a shitpost mod pack and one of the mods in the pack is corail tombstone and for some reason there is a problem with it, where on death to fire the player will get kicked out of the server and the tombstone will not spawn basically deleting an entire inventory, it doesn't matter what type of fire it is, whether it's from vanilla fire/lava, or from modded fire like ice&fire/lycanites and it's common enough to where everyone on the server has experienced at least once or twice and it doesn't give any crash log. a solution to this would be much appreciated thank you!
    • It is 1.12.2 - I have no idea if there is a 1.12 pack
  • Topics

×
×
  • Create New...

Important Information

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