Solved Help with Inventories

So i am working on making a backpack plugin(i am planning to release it ) and i am having trouble with keeping the inventory on close and making it different for each player. I have everything working, with out any errors other than well it deleting the inventory on close. please help if possible!

Main class:

Show Spoiler Hide Spoiler

Code:java

package me.rbrick.backpack;
 
import org.bukkit.Bukkit;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
 
public void onEnable(){
Bukkit.addRecipe(new Recipe().BackPackRecipe());
Bukkit.getServer().getPluginManager().registerEvents(new BackPackListener(), this);
}
 
 
}
 

Recipe Class:

Show Spoiler Hide Spoiler

Code:java

import java.util.List;
 
import org.bukkit.Material;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.ShapedRecipe;
import org.bukkit.inventory.meta.ItemMeta;
 
public class Recipe extends Main {
 
public ItemStack backpack(){
ItemStack backpack = new ItemStack(Material.CHEST);
ItemMeta backpackmeta = backpack.getItemMeta();
backpackmeta.setDisplayName("§b§lBackPack");
List<String> backpacklore = new ArrayList<String>();
backpacklore.add("§7Right Click to bring up a inventory for storing your stuff on the go!");
backpackmeta.setLore(backpacklore);
backpack.setItemMeta(backpackmeta);
return backpack;
}
 
public ShapedRecipe BackPackRecipe(){
ShapedRecipe backpack = new ShapedRecipe(this.backpack());
backpack.shape(new String[] {"LLL","L L","LLL" });
backpack.setIngredient('L', Material.LEATHER);
return backpack;
 
}
 
}

Inventory/Listener class:

Show Spoiler Hide Spoiler

Code:java

package me.rbrick.backpack;
 
import org.bukkit.Bukkit;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.Inventory;
 
public class BackPackListener extends Main implements Listener {
 
@EventHandler
public void onPlayerInteract(PlayerInteractEvent e){
 
if(e.getAction().equals(Action.RIGHT_CLICK_AIR) || e.getAction().equals(Action.RIGHT_CLICK_BLOCK)){
if(e.getPlayer().getItemInHand().equals(new Recipe().backpack())){
e.setCancelled(true);
e.getPlayer().openInventory(this.BackPackInv());
 
}
}
 
}
public Inventory BackPackInv(){
Inventory inv = Bukkit.createInventory(null, 9,"§b§lBackPack");
return inv;
}
}
 

Figured id put my code in there. thanks again!

 

rbrick Whats the exact issue?

Also, check out my link for Inventory Menu's. Its a in-depth tutorial.

And for now, could you describe the error?

 

Its not an error. i just do not know how to make virtual inventories keep there inventory on closing of said inventory(basically making it act like a placed chest/enderchest)

 

rbrick Heres how I would do it (Methods for giving a pack, and creating):

Code:java

public static HashMap<String, Inventory> inv = new HashMap<String, Inventory>(); //HashMap of players with backpack
 
public static void givePack(Player player) { //Opens the inventory, or creates on if non existent
if (inv.containsKey(player.getName())) { //Menu has it
player.openInventory(inv.get(player.getName()));
 
} else { //Hashmap doesn't have player, create
Inventory inventory = Bukkit.createInventory(null, 18, player.getName() + "'s backpack!"); //Create inventory
inv.put(player.getName(), inv); //Put them in map
}
 
public static Inventory getPack(Player player) { //Gets a players inventory (back pack)
if (inv.containsKey(player.getName())) { //Playrers in map
return inv.get(player.getName()); //Return inv
} else { //Player doesn't have a pack
givePack(player); //Runs the method above, will create a inventory, see the lines above
}
}

 

rbrick Make a HashMap<String, Inventory>, add them to it on join, then when you open the inventory do player.openInventory((Inventory) hasMapName.get(player.getName()));

 

rbrick And how to use mine:

Code:java

public boolean onCommand(CommandSender sender, Command command, String alias, String[] args) {
Player player = (Player) sender;
if (alias.equalsIgnoreCase("backpack")) {
givePack(player);
 
} else if (alias.equalsIgnoreCase("giveitempack")) {
getPack(player).addItem(new ItemStack(Material.DIAMOND, 1));
}
}

NOTE: My command isn't the best, because I didn't check for sender etc. it was an example

ALSO: the 'givePack' and other methods are assumed to be in the same class as the command/where ver you user. If not, simply CLASS_THE_METHODS_IN.givePack("name"); etc

sgavster Off topic, but don't store Player objects in list's: Store there name's instead

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

 

JPG2000 Oops, should have done that, i'll edit my post(I usually do, I use ArrayLists more often)

 

sgavster ArrayList's and list's are the same...

 

JPG2000 I know, I just wasn't thinking at the time! Everyone makes mistakes!

 

sgavster Hahah, man, were so spamming and bumping this!

Anyways, rbrick try my code examples, it works

 

to open it on right click of the backpack i would call the method in the interact listener right?

 

rbrick Yes.