Jump to content

How do I store entities as a data on an Item


TheRPGAdventurer

Recommended Posts

See, I want a dragon whistle that when right clicked will open a gui, and have an array of commands, fly, circle, come, nothing. The commands seem to work but problem is, when i do the command, all the dragons in the world will do the same command, so I want a dragon to be right clicked by the owner to add a lists of dragons to control, the maximum is 3 dragons per whistle. and I can change what dragon to control by shift + right clicking the whistle, it will send a status message at the hotbar with the dragons displayName.  

 

// This code is an attempt, not really finished.

package com.TheRPGAdventurer.ROTD.client.items;

import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

import javax.annotation.Nullable;

import com.TheRPGAdventurer.ROTD.DragonMounts;
import com.TheRPGAdventurer.ROTD.client.userinput.StatCollector;
import com.TheRPGAdventurer.ROTD.server.entity.EntityTameableDragon;
import com.TheRPGAdventurer.ROTD.server.network.MessageDragonWhistle;

import net.minecraft.client.util.ITooltipFlag;
import net.minecraft.entity.Entity;
import net.minecraft.entity.EntityLivingBase;
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.ActionResult;
import net.minecraft.util.EnumActionResult;
import net.minecraft.util.EnumHand;
import net.minecraft.util.ResourceLocation;
import net.minecraft.util.text.TextComponentTranslation;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class ItemDragonWhistle extends Item {

    private final MessageDragonWhistle dcw = new MessageDragonWhistle();
    private List<EntityTameableDragon> dragons = new ArrayList(); 
    EntityTameableDragon dragon;
	
	public ItemDragonWhistle() {
		this.setUnlocalizedName("dragon_whistle");
		this.setRegistryName(new ResourceLocation(DragonMounts.MODID, "dragon_whistle"));
		this.setMaxStackSize(1);
		this.setCreativeTab(DragonMounts.TAB);
	}
	
	@Override
	@SideOnly(Side.CLIENT)
	public void addInformation(ItemStack stack, @Nullable World worldIn, List<String> tooltip, ITooltipFlag flagIn) {
		tooltip.add(StatCollector.translateToLocal(dragons.toString()));
	}
	
	
	@Override
	public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer player, EnumHand hand) {
		ItemStack stack = player.getHeldItem(hand);

        if(worldIn.isRemote && !player.isSneaking()) {
        	DragonMounts.proxy.openDragonWhistleGui(dragon, new ItemStack(this));
        }
        
        return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, stack);
            		
	}
	
	@Override
	public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity target) {
		if(target instanceof EntityTameableDragon) {
			EntityTameableDragon dragon = (EntityTameableDragon)target;
			this.dragon = dragon;
			if(dragon.isTamedFor(player)) { 			
				NBTTagCompound nbt;
				if(stack.hasTagCompound()) {
					nbt = stack.getTagCompound();
				} else {
					nbt = new NBTTagCompound();
				}
				
				if(nbt.hasKey("dragons")) {
					
				} else {
					
				}
				
				player.sendStatusMessage(new TextComponentTranslation("dragon.added" + dragon.getUniqueID().toString(), new Object[0]), true);
			} else {
				player.sendStatusMessage(new TextComponentTranslation("dragon.notTamed", new Object[0]), true);
			}
			return true;
		} else {
			return false;
		}
	}
	
/*	if(player.isSneaking()) {
    	controllableDragons.iterator().next();
    	player.sendStatusMessage(dragon.getDisplayName(), true);
    	
    	// setting the value of N as 4 
        int limit = 3; 
        int count = 0; 
        Iterator<EntityTameableDragon> it = controllableDragons.iterator(); 
  
        // Iterating through the list of integers 
        while (it.hasNext()) { 
            it.next(); 
            count++; 
  
            // Check if first four i.e, (equal to N) 
            // integers are iterated. 
            if (count > limit) { 
  
                // If yes then remove all the remaining integers. 
                it.remove(); 
            } 
        } 
  
        System.out.print("New stream of length N"
                         + " after truncation is : "); 
  
        // Displaying new stream of length 
        // N after truncation 
        for (EntityTameableDragon number : controllableDragons) { 
            System.out.print(number + " "); 
        } 
    } */
}

 

