Rename chest with code

Hi. Can i rename chests ? I tries but i cant .actualy i dont know if its possible . I want this because i checking InventoryOpenEvent with inventory title or name

 

crzytlp

Eh... it would require jumping through a few hoops, such as perhaps waiting for an interact event on the chest.. cancelling the event BUT grabbing the chests content, then opening up a new inventory for the player with said content.

Of course the issue then is if multiple people attempt to open it.. you'll also need to account for saving the contents.

May I ask why exactly you need the change the chests name upon opening, as opposed to simply changing the title of the chest before opening?

 

crzytlp If you want to actually change the name, could either use Reflection or edit the Chest's NBT Data (CustomName is a StringTag that stores the name).
Otherwise, could do what valon750 suggested and make a new Inventory and copy contents.

 

Basicly i create a chest which has some items in it when player open up the specific chest which i created i broadcast a message to server player found the chest bla bla bla then i break the chest but i need chest name to break the or location but i cant figure out the location solution and i cant rename the chest and i m stuck now valon750

Code:java

package io.github.taifuru.talipplugin;
 
import java.util.Random;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.inventory.InventoryOpenEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
 
public class RandomChests implements Listener
{
    private final TalipPlugin plugin;
 
    public RandomChests(TalipPlugin instance)
    {
        plugin = instance;
    }
 
    public void chestItemAdd(Location location, Inventory inventory)
    {
        ItemStack itemStack = new ItemStack(Material.APPLE);
 
        int chestSayisi = plugin.getConfig().getInt("ChestSayisi.sayi");
 
        Random rand = new Random();
        int chestSirasi = -1;
 
        while(true)
        {
            if( chestSirasi <= 0)
            {
                chestSirasi = ( 1 ) + rand.nextInt()%chestSayisi;
 
            }else if (chestSirasi > 0)
                break;
        }
 
        int itemSayisi = plugin.getConfig().getInt("ChestSayisi." + chestSirasi + ".10" + chestSirasi );
 
        for(int i=1; i <= itemSayisi; i++)
        {
            itemStack.setType( Material.getMaterial( plugin.getConfig().getString( "ChestSayisi." + chestSirasi + "." + i ) ) );
            inventory.addItem( itemStack );
        }
    }
 
    public static void chestSpawn(TalipPlugin p)
    {
        int coords = p.getConfig().getInt("ChestSayisi.chestSpawnCoords");
 
        while(true)
        {
            Random r = new Random();
            double x = ( -coords ) + ( ( coords ) - ( -coords )) * r.nextDouble();
            double y = ( 64 ) + ( ( 70 ) - ( 64 )) * r.nextDouble();
            double z = ( -coords ) + ( ( coords ) - ( -coords )) * r.nextDouble();
 
            Location loc = new Location(Bukkit.getWorld( p.getConfig().getString("ChestSayisi.world") ), x, y, z);
 
            if(  ( loc.getBlock().getType() == Material.AIR ) )
            {
                Bukkit.getServer().broadcastMessage( ChatColor.BLUE + "[" + p.getConfig().getString("ChestSayisi.randomChest") + "] " + ChatColor.GRAY + p.getConfig().getString("ChestSayisi.chestSpawn") + " :" );
                Bukkit.getServer().broadcastMessage( ChatColor.BLUE + "[" + p.getConfig().getString("ChestSayisi.randomChest") + "] " + ChatColor.GREEN + "(" + (int) x + "," + (int) y + "," + (int) z + ")" );
 
                loc.getBlock().setType(Material.CHEST);
 
                break;
            }
        }
    }
 
    @EventHandler
    public void chestOpenUp( InventoryOpenEvent event )
    {
        Player player = (Player) event.getPlayer();
        Inventory inv = event.getInventory();
 
        if( event.getInventory().getTitle().equalsIgnoreCase( plugin.getConfig().getString("ChestSayisi.randomLoot") ) )
        {
            Location loc = player.getLocation();
            loc = loc.add(5, -5, 5);
 
            for(int y=0; y < 11; y++)
            {
                for(int x=0; x < 11; x++)
                {
                    for(int z =0; z < 11; z++ )
                    {
                        if( loc.getBlock().getType() == Material.CHEST )
                        {
                            if( inv.getTitle().equalsIgnoreCase( plugin.getConfig().getString("ChestSayisi.randomLoot") ) )
                            {
                                chestItemAdd(loc, event.getInventory());
                                loc.getBlock().breakNaturally();
                                Bukkit.getServer().broadcastMessage(ChatColor.BLUE + "[" + plugin.getConfig().getString("ChestSayisi.randomChest") + "] " + ChatColor.GREEN + player.getName().toString() + ": " + ChatColor.GRAY + plugin.getConfig().getString("ChestSayisi.chestFind"));
                            }
                        }
                        loc = loc.add(0,0,-1);
                    }
                    loc = loc.add(-1,0,11);
                }
                loc = loc.add(11,1,0);
            }
            loc = loc.add(0,-11,0);
        }
    }
 
    public TalipPlugin getPlugin() {
        return plugin;
    }
 
}
 

This my RandomChests class file xTigerRebornx valon750

 

crzytlp A Player can only ever have one Inventory open at once (InventoryView is considered as one Inventory in this context), so having something that logs the Location of the chest when they open it to the Player would work (a Map<String, Location> would suffice), then when they close it, you can simply grab them from the Map and destroy the chest at the Location in the map, then remove them from the Map after destroying the Block.

 

how can i know player open up the specific chest witout the name. i cant get the location with InteractInventoryEvent xTigerRebornx

 

crzytlp PlayerInteractEvent, you can get a Block from this. chests are blocks, therefor you are able to get the chest they've interacted with.
Also, in your code, you set the chest. When you set the chest, you have its Location. You need that to determine that the chest the Player opens is your chest.

 

xTigerRebornx Himm then i need store the locations they can be thousands chest around if i store in list or map it can be ram eater

 

crzytlp Are all chests your "chest"?

 

xTigerRebornx Not all are mine they can be normal chests and loot chests

 

crzytlp You could use Metadata to differentiate between your special chest and normal ones.

 

crzytlp
You could save the location of the chest,
then when a player opens a chest, and its location is equal to the saved location, do something with it.

 

xTigerRebornx it looks like the only way using metadata now i m going to learn how metadata works thanks for helping

Dubehh but it can be 1000 chests around its so much location saving

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

crzytlp It honestly shouldn't be that bad, even with 1000 chests (you can replace the Location with a wrapper class that uses less intensive things like 3 ints and a String (world's name)). Metadata isn't persistent, keep that in mind.

 

crzytlp
Then MetaData is the way for you.
I thought you ment 'treasures'. Not a 1000 chests

 

xTigerRebornx which way will be efficient ?

 

crzytlp You could completely go with Metadata, completely go with Maps, or mix them.
I'd use Metadata for both determining if the chest is 'your chest' and the Player's current open Inventory Location.
You can simply attach Metadata to the Player that determines both if they have your Inventory open and if it is your chest (a FixedMetadataValue storing a clone of the Location would work). You'd attach the Metadata to a Player if the Block they clicked has the Metadata that decides if it is your chest.