Search the Forums BEFORE making a new post.

[1.7.10] Get Access To NBT From A Different Class

Posted November 2, 201410 yr

I want to use the NBT I have made in one class, in another, how would I do this and here is my code

NBT

package com.bugzoo.FinancialMod;

import java.util.List;

import net.minecraft.entity.Entity;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.ChatComponentText;
import net.minecraft.util.EnumChatFormatting;
import net.minecraft.world.World;

public class ItemDebitCard extends Item {
	public ItemDebitCard(){
		setMaxStackSize(1);
		setUnlocalizedName("DebitCard");
		setCreativeTab(FinancialMod.financialTab);
	}

	int UniquePin = 1000 + (int)(Math.random()*9999); 
	public static double balance;

	    @Override
	    public void onCreated(ItemStack itemStack, World world, EntityPlayer player) {
	    	 itemStack.stackTagCompound = new NBTTagCompound();
	    	 itemStack.stackTagCompound.setString("owner", player.getDisplayName());
	    	 itemStack.stackTagCompound.setInteger("pin", UniquePin);
	    	 itemStack.stackTagCompound.setDouble("balance", balance);
	    	 if(!world.isRemote){
	    	 player.addChatComponentMessage(new ChatComponentText(EnumChatFormatting.RED + "Your Unique Pin is " + UniquePin));
	    	 }
	    }

	    public void addInformation(ItemStack itemStack, EntityPlayer player, List list, boolean par4) {
	    	 if (itemStack.stackTagCompound != null) {
                    String owner = itemStack.stackTagCompound.getString("owner");
                    String pin = itemStack.stackTagCompound.getString("pin");
                     list.add("Owner: " + owner);
                     list.add("Balance: " + balance);
                     
                     
                     if (!owner.equals(player.getDisplayName())) {
                    	  list.add("Pin: " + EnumChatFormatting.OBFUSCATED + pin);
    		    	 } else {
    		    		 list.add("Pin: " + pin);
    		    	 }
	    	 }
	    	 
             }
	    
	    
	    }

GUI

package com.bugzoo.FinancialMod;

import net.minecraft.client.gui.GuiButton;
import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.util.ResourceLocation;

import org.lwjgl.opengl.GL11;



