Need help with playerjoin event.

Hello,
Me and my friend made a plugin what vanish all players who is online.
But if anyone connect, then joined player isnt vanished.
Code:

Code:java

package me.sailorerik.mobs;
 
import java.util.ArrayList;
import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerDropItemEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
 
public class main extends JavaPlugin
  implements Listener
{
  ArrayList<Player> cooldown = new ArrayList();
 
  public void onEnable() {
    Bukkit.getServer().getPluginManager().registerEvents(this, this);
  }
 
  @EventHandler
  public void onPlayerJoin(PlayerJoinEvent event) {
    Player player = event.getPlayer();
    ItemStack is = new ItemStack(Material.SLIME_BALL);
    if (!player.getInventory().contains(is))
    {
      player.getInventory().addItem(new ItemStack[] { is });
    }
  }
 
  @EventHandler
  public void Nodrop(PlayerDropItemEvent e)
  {
    if (e.getItemDrop().getItemStack().getType()
      .equals(Material.SLIME_BALL))
      e.setCancelled(true);
    else if (e.getItemDrop().getItemStack().getType().equals(Material.MAGMA_CREAM))
      e.setCancelled(true);
  }
 
  @EventHandler
  public void playerInteract(PlayerInteractEvent event)
  {
    Player player = event.getPlayer();
    final Player player2 = event.getPlayer();
    Action action = event.getAction();
    ItemStack Slime = new ItemStack(Material.MAGMA_CREAM);
    ItemStack magma = new ItemStack(Material.SLIME_BALL);
 
    if ((player.getItemInHand().getType().equals(Material.SLIME_BALL)) && ((event.getAction() == Action.RIGHT_CLICK_AIR) || (event.getAction() == Action.RIGHT_CLICK_BLOCK))) {
      if (this.cooldown.contains(player2)) {
        player2.sendMessage("&cError: &4You are not allowed to use Slimeball too often.");
      }
      else
      {
        for (Player p : Bukkit.getServer().getOnlinePlayers())
          player.showPlayer(p);
        player.sendMessage("§aAll Players: §cShown!");
 
        player.setItemInHand(Slime);
 
        this.cooldown.add(player2);
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {
          public void run() {
            main.this.cooldown.remove(player2);
          }
        }
        , 150L);
      }
 
    }
    else if ((player.getItemInHand().getType().equals(Material.MAGMA_CREAM)) && ((event.getAction() == Action.RIGHT_CLICK_AIR) || (event.getAction() == Action.RIGHT_CLICK_BLOCK)))
      if (this.cooldown.contains(player2)) {
        player2.sendMessage("&cError: &4You are not allowed to use Magma Cream too often.");
      }
      else
      {
        for (Player p : Bukkit.getServer().getOnlinePlayers())
          player.hidePlayer(p);
        player.sendMessage("§aAll Players: §cHidden!");
 
        player.setItemInHand(magma);
        this.cooldown.add(player2);
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {
          public void run() {
            main.this.cooldown.remove(player2);
          }
        }
        , 150L);
        return;
      }
  }
}

 

sailorerik Keep a list of all the players who don't want to see players. On join, hide the player from all of those players.

 

You could create a HashMap to store the all players in, add them to the HashMap when they join the server and then hide everyone in the HashMap, that should work...I think.

 

How i do it then? If anyone can waste time and give code

 

I dont really understand your english (obviously pretty bad at it), but I noticed your Itemstack variables for slimeball and magma cream are the wrong way around.

Code:java

 ItemStack Slime = new ItemStack(Material.MAGMA_CREAM);
    ItemStack magma = new ItemStack(Material.SLIME_BALL);

Kind of annoyed me

 

Yes, am sorry for my bad english, and i fixed it.
But still i need code for hiding players when they connect.

 

Step 1: Create The HashMap
Step 2: Create a onPlayerJoinEvent and add players to it when they join
Step 3: Hide Everyone in HashMap
Step 4: Remove player from HashMap when they leave

The reason your code isn't working is because it checked for players who were online at the time of hiding them, so whenever a new player joins they will be visible since there is no code to update it.

I would add the code of the steps if I had my coding IDE, its a bit tricky without it, if the issue is still not solved when I am home and am able to use my IDE I will write out the code for you.

EDIT:

This may be easier;
Step 1: Create ArrayList

ArrayList al = new ArrayList():

Step 2: Add Players when they join the server

assume I have done the correct onPlayerJoinEvent and I have initialized the Player entity (Player p = event.getPlayer()

al.add(p);

Step 3: Hide everyone in the ArrayList (done nothing here since you should know how to hide the players!)

Step 4: Remove Players from the ArrayList on leaving

assume I have the correct player leaving event

al.remove(p);


That Should work.

 

Alright.
If you are Home, then add code.

All bests,
Sailorerik