Solved Why is this happening when i join i get 2 items instead of 1?

When i join the server i get my custom item in slot 0 and 8 but i specifficly made it for slot 8
It's like when ever i logged out of the server and log back in the item spawns in the slot that i selected before i logged out

Code:java

@EventHandler
    public void join(PlayerJoinEvent e){
        Player player = e.getPlayer();
        if (!player.getInventory().contains(Material.COMPASS))
        {
            player.setItemInHand(new ItemStack(Material.COMPASS, 1));
            ItemStack spawnItem = player.getItemInHand();
            ItemMeta im =  spawnItem.getItemMeta();
            im.setDisplayName(ChatColor.YELLOW + "" + ChatColor.BOLD + "Lobby Selector");
            im.setLore(Arrays.asList(ChatColor.AQUA + "Right click to open Lobby Selector", ChatColor.GRAY + "If you lose the Lobby Selector ", ChatColor.GRAY + "Type /ls "));
            spawnItem.setItemMeta(im);
            player.getInventory().setItem(8, spawnItem);
            }
        }

And i don't know what i am doing wrong to create this mistake.

 

Look through your code looking SPECIFICALLY at code that says the word 'set'.
I could tell you the problem , but then you wouldn't learn from your mistakes.

 

I looked over the code and i still don't see what my mistake was lol

 

McKiller5252 Here is another hint. Hand

 

I still don't get it lol I checked everything that has "Hand" and i don't see the mistake

Code:java

 player.setItemInHand(new ItemStack(Material.COMPASS));

i removed the 1

 

McKiller5252 Okay, let me make it clearer....You are calling setItemInHand(), which will set the Item into whatever slot they've selected, and since you have your 1st selected, its adding the compass

 

Look at your code again, You setItemInHand to compass AND you set the Item in slot 8 to an compass.

 

Gee Gee Killer you big derp How did i not see that LOLOL

 

Instead of setting the item in hand to a compass and then using that to get the compass, make a new itemstack and just use that.

Code:java

ItemStack item = new ItemStack(Material.COMPASS);
//do all your other fancy stuff here
player.getInventory().setItem(8, item);