public class ATMPinGUI extends GuiContainer{
public final int xSizeBackground1 = 232;
public final int ySizeBackground1 = 240;

private String enteredPin = "";
private String underScore = "";
private int tick = 0;




public final int guiX = (this.width - xSizeBackground1) / 2;
public final int guiY = (this.height - xSizeBackground1) / 2;

public ResourceLocation background1 = new ResourceLocation("financialmod", "textures/gui/ATM.png");


public ATMPinGUI(InventoryPlayer player, TileEntityATM atm){
	super(new ContainerATM(player, atm));
}


public void initGui(){
	buttonList.clear();
	buttonList.add(new ATMButton(1, guiLeft + 181, guiTop + 149, 24, 15, "1")); // 1
    	buttonList.add(new ATMButton(2, guiLeft + 213, guiTop + 149, 24, 15, "2")); // 2
    	buttonList.add(new ATMButton(3, guiLeft + 245, guiTop + 149, 24, 15, "3")); // 3
    	buttonList.add(new ATMRedButton(4, guiLeft + 281, guiTop + 149, 24, 15, "")); // Cancel
    	buttonList.add(new ATMButton(5, guiLeft + 181, guiTop + 173, 24, 15, "4")); // 4
    	buttonList.add(new ATMButton(6, guiLeft + 213, guiTop + 173, 24, 15, "5")); // 5
    	buttonList.add(new ATMButton(7, guiLeft + 245, guiTop + 173, 24, 15, "6")); // 6
    	buttonList.add(new ATMYellowButton(8, guiLeft + 281, guiTop + 173, 24, 15, "")); // Clear
    	buttonList.add(new ATMButton(9, guiLeft + 181, guiTop + 197, 24, 15, "7")); // 7
    	buttonList.add(new ATMButton(10, guiLeft + 213, guiTop + 197, 24, 15, "8")); // 8
    	buttonList.add(new ATMButton(11, guiLeft + 245, guiTop + 197, 24, 15, "9")); // 9
    	buttonList.add(new ATMConfirmButton(12, guiLeft + 281, guiTop + 197, 24, 15, "")); // Confirm
    	buttonList.add(new ATMButton(13, guiLeft + 213, guiTop + 221, 24, 15, "0")); // 0

    	buttonList.add(new ATMLeftButton(14, guiLeft + 130, guiTop + 33, 24, 15, "")); //Top-Left
    	buttonList.add(new ATMLeftButton(15, guiLeft + 130, guiTop + 60, 24, 15, "")); //Mid-Upper Left
    	buttonList.add(new ATMLeftButton(16, guiLeft + 130, guiTop + 87, 24, 15, "")); //Mid-Lower Left
    	buttonList.add(new ATMLeftButton(17, guiLeft + 130, guiTop + 114, 24, 15, "")); //Bottom Left
    	
    	buttonList.add(new ATMRightButton(18, guiLeft + 325, guiTop + 33, 24, 15, "")); //Top-Right
    	buttonList.add(new ATMRightButton(19, guiLeft + 325, guiTop + 60, 24, 15, "")); //Mid-Upper Right
    	buttonList.add(new ATMRightButton(20, guiLeft + 325, guiTop + 87, 24, 15, "")); //Mid-Lower Right
    	buttonList.add(new ATMRightButton(21, guiLeft + 325, guiTop + 114, 24, 15, "")); //Bottom Right

}

public void actionPerformed(GuiButton guibutton){
	if(guibutton.id==1){
		enteredPin = enteredPin + "1";
		System.out.println(enteredPin);
	}
	if(guibutton.id==2){
		enteredPin = enteredPin + "2";
		System.out.println(enteredPin);
	}
	if(guibutton.id==3){
		enteredPin = enteredPin + "3";
		System.out.println(enteredPin);
	}
	if(guibutton.id==4){
		this.enteredPin = "";
	}
	if(guibutton.id==5){
		enteredPin = enteredPin + "4";
		System.out.println(enteredPin);
	}
	if(guibutton.id==6){
		enteredPin = enteredPin + "5";
		System.out.println(enteredPin);
	}
	if(guibutton.id==7){
		enteredPin = enteredPin + "6";
		System.out.println(enteredPin);
	}
	if(guibutton.id=={

	}
	if(guibutton.id==9){
		enteredPin = enteredPin + "7";
		System.out.println(enteredPin);
	}
	if(guibutton.id==10){
		enteredPin = enteredPin + "8";
		System.out.println(enteredPin);
	}
	if(guibutton.id==11){
		enteredPin = enteredPin + "9";
		System.out.println(enteredPin);
	}
	if(guibutton.id==12){
		if(enteredPin.equals(ItemDebitCard.owner)){
			System.out.println("THEY HAVE THE SAME PIN!");
		}
	}
}


public boolean doesGuiPauseGame(){
	return false;
}

public void drawScreen(int i, int j, float f){



	//Background Texture
	GL11.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
        this.mc.getTextureManager().bindTexture(background1);
        int k1 = (this.width - this.xSizeBackground1) / 2;
        int l1 = (this.height - this.ySizeBackground1) / 2;
        this.drawTexturedModalRect(k1, l1, 0, 0, this.xSizeBackground1, this.ySizeBackground1);
        
        String underScore = "";
    	if (tick < 60) {
    		tick++;
    	} else if (tick >= 60) {
    		tick = 0;
    	}
    	
    	if (tick < 30) {
    		underScore = "_";
    	}
    	
    	if (enteredPin.length() == 1) {
		drawString(fontRendererObj, "*" + underScore, width - 295, 90, 0xff00);
	} else if (enteredPin.length() == 2) {
		drawString(fontRendererObj, "**" + underScore, width - 295, 90, 0xff00);
	} else if (enteredPin.length() == 3) {
		drawString(fontRendererObj, "***" + underScore, width - 295, 90, 0xff00);
	} else if (enteredPin.length() >= 4) {
		drawString(fontRendererObj, "****" + underScore, width - 295, 90, 0xff00);
	}
        
        this.drawCenteredString(fontRendererObj, "Welcome!", width/2, 45, 0xff00);
        this.drawString(fontRendererObj, "Please Enter Your Pin", width - 295, 60, 0xff00);
        if(enteredPin.length() == 0){
        this.drawString(fontRendererObj, underScore, width - 295, 90, 0xff00);
        }
        super.drawScreen(i, j, f); 
}


@Override
protected void drawGuiContainerBackgroundLayer(float p_146976_1_,
		int p_146976_2_, int p_146976_3_) {


}
}

Quote

November 2, 201410 yr

Author

Grmbl, you still have
UniquePin
and
balance
as fields in your Item class. This does not work! You have to use the ItemStack every time you want to access those values. You cannot store them in the Item class.

For accessing those values from your Gui: Use the ItemStack, just like you do in your Item class already. For writing values to it from a Gui you will need packets though, as a Gui is client-only and you cannot change things in the world/in inventories/etc. from the client.

So like this?

itemStack.stackTagCompound = new NBTTagCompound();
	    	 itemStack.stackTagCompound.setString("owner", player.getDisplayName());
	    	 itemStack.stackTagCompound.setInteger("pin", 1000 + (int)(Math.random()*9999));

Quote

November 2, 201410 yr

Author

That would overwrite any previous data, yes. I suggest you only create a new NBTTagCompound if stackTagCompound is null, to avoid overwriting any previous data like e.g. Enchantments.

Also you should not store the display name. It potentially includes stuff like color codes and stuff. Use the UUID for the player, as Mojang will soon allow username changes.

So, how would I get the pin into my gui class, I used this but it gave me a null pointer

if(enteredPin.equals(itemstack.stackTagCompound.getString("pin")))

Quote

November 2, 201410 yr

Either

enteredPin

itemstack

stackTagCompound

null. Put a breakpoint on that line so you can see which was is null.

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

1.12 -> 1.13 primer by williewillus.

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

http://www.howoldisminecraft1710.today/

Quote

November 2, 201410 yr

Author

Either
enteredPin
,
itemstack
or
stackTagCompound
null. Put a breakpoint on that line so you can see which was is null.

Here is the whole class. I think that the itemstack might be null

package com.bugzoo.FinancialMod;

import java.io.IOException;

import net.minecraft.client.gui.GuiButton;
import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.item.ItemStack;
import net.minecraft.util.ResourceLocation;

import org.lwjgl.opengl.GL11;



public class ATMPinGUI extends GuiContainer{
public final int xSizeBackground1 = 232;
public final int ySizeBackground1 = 240;

private String enteredPin = "";
private String underScore = "";
private int tick = 0;

public ItemStack itemstack;


public final int guiX = (this.width - xSizeBackground1) / 2;
public final int guiY = (this.height - xSizeBackground1) / 2;

public ResourceLocation background1 = new ResourceLocation("financialmod", "textures/gui/ATM.png");


public ATMPinGUI(InventoryPlayer player, TileEntityATM atm){
	super(new ContainerATM(player, atm));
}


public void initGui(){
	buttonList.clear();
	buttonList.add(new ATMButton(1, guiLeft + 181, guiTop + 149, 24, 15, "1")); // 1
    	buttonList.add(new ATMButton(2, guiLeft + 213, guiTop + 149, 24, 15, "2")); // 2
    	buttonList.add(new ATMButton(3, guiLeft + 245, guiTop + 149, 24, 15, "3")); // 3
    	buttonList.add(new ATMRedButton(4, guiLeft + 281, guiTop + 149, 24, 15, "")); // Cancel
    	buttonList.add(new ATMButton(5, guiLeft + 181, guiTop + 173, 24, 15, "4")); // 4
    	buttonList.add(new ATMButton(6, guiLeft + 213, guiTop + 173, 24, 15, "5")); // 5
    	buttonList.add(new ATMButton(7, guiLeft + 245, guiTop + 173, 24, 15, "6")); // 6
    	buttonList.add(new ATMYellowButton(8, guiLeft + 281, guiTop + 173, 24, 15, "")); // Clear
    	buttonList.add(new ATMButton(9, guiLeft + 181, guiTop + 197, 24, 15, "7")); // 7
    	buttonList.add(new ATMButton(10, guiLeft + 213, guiTop + 197, 24, 15, "8")); // 8
    	buttonList.add(new ATMButton(11, guiLeft + 245, guiTop + 197, 24, 15, "9")); // 9
    	buttonList.add(new ATMConfirmButton(12, guiLeft + 281, guiTop + 197, 24, 15, "")); // Confirm
    	buttonList.add(new ATMButton(13, guiLeft + 213, guiTop + 221, 24, 15, "0")); // 0

    	buttonList.add(new ATMLeftButton(14, guiLeft + 130, guiTop + 33, 24, 15, "")); //Top-Left
    	buttonList.add(new ATMLeftButton(15, guiLeft + 130, guiTop + 60, 24, 15, "")); //Mid-Upper Left
    	buttonList.add(new ATMLeftButton(16, guiLeft + 130, guiTop + 87, 24, 15, "")); //Mid-Lower Left
    	buttonList.add(new ATMLeftButton(17, guiLeft + 130, guiTop + 114, 24, 15, "")); //Bottom Left
    	
    	buttonList.add(new ATMRightButton(18, guiLeft + 325, guiTop + 33, 24, 15, "")); //Top-Right
    	buttonList.add(new ATMRightButton(19, guiLeft + 325, guiTop + 60, 24, 15, "")); //Mid-Upper Right
    	buttonList.add(new ATMRightButton(20, guiLeft + 325, guiTop + 87, 24, 15, "")); //Mid-Lower Right
    	buttonList.add(new ATMRightButton(21, guiLeft + 325, guiTop + 114, 24, 15, "")); //Bottom Right

}

public void actionPerformed(GuiButton guibutton){
	if(guibutton.id==1){
		enteredPin = enteredPin + "1";
		System.out.println(enteredPin);
	}
	if(guibutton.id==2){
		enteredPin = enteredPin + "2";
		System.out.println(enteredPin);
	}
	if(guibutton.id==3){
		enteredPin = enteredPin + "3";
		System.out.println(enteredPin);
	}
	if(guibutton.id==4){
		this.enteredPin = "";
	}
	if(guibutton.id==5){
		enteredPin = enteredPin + "4";
		System.out.println(enteredPin);
	}
	if(guibutton.id==6){
		enteredPin = enteredPin + "5";
		System.out.println(enteredPin);
	}
	if(guibutton.id==7){
		enteredPin = enteredPin + "6";
		System.out.println(enteredPin);
	}
	if(guibutton.id=={


		/*String URL = "https://www.google.co.uk/search?q=atm+screen&client=firefox-a&hs=KRv&rls=org.mozilla:en-GB:official&channel=np&tbm=isch&imgil=4qxgegDeEWCSaM%253A%253BMH8IbcHAhg9-3M%253Bhttps%25253A%25252F%25252Fsufiazamir.wordpress.com%25252Ftag%25252Fatm-screens%25252F&source=iu&pf=m&fir=4qxgegDeEWCSaM%253A%252CMH8IbcHAhg9-3M%252C_&usg=__0fIcaFRq-6p0shy1i8dW3VNWoa4%3D&biw=1536&bih=755&ved=0CC4Qyjc&ei=MRlWVPz-FOat7AaP04HwCw#facrc=_&imgdii=_&imgrc=Hxd6rXd8p_O0NM%253A%3B4r3sPIEVxepLfM%3Bhttp%253A%252F%252Fzesty.ca%252Flj%252Flusaka-atm-3.jpg%3Bhttp%253A%252F%252Fzestyping.livejournal.com%252F251102.html%3B400%3B300";
		try {
			java.awt.Desktop.getDesktop().browse(java.net.URI.create(URL));
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
		*/
	}
	if(guibutton.id==9){
		enteredPin = enteredPin + "7";
		System.out.println(enteredPin);
	}
	if(guibutton.id==10){
		enteredPin = enteredPin + "8";
		System.out.println(enteredPin);
	}
	if(guibutton.id==11){
		enteredPin = enteredPin + "9";
		System.out.println(enteredPin);
	}
	if(guibutton.id==12){
		if(enteredPin.equals(itemstack.stackTagCompound.getString("pin"))){

		}
	}
}


public boolean doesGuiPauseGame(){
	return false;
}

public void drawScreen(int i, int j, float f){



	//Background Texture
	GL11.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
        this.mc.getTextureManager().bindTexture(background1);
        int k1 = (this.width - this.xSizeBackground1) / 2;
        int l1 = (this.height - this.ySizeBackground1) / 2;
        this.drawTexturedModalRect(k1, l1, 0, 0, this.xSizeBackground1, this.ySizeBackground1);
        
        String underScore = "";
    	if (tick < 60) {
    		tick++;
    	} else if (tick >= 60) {
    		tick = 0;
    	}
    	
    	if (tick < 30) {
    		underScore = "_";
    	}
    	
    	if (enteredPin.length() == 1) {
		drawString(fontRendererObj, "*" + underScore, width - 295, 90, 0xff00);
	} else if (enteredPin.length() == 2) {
		drawString(fontRendererObj, "**" + underScore, width - 295, 90, 0xff00);
	} else if (enteredPin.length() == 3) {
		drawString(fontRendererObj, "***" + underScore, width - 295, 90, 0xff00);
	} else if (enteredPin.length() >= 4) {
		drawString(fontRendererObj, "****" + underScore, width - 295, 90, 0xff00);
	}
        
        this.drawCenteredString(fontRendererObj, "Welcome!", width/2, 45, 0xff00);
        this.drawString(fontRendererObj, "Please Enter Your Pin", width - 295, 60, 0xff00);
        if(enteredPin.length() == 0){
        this.drawString(fontRendererObj, underScore, width - 295, 90, 0xff00);
        }
        super.drawScreen(i, j, f); 
}


@Override
protected void drawGuiContainerBackgroundLayer(float p_146976_1_,
		int p_146976_2_, int p_146976_3_) {


}
}

Quote

November 2, 201410 yr

Yeah, you don't set the

itemstack

variable anywhere, so how would you expect it to not be null?

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

1.12 -> 1.13 primer by williewillus.

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

http://www.howoldisminecraft1710.today/

Quote

November 2, 201410 yr

Author

Yeah, you don't set the
itemstack
variable anywhere, so how would you expect it to not be null?

Ok, so what should I set the itemstack equal to?

ItemStack itemstack = null;
itemstack.stackTagCompound = new NBTTagCompound();
		if(enteredPin.equals(itemstack.stackTagCompound.getString("pin"))){
			System.out.println("The Right Pin!");
		}

Quote

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.

Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.

Reply to this topic...

https://forums.minecraftforge.net/topic/24662-1710-get-access-to-nbt-from-a-different-class/

Followers

Go to topic listing

IPS spam blocked by CleanTalk.

Announcements

I cannot create a minecraft server with these mods or i get ´´Exit code: 1´´, but i can play in singleplayer with no problems

The link is not working

By TileEntity · 3 hours ago3 hr
I cannot create a minecraft server with these mods or i get ´´Exit code: 1´´, but i can play in singleplayer with no problems

Okay, i can join the server succesfully but around 30 seconds later the server crash, heres the new log https://client.falixnodes.net/shared-logs/6b5e1611-93c7-476b-a6a5-ecdbeb1af3ba

By ivanironman10 · 4 hours ago4 hr
1.20.1 Client crashing when joining the ender portal

It is an issue with the mod unusualend - make a test without it - backup the world before

By TileEntity · 8 hours ago8 hr
1.20.1 Client crashing when joining the ender portal

I see I see
It's wild to know my crash log is genuinely confusing, but your comment definitely is helpful in guiding me to a possible solution
Thank you for taking the time to look through it, I appreciate your time!

By Dork-A-Doodle · 11 hours ago11 hr
1.20.1 Client crashing when joining the ender portal

Hey man, I looked at your crash log, and I gotta' say: I'm not exactly sure wth is goin' on here either. But I believe I can offer some advice to you to hopefully help you better pinpoint the issue here. Now, I checked the crash file and you have a TON of mods. When making a mod pack, adding a lot of mods together can cause some issues, as their transformer functions can interfere with each other often. Because of this, you might have choices of mods that you can try to remove instead of just one singular mod being the culprit. You just need to make sure that the mods that aren't exactly compatible with each other are removed. Since you have many mods installed, you should try first backing up your world. Then, create a copy of the back up and remove a large chunk of the mods from your mod folder. Then, load your copy of the back up. If there are no errors, then you know one or both, maybe even all, of the problematic mods is in the chunk you selected. If not, then it's the other way around. Try and use this kind of logic to rapidly hone in on the problematic mod, constantly creating a new copy of your world from the back up each time and loading it from there. Let me know if this is feasible for you. If not, I can try'n brain storm another solution for ya'.

Hope this helps man!

By Jackson Perr · 11 hours ago11 hr

TileEntity

Sign In

[1.7.10] Get Access To NBT From A Different Class

Featured Replies

Join the conversation

Posts

I cannot create a minecraft server with these mods or i get ´´Exit code: 1´´, but i can play in singleplayer with no problems

I cannot create a minecraft server with these mods or i get ´´Exit code: 1´´, but i can play in singleplayer with no problems

1.20.1 Client crashing when joining the ender portal

1.20.1 Client crashing when joining the ender portal

1.20.1 Client crashing when joining the ender portal

Who's Online (See full list)

Important Information

Account

Navigation

Search

Configure browser push notifications

Chrome (Android)

Chrome (Desktop)

Safari (iOS 16.4+)

Safari (macOS)

Edge (Android)

Edge (Desktop)

Firefox (Android)

Firefox (Desktop)