Solved GUI Inventory Help

Hey, Sorry if this sounds like a noob question but im only learning java so yeah.... My problem is that i have a custom inventory defined in a Listener class but i want the inventory to be visible in another Listener class. Im making a gui plugin where a person right clicks a peice of paper and it open the Inventory But my inventory and PlayerInteractEvent are in seperate classes and i want to make it though so that when the person right clicks the paper it open the menu.

Code:java

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        Player player = event.getPlayer();
        if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK);
        if (player.getItemInHand().getType() == Material.PAPER);
        player.openInventory(inv);

 

jrobi230

Use

Code:java

getServer().getPluginManager().registerEvents(new YourClassName(), this);

 

Wouldn't it be this.<YourClassName>, this
Or does it not matter?

 

KaiPol

You need to instance your class. So it would be new yourClassName. The beauty of OOP

 

Unless you do that before hand with a public <YourClassName> class = new <YourClassName>();

 

mrkirby153

When i use that i get an error saying "The method getServer() is undefined for the type <MYCLASSNAME>. And where would i put it

mrkirby153

Just with the same plugin when players put items in their enderchest and the server closes, the enderchest wont save the inventory. Do you need to use a hashmap or something?

 

jrobi230

getServer() is extended through JavaPlugin. You may want to either pass an instance of your plugin in the constructor and instead use plugin.getServer(). I thought you were working in your main plugin class. As for the inventory not saving, I would suggest you listen for an onInventoryClose (probably not the correct name) and write the data of the inventory to a hashmap<String, ItemStack[]> for "caching" and when the server stops you can write all the data to a file like enderchests.yml and load from it when the server starts up. The only time stuff won't be written to the file if the server crashes or something

 

mrkirby153

how would i write to the hashmap and load it when the server starts up i have made a hash map and an InventoryCloseEvent.

 

jrobi230

Code:java

for(String s : HashMap.keySet()){
getConfig().set(s+".value", HashMap.get(s));
}
saveConfig()
 

to save to a config

Code:java

for(String s : getConfig().getKeys(false)){
HashMap.put(s, config.getString(s+".value))
}

To load. Please don't copy-pasete this in as your attributes may be different. Also make sure what you are trying to save is serializable. Sorry for bad formatting. On an ipad

 

mrkirby153

Thanks for this i was just wondering do i put this in the onEnable() or in the InventoryCloseEvent?

 

jrobi230

Define the hashmap at the top of your main class file and save it to a file on InventoryCloseEvent. Make sure you're saving the correct inventory, not just any inventory that's closed

 

mrkirby153
Im getting errors with your coding in your previous comment and how would i go about saving an actual inventory would i go enderchest.put(Player, einv); or enderchest.put(null, einv); My enderchest inventory is called einv to btw.

And I'm really sorry for asking you so many questions but I'm very new to java and I'm glad your'e not one of those people to pee off and go learn java. Again thanks for persevering with me!

 

Use the player name to save/get.

 

Garris0n

I only want to save the enderchest after the player has exited it not any other inventories the player has.

 

Then only save it in the Map when they close the inventory. Also, note that it won't save through reloads/restarts.

 

Garris0n How would i do that because i have my HashMap and my InvetoryCloseEvent. Also how would i listen for a specific inventory closing?

 

jrobi230 Try InventoryCloseEvent and add a check to see if the inventory is in the HashMap

 

jrobi230 Check the name of the inventory that is being closed......Ninjad?

 

JRL1004
How would i do that would i go enderchest.putAll(einv); or something along those lines?

 

Do some research on HashMaps, that doesn't make any sense.

 

(Total Self Promotion)
http://dev.bukkit.org/bukkit-plugins/gui/

Code:java

GuiAPI api = new GuiAPI();
 
api.tempGUI(
    "My Special Title",
    1,
    new ItemStack[] {
        new ItemStack(Material.PAPER),
        new ItemStack(Material.STONE),
        new ItemStack(Material.BUCKET)
    },
    new String[] {
        "doPaper",
        "doStone",
        "doBucket"
    }
).open(player);

(I hope this is what you are asking for)

Edit: Supid Formatting

 

Don't worry i fixed it myself guys THANKS for ALL YOUR HELP

But special thanks to:
Garris0n
mrkirby153