Solved Saving and Retrieving inventory contents

Okay, so, I made a thread asking how to do this and a guy helped me, I made two methods 'saveContents' and 'retrieveContents' but never used them because I was gonna make it the last thing I do. Now that I actually test them, they don't work.
They are here:

Code:

 
public static HashMap<String, ItemStack[]> invents = new HashMap<String, ItemStack[]>();
public static void saveContents(Player player) {
        if(!invents.containsKey(player.getName())) {
            invents.put(player.getName(), player.getInventory().getContents());
        } else if(invents.containsKey(player.getName())) {
            return;
        }
    }
 
    public static void retrieveContents(Player player) {
        if(!invents.containsKey(player.getName())) {
            return;
        } else if(invents.containsKey(player.getName())) {
            player.getInventory().setContents(invents.get(player.getName()));
        }
    }

Any help?
- Loogeh

help?

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

 

What doesn' t work?

 

That's the thing, I have no idea.

 

Loogeh I'm assuming you want to save the inventories so that went the server restarts you still have them?

If not then this is some clean up code:

Code:java

 
 
public static Map<String, ItemStack[]> invents = new HashMap<String, ItemStack[]>();
 
public static void saveContents(Player player) {
    if(!invents.containsKey(player.getName())) {
        invents.put(player.getName(), player.getInventory().getContents());
    }
}
 
public static void retrieveContents(Player player) {
    if(invents.containsKey(player.getName())) {
        player.getInventory().setContents(invents.get(player.getName()));
    }
}
 
 

If it is then you'll need to save all the inventories into a text file in some format

 

thehutch Thanks for your help but I just figured it out, also, it's for spleef (minigame).

 

May I ask what you did to fix it and the code

 

All I did was remove the 'if' statements, so it's not that good and I would get NPE's if the player isn't in it lol,

Code:

    public static void saveContents(Player player) {
        invents.put(player.getName(), player.getInventory().getContents());
        armours.put(player.getName(), player.getInventory().getArmorContents());
    }
   
    public static void retrieveContents(Player player) {
        player.getInventory().setContents(invents.get(player.getName()));
        player.getInventory().setArmorContents(armours.get(player.getName()));
    }

Code:

    public static HashMap<String, ItemStack[]> invents = new HashMap<String, ItemStack[]>();
    public static HashMap<String, ItemStack[]> armours = new HashMap<String, ItemStack[]>();