2018-12-15_23.08.03.png

Link to comment
Share on other sites

Just store the dragon's UUID in the NBT, then you can find said dragon by UUID.

As for the cycling - store a byte indicating the position in the list in the NBT and manipulate it. When getting the UUID for the dragon get it based on that byte. Eiher name your tags dragon_0, dragon_1, etc. or store them in an NBTTagList.

Link to comment
Share on other sites

27 minutes ago, TheRPGAdventurer said:

would these be included ?

Obviously not as you can't store anything per instance of ItemStack in a field in an Item class since items are singletons.

28 minutes ago, TheRPGAdventurer said:

What methods do I call to store UUID's im not so sure.

NBTUtil.createUUIDTag/NBTUtil.getUUIDFromTag

Link to comment
Share on other sites

I know theres something wrong and missing here and Im asking you what.

	@Override
	public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity target) {
		if(target instanceof EntityTameableDragon) {
			EntityTameableDragon dragon = (EntityTameableDragon)target;
			this.dragon = dragon;
			if(dragon.isTamedFor(player)) { 			
				
				NBTTagCompound nbt;
				NBTTagList nbtList;
				if(stack.hasTagCompound()) {
					nbt = stack.getTagCompound();
				} else {
					nbt = new NBTTagCompound(); 
				}
				
				if(nbt.hasKey("dragons")) {
					UUID uuid = dragon.getUniqueID();  					
					nbt.setTag("dragons", NBTUtil.createUUIDTag(uuid));
					
				//} else {
					
				}
				
				player.sendStatusMessage(new TextComponentTranslation("dragon.added" + dragon.getUniqueID().toString(), new Object[0]), true);
			} else {
				player.sendStatusMessage(new TextComponentTranslation("dragon.notTamed", new Object[0]), true);
			}
			return true;
		} else {
			return false;
		}
	}

 

Link to comment
Share on other sites

11 minutes ago, TheRPGAdventurer said:

Im asking you what.

You never use 

11 minutes ago, TheRPGAdventurer said:

NBTTagList nbtList;

Which you first need to initialize a lot like you do your nbt variable.

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

    
    @Override
    public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity target) {
        if(target instanceof EntityTameableDragon) {
            EntityTameableDragon dragon = (EntityTameableDragon)target;
            this.dragon = dragon;
            if(dragon.isTamedFor(player)) {             
                
                NBTTagCompound nbt = new NBTTagCompound();
                NBTTagList nbtList;
                if(stack.hasTagCompound()) {
                    nbt = stack.getTagCompound();
                    nbtList = nbt.getTagList("dragons", 3); // limit as you said
                } else {
                    stack.setTagCompound(nbt);
                }
                
                if(nbt.hasKey("dragons")) {
                    UUID uuid = dragon.getUniqueID();                      
                    nbt.setTag("dragons", NBTUtil.createUUIDTag(uuid));    
                    nbt.setInteger("dragons", dragon.getEntityId()); 
                //} else {                
                }
                
                
                player.sendStatusMessage(new TextComponentTranslation("dragon.added" + dragon.getUniqueID().toString(), new Object[0]), true);
            } else {
                player.sendStatusMessage(new TextComponentTranslation("dragon.notTamed", new Object[0]), true);
            }
            return true;
        } else {
            return false;
        }
    }

 

Something is missing perhaps, please tell me what, and what methods to use.

Link to comment
Share on other sites

16 minutes ago, TheRPGAdventurer said:

this.dragon = dragon;

You can't store information that is per itemstack in item fields since items are singletons.

 

17 minutes ago, TheRPGAdventurer said:

nbtList = nbt.getTagList("dragons", 3); // limit as you said

First of all the second parameter in NBTTagCompound#getTagList is not it's limit, it's the type of tag in said list. Second of all you need to account for a case where the list is not present in the tag, or the stack has no tag at all. Currently your list will be null in 2/3 cases.

 

18 minutes ago, TheRPGAdventurer said:

 nbt.setTag("dragons", NBTUtil.createUUIDTag(uuid));    
                    nbt.setInteger("dragons", dragon.getEntityId()); 

wasn't your point to store the dragon's UUID in a list? Why are you storing it in the item's tag?

 

You are also not storing the list in the tag, so this

22 hours ago, Animefan8888 said:

You never use 

22 hours ago, TheRPGAdventurer said:

NBTTagList nbtList;

Which you first need to initialize a lot like you do your nbt variable.

is still the case.

Link to comment
Share on other sites

package com.TheRPGAdventurer.ROTD.client.items;

import java.util.ArrayList;
import java.util.List;
import java.util.UUID;

import javax.annotation.Nullable;

import com.TheRPGAdventurer.ROTD.DragonMounts;
import com.TheRPGAdventurer.ROTD.client.userinput.StatCollector;
import com.TheRPGAdventurer.ROTD.server.entity.EntityTameableDragon;
import com.TheRPGAdventurer.ROTD.server.network.MessageDragonWhistle;

import net.minecraft.client.util.ITooltipFlag;
import net.minecraft.entity.Entity;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.Item;
import net.minecraft.item.ItemMonsterPlacer;
import net.minecraft.item.ItemStack;
import net.minecraft.item.ItemWrittenBook;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.nbt.NBTTagList;
import net.minecraft.nbt.NBTUtil;
import net.minecraft.util.ActionResult;
import net.minecraft.util.EnumActionResult;
import net.minecraft.util.EnumHand;
import net.minecraft.util.ResourceLocation;
import net.minecraft.util.text.TextComponentTranslation;
import net.minecraft.world.World;
import net.minecraft.world.WorldServer;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class ItemDragonWhistle extends Item {

    private final MessageDragonWhistle dcw = new MessageDragonWhistle();
 //   private List<EntityTameableDragon> dragons = new ArrayList(); 
 //   EntityTameableDragon dragon;
    private byte listPos;
    
    public ItemDragonWhistle() {
        this.setUnlocalizedName("dragon_whistle");
        this.setRegistryName(new ResourceLocation(DragonMounts.MODID, "dragon_whistle"));
        this.setMaxStackSize(1);
        this.setCreativeTab(DragonMounts.TAB);
    }
    
    @Override
    @SideOnly(Side.CLIENT)
    public void addInformation(ItemStack stack, @Nullable World worldIn, List<String> tooltip, ITooltipFlag flagIn) {
//        tooltip.add(StatCollector.translateToLocal(dragons.toString()));
    }
    
    
    @Override
    public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer player, EnumHand hand) {
        ItemStack stack = player.getHeldItem(hand);

        if(worldIn.isRemote && !player.isSneaking() && stack.hasTagCompound()) {
            DragonMounts.proxy.openDragonWhistleGui(stack.getTagCompound().getUniqueId("dragons"), new ItemStack(this), (WorldServer)worldIn);
        }
        
        return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, stack);
                    
    }
    
    @Override
    public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity target) {
        if(target instanceof EntityTameableDragon) {
            EntityTameableDragon dragon = (EntityTameableDragon)target;
        //    this.dragon = dragon;
            if(dragon.isTamedFor(player)) {             
                
                NBTTagCompound nbt = new NBTTagCompound();
                NBTTagList nbtList;
                if(stack.hasTagCompound()) {
                    nbt = stack.getTagCompound();
                    nbtList = nbt.getTagList("dragons", 8); // copied from guiscreen book
                } else {
                    stack.setTagCompound(nbt);
                }
                
                if(nbt.hasKey("dragons")) {
                    UUID uuid = dragon.getUniqueID();                      
                    nbt.setTag("dragons", NBTUtil.createUUIDTag(uuid));    
                    //nbt.setInteger("dragons", dragon.getEntityId()); 
                //} else {                
                }
                
                
                player.sendStatusMessage(new TextComponentTranslation("dragon.added" + dragon.getUniqueID().toString(), DragonMounts.MODID), true);
            } else {
                player.sendStatusMessage(new TextComponentTranslation("dragon.notTamed", DragonMounts.MODID), true);
            }
            return true;
        } else {
            return false;
        }
    }
    
/*    if(player.isSneaking()) {
        controllableDragons.iterator().next();
        player.sendStatusMessage(dragon.getDisplayName(), true);
        
        // setting the value of N as 4 
        int limit = 3; 
        int count = 0; 
        Iterator<EntityTameableDragon> it = controllableDragons.iterator(); 
  
        // Iterating through the list of integers 
        while (it.hasNext()) { 
            it.next(); 
            count++; 
  
            // Check if first four i.e, (equal to N) 
            // integers are iterated. 
            if (count > limit) { 
  
                // If yes then remove all the remaining integers. 
                it.remove(); 
            } 
        } 
  
        System.out.print("New stream of length N"
                         + " after truncation is : "); 
  
        // Displaying new stream of length 
        // N after truncation 
        for (EntityTameableDragon number : controllableDragons) { 
            System.out.print(number + " "); 
        } 
    } */
}

 

Edited by diesieben07
syntax highlighting
Link to comment
Share on other sites

package com.TheRPGAdventurer.ROTD.client.items;

import java.util.ArrayList;
import java.util.List;
import java.util.UUID;

import javax.annotation.Nullable;

import com.TheRPGAdventurer.ROTD.DragonMounts;
import com.TheRPGAdventurer.ROTD.client.userinput.StatCollector;
import com.TheRPGAdventurer.ROTD.server.entity.EntityTameableDragon;
import com.TheRPGAdventurer.ROTD.server.network.MessageDragonWhistle;

import gnu.trove.impl.Constants;
import net.minecraft.client.util.ITooltipFlag;
import net.minecraft.entity.Entity;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.Item;
import net.minecraft.item.ItemMonsterPlacer;
import net.minecraft.item.ItemStack;
import net.minecraft.item.ItemWrittenBook;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.nbt.NBTTagList;
import net.minecraft.nbt.NBTUtil;
import net.minecraft.util.ActionResult;
import net.minecraft.util.EnumActionResult;
import net.minecraft.util.EnumHand;
import net.minecraft.util.ResourceLocation;
import net.minecraft.util.text.TextComponentTranslation;
import net.minecraft.world.World;
import net.minecraft.world.WorldServer;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class ItemDragonWhistle extends Item {

 //   private final MessageDragonWhistle dcw = new MessageDragonWhistle();
 //   private List<EntityTameableDragon> dragons = new ArrayList(); 
 //   EntityTameableDragon dragon;
    private byte listPos;
	
	public ItemDragonWhistle() {
		this.setUnlocalizedName("dragon_whistle");
		this.setRegistryName(new ResourceLocation(DragonMounts.MODID, "dragon_whistle"));
		this.setMaxStackSize(1);
		this.setCreativeTab(DragonMounts.TAB);
	}
	
	@Override
	@SideOnly(Side.CLIENT)
	public void addInformation(ItemStack stack, @Nullable World worldIn, List<String> tooltip, ITooltipFlag flagIn) {
//		tooltip.add(StatCollector.translateToLocal(dragons.toString()));
	}
	
	
	@Override
	public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer player, EnumHand hand) {
		ItemStack stack = player.getHeldItem(hand);

        if(!worldIn.isRemote && !player.isSneaking() && stack.hasTagCompound()) {
        	DragonMounts.proxy.openDragonWhistleGui(stack.getTagCompound().getUniqueId("dragons"), new ItemStack(this), (WorldServer)worldIn);
        }
        
        return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, stack);
            		
	}
	
	@Override
	public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity target) {
		if(target instanceof EntityTameableDragon) {
			EntityTameableDragon dragon = (EntityTameableDragon)target;
		//	this.dragon = dragon;
			if(dragon.isTamedFor(player)) { 			
				
				NBTTagCompound nbt = new NBTTagCompound();
				NBTTagList nbtList;
				if(stack.hasTagCompound()) {
					nbt = stack.getTagCompound();
					nbtList = nbt.getTagList("dragons", Constants.DEFAULT_BYTE_NO_ENTRY_VALUE); 
				} else {
					stack.setTagCompound(nbt);
				}
				
				if(nbt.hasKey("dragons")) {
					UUID uuid = dragon.getUniqueID();  					
					nbt.setTag("dragons", NBTUtil.createUUIDTag(uuid));	
					//nbt.setInteger("dragons", dragon.getEntityId()); 
				//} else {				
				}
				
				
				player.sendStatusMessage(new TextComponentTranslation("dragon.added" + dragon.getUniqueID().toString(), DragonMounts.MODID), true);
			} else {
				player.sendStatusMessage(new TextComponentTranslation("dragon.notTamed", DragonMounts.MODID), true);
			}
			return true;
		} else {
			return false;
		}
	}
	
/*	if(player.isSneaking()) {
    	controllableDragons.iterator().next();
    	player.sendStatusMessage(dragon.getDisplayName(), true);
    	
    	// setting the value of N as 4 
        int limit = 3; 
        int count = 0; 
        Iterator<EntityTameableDragon> it = controllableDragons.iterator(); 
  
        // Iterating through the list of integers 
        while (it.hasNext()) { 
            it.next(); 
            count++; 
  
            // Check if first four i.e, (equal to N) 
            // integers are iterated. 
            if (count > limit) { 
  
                // If yes then remove all the remaining integers. 
                it.remove(); 
            } 
        } 
  
        System.out.print("New stream of length N"
                         + " after truncation is : "); 
  
        // Displaying new stream of length 
        // N after truncation 
        for (EntityTameableDragon number : controllableDragons) { 
            System.out.print(number + " "); 
        } 
    } */
}

 

Edited by diesieben07
syntax highlighting
Link to comment
Share on other sites

Current Problem :

1. really need a docs to understand NBT's and know what methods to call. Vanilla codes cannot be trusted that much "too many much magic numbers and the effort of cycling through each class used for different problems"

2. Constant has been cast on a wrong import lmao, use forge one

3. Cannot diffrentiate NBTLists to NBTTagCompound, it seems like NBTTagCompound is required to use NBTTagLists,

4. Not knowing Cycling through dragons as mentioned above, with byte and what to call to cycle

5. Can I really use Guis to set dragon with commands, pic from the first post

6. Limits as mentioned, no idea how

 to set it it with NBTTagLists

 

Things I Understand:

1. Items are singletons use ItemStacks for some reason (if have a hacked 64 stack diamond sword then separate them from stack in an inventory its low red bar durability is retained)

2. NBT's as I vaguely understood are a way to store data on stuff. Like entities, items, guis? (GUIScreenBook stored on item or GUI)

3. We will use stored ID's to identify what dragon to control no brainer.

 

 

Edited by TheRPGAdventurer
Link to comment
Share on other sites

26 minutes ago, diesieben07 said:

I... what?

 

Limits? Mentioned where?

 

 

Per Whistle, I mentioned that it can carry 3 dragon datas. So i want a limit for 3 dragons, I also have to ensure the dragons have the player who used it is the dragon's owner, but since items are singletons....

 

So i need an NBTTagList to store 3 dragons and cycle which dragon to control with shift + rightCLicking it and it will print a message in the hotbar saying The details what the dragon is being controlled, Owner  or UUID breed, I can choose maybe at least one so it wont be a hassle.

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.