Помогите Как из HashMap вытащить всех игроков которые там есть?

Мне нужно чтобы у кого в HashMap<String, Boolean> было true, тому сообщение отправлялось с префиксом [Чат-Персонала] Имя игрок : Сообщение. А тех у кого в HashMap false, тем писались обычные сообщение от игроков

 

for (String a : hashmap.keySet()){
if (hashmap.get(a)) {
//do something
}}

 

Код:

        HashMap<String, Boolean> hash = new HashMap<String,Boolean>();
        for(String key : hash.keySet()){
            if(hash.get(key)){
                player.sendmessage(prefix + player.getDisplayName() + ":"+ msg);
            }
            else{
                player.sendmessage(player.getDisplayName() + ":"+ msg);
              
            }
        }

 

Зачем тебе Map? Можно обойтись HashSet добавляя в него только тех игроков, для которых нужен префикс. Map нужен, если тебе нужно хранить больше состояний.

 

Код:

for(Map.Entry<String, Boolean> entry : hashMap.entrySet()) {
    if(entry.getValue) {
        String playerName = entry.getKey();
        ...
    }
}

А вообще человек выше прав. Если это возможно, лучше просто не добавлять игроков в список. При этом HashMap можно будет заменить на HashSet...

 

+1

Код:

        HashSet<Player> hash = new HashSet<Player>();
        for(Player p : Bukkit.getOnlinePlayers()) {
            if(hash.contains(p)) {
              
            } else {
              
            }
        }

 

-1 за хранение ссылок на игроков.

 

Причем тут ссылки на игроков?

 

Уже не раз обсуждалось это. Хранить ссылки на локации, миры и игроков не есть хорошо.

Можно просто обойтись правами. Баккитовскими. И потом Bukkit.broadcast("You are now op!", "special.permission.for.nerds");

 

Помоему просто broadcast нужно

 

Да, broadcast(String message, String permission).

 

Вот с правами стафф чат

Код:

     if ((command.getName().equalsIgnoreCase("staffchat")) &&
       (p.hasPermission("staff.chat"))) {
       if (args.length == 0) {
          sender.sendMessage("Мало аргументов!");
         return false;
       }
       StringBuilder str = new StringBuilder();
       for (int i = 0; i < args.length; i++) {
         str.append(args[I] + " ");
       }
       String msg = str.toString();
       for (Player player : Bukkit.getOnlinePlayers()) {
         if (player.hasPermission("staff.chat")) {
           player.sendMessage(ChatColor.DARK_GRAY + "[" + ChatColor.YELLOW + "Персонал" + ChatColor.DARK_GRAY + "] " + p.getDisplayName() + ChatColor.YELLOW + ": " + msg);
         }
       }
     }

 

Можно тупо Bukkit.broadcast(msg, "perm")

 

Код:

Set<Player> set = Collections.newSetFromMap(new WeakHashMap<>());

Иногда это необходимо, так как частое использование getPlayer(UUID) - дорогое удовольствие при большом количестве игроков.

 

Сборщик мусора не сможет очищать объекты вышедших игроков, из-за того что ссылки на них висят в коллекции. Также после перезахода для игрока создаётся новый объект.

 

Утечка памяти будет и на обычных String, если не чистить.

 

При более 250 игроков - да, уже дороговато.

 

Весит значительно меньше, нежели объект игрока со многими свойствами.

Для HashSet/Map это ничто. Они и с 5К элементов хорошо работать будут.

 

Uuid, либо ники хранить то можно. Но если получать через них постоянно игрока, то это может очень плохо кончиться для сервера.

 

Вот только getPlayer(UUID) дёргает не HashMap.get(Object), а getOnlinePlayers(), который каждый раз создаёт копию коллекции с игроками. Скорее всего, в последних версиях это исправлено, но на 1.7.10 у одного из моих клиентов getPlayer(UUID) отъедал около 20% CPU.
Так что в критичных участках WeakReference/WeakHashMap и прослушивание PlayerQuitEvent - наше всё.