Solved HashMap adding value help!

I made a hashmap to store a player's xp (I'm not talking about actual Minecraft xp, I'm talking about the hashmap which is names "playerXp" that has an integer value).

The problem is, I don't know how to add to the value of the hashmap.

Here's what I did so far:

public HashMap<Player,Integer> playerXp = new HashMap<Player,Integer>();

Then, in a method, I said:

playerXp.put(p,10); (I want to change the "10" to the value of the hashmap plus ten)

Please tag me if you can help me! Help is appreciated! I don't know why it wouldn't be!

 

Get the value first, then add to it.

Also, store by the player's UUID, not the player object.

 

I don't know what you mean

Here's my code:

Code:java

package minecraft.Charliechumbuck.me;
 
import java.util.HashMap;
 
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockBreakEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class RPGMain extends JavaPlugin implements Listener{
 
    public int level;
    public int coins;
    public int neededXp;
    public int xp;
    public int world;
    public double maxHealth;
    public HashMap<Player,Integer> playerXp = new HashMap<Player,Integer>();
 
    public void onEnable(){
        getServer().getPluginManager().registerEvents(this,this);
    }
    public void onDisable(){}
 
    public void getWorld(){
        switch(world){
        case 1:
 
        }
    }
 
    @EventHandler
    public void onStartup(PlayerJoinEvent e){
        Player p = e.getPlayer();
        world = 1;
        level = 1;
        playerXp.put(p, 0);
        coins = 0;
        neededXp = 50;
        maxHealth = 20;
    }
 
    @EventHandler
    public void onBlockBreak(BlockBreakEvent e){
        Player p = e.getPlayer();
        if(e.getBlock().getType().equals(Material.IRON_ORE)){
            //playerXp.put(p, playerXp.get(p)+10);
            p.sendMessage("PlayerXp: "+playerXp.get(p));
        }
        //if(xp >= neededXp){
            //level ++;
            //neededXp += 10;
            //xp = 0;
            //maxHealth++;
            //p.setMaxHealth(maxHealth);
            //p.setHealth(maxHealth);
            //p.setExhaustion(20);
        //}   
    }
}

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

 

He means get the value of of the Map (map.get(yourKey)) and then set the value to whatever you want.

He also means that you should use a HashMap<UUID, Integer> instead of a HashMap<Player, Integer> because storing player instances is bad. You can then use put(player.getUniqueId(), 10) and get(player.getUniqueId()). Alternatively, you could use a WeakHashMap, but I don't think trying to explain that is a good idea seeing as don't seem to know how to use a map at all.

 

How am I supposed to get the player UUID from the event?

Now I have "Player p = e.getPlayer();" Garris0n

 

You could try reading what I said again.

 

Garris0n
I did read what you said! I'm saying I don't know how to get they UUID of the player that performed a certain event.

Here's how I get the player who performed the event:

public void blockBreak(BlockBreakEvent e){
Player p = e.getPlayer();
}

I'm saying that I don't know how to do the same with the Player UUID.

Garris0n
Hey! I finally got the whole UUID thing down pat! Only one problem: I still don't know how to add to the integer value.

Here's the line of coding where I put the new Integer value in (I changed the HashMap from <Player, Integer> to <UUID, Integer>):

playerXp.put(p.getUniqueId(),the current integer value goes here+10);

So, the problem is I don't know how to add on to the current integer.

By the way, I might not reply until tomorrow.

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

 

http://docs.oracle.com/javase/7/docs/api/java/util/Map.html

 

Garris0n
Okay, I tried to do this:

playerXp.put(p.getUniqueID,playerXp.get(p.UniqueID)+10);

However, when I break an iron ore block (because that's what triggers this code), it has an error, but when I take out the "playerXp.get(p.UniqueID)" and leave the code at "playerXp.put(p.getUniqueID,10)", it has no error, but it doesn't do what I want it to do as it keeps the current value at ten and doesn't increase it (but you already knew that )

 

You need the parenthesis/brackets/whatever people call them at the end of getUniqueId so It's getUniqueId()

Code:java

playerXp.put(p.getUniqueId(), playerXp.get(p.getUniqueId()) + 10);

However you'll want to make sure they are in the hashmap before you do that because if they're not you can't add 10 to NULL

Code:java

if (playerXp.containsKey(p.getUniqueId())) {
playerXp.put(p.getUniqueId(), playerXp.get(p.getUniqueId()) + 10);
} else {
playerXp.put(p.getUniqueId(), 10);
}

 

Garris0n
FORGET THAT POST!!!!!! I FINALLY DID IT!!! Thanks to you, I can finally do a variable that changes only to a player which can lead up to some great mini-games possibly in the near future.