Jump to content
View in the app

A better way to browse. Learn more.

Forge Forums

A full-screen app on your home screen with push notifications, badges and more.

To install this app on iOS and iPadOS
  1. Tap the Share icon in Safari
  2. Scroll the menu and tap Add to Home Screen.
  3. Tap Add in the top-right corner.
To install this app on Android
  1. Tap the 3-dot menu (⋮) in the top-right corner of the browser.
  2. Tap Add to Home screen or Install app.
  3. Confirm by tapping Install.

Featured Replies

Posted

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

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.

Define "wrong".

1 hour ago, TheRPGAdventurer said:

I should prefer using NBTTaglist then remove NBTTagCompound?

I have no idea what this means, sorry.

  • Author

Sorry my bad. What methods do I call to store UUID's im not so sure. Also 

private List<EntityTameableDragon> dragons = new ArrayList(); 
   EntityTameableDragon dragon;

would these be included ?

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

  • Author

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;
		}
	}

 

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.

15 hours ago, TheRPGAdventurer said:

nbt = new NBTTagCompound();

Make sure you attach this to the itemstack too.

I think its my java of the variables.

  • Author
1 hour ago, RANKSHANK said:

ItemStack.setTagCompound(NbtTagCompound)

what method do i call to get the dragon from uuid or nbt, does it have something to do with world?

  • Author

    
    @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.

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.

  • Author
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

  • Author
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

  • Author

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

  • Author
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.

  • Author
21 hours ago, diesieben07 said:

newIndex = (oldIndex + 1) % Math.min(3, [length of the list]). That will fail if the list is empty

So after I made this, what getter should I call it in the nbt?

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...

Important Information

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

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.