Помогите Как получить всех игроков которое не в режиме спектатора

[Conorfish]

Нужно через запятую получить всех игроков онлайн, у которых геймод не равен спектатору(всех у кого гм 1, гм 2, гм 0), и написать это все игроку "sender". Как это можно сделать?

 

[SERR]

Циклом перебираешь всех игроков, и делаешь проверку на игровой режим.

 

[Conorfish]

Циклом перебираешь всех игроков, и делаешь проверку на игровой режим.

через запятую получить всех игроков

 

[imDaniX]

Далее просто используешь StringBuilder и builder.append(player.getName()).append(" ,"), а в конце отрезаешь последние два символа.
Вот здесь есть ещё вариант с потоком - http://rubukkit.org/threads/poluchenie-vsex-onlajn-igrokov-s-opredelennym-pravom.171534/

 

[alexandrage]

    public void send(Player sender) {
        List<String> players = new ArrayList<String>();
        Bukkit.getOnlinePlayers().stream().filter(player -> !player.getGameMode().equals(GameMode.SPECTATOR)).forEach(player -> {
            players.add(player.getName());
        });
        sender.sendMessage(String.join(",", players));
    }

 

[imDaniX]

Ну можно обойтись обычным циклом for, хотя, каждый дрочит как он хочет)

Я лишь дополнил твой ответ - обычное совмещение строк через + не стоит использовать в цикле - утечка памяти из-за создания кучи объектов String. Вариант из другой темы как альтернатива.

 

[LuckyZeeRo]

Я лишь дополнил твой ответ - обычное совмещение строк через + не стоит использовать в цикле - утечка памяти из-за создания кучи объектов String. Вариант из другой темы как альтернатива.

Какая еще утечка памяти

 

[Dymeth]

Какая еще утечка памяти

Речь про забитие памяти мусором. Правда я сомневаюсь, что это будет хоть как-то ощутимо в текущей задаче

 

[Conorfish]

    public void send(Player sender) {
        List<String> players = new ArrayList<String>();
        Bukkit.getOnlinePlayers().stream().filter(player -> !player.getGameMode().equals(GameMode.SPECTATOR)).forEach(player -> {
            players.add(player.getName());
        });
        sender.sendMessage(String.join(",", players));
    }

public void? насчет sender, я про ввод команды

 

[imDaniX]

public void? насчет sender, я про ввод команды

Ну, просто переделай под себя? В чем проблема?

 

[alexandrage]

public void? насчет sender, я про ввод команды

Головой часто в детстве падал?

 

[Nordia]

    public void send(Player sender) {
        List<String> players = new ArrayList<String>();
        Bukkit.getOnlinePlayers().stream().filter(player -> !player.getGameMode().equals(GameMode.SPECTATOR)).forEach(player -> {
            players.add(player.getName());
        });
        sender.sendMessage(String.join(",", players));
    }

sender.sendMessage(Bukkit.getOnlinePlayers().stream()
        .filter(p -> !p.getGameMode().equals(GameMode.SPECTATOR))
        .map(Object::toString)
        .collect(Collectors.joining(",")));

 

[Nordia]

Я лишь дополнил твой ответ - обычное совмещение строк через + не стоит использовать в цикле - утечка памяти из-за создания кучи объектов String. Вариант из другой темы как альтернатива.

bruh, ради рофла потестил, 1кк стрингов занимают 5 мегабайт

 

[imDaniX]

bruh, ради рофла потестил, 1кк стрингов занимают 5 мегабайт

В конце концов, всегда стоит поддерживаться рекомендуемых практик, если только у тебя нет вариантов по-лучше. Мало ли для каких дел будет использоваться этот или подобный код.

 

[Nordia]

В конце концов, всегда стоит поддерживаться рекомендуемых практик, если только у тебя нет вариантов по-лучше. Мало ли для каких дел будет использоваться этот или подобный код.

в чем проблема стринг билдер в цикле юзать, о каких ты практиках говоришь чел

 

[alexandrage]

sender.sendMessage(Bukkit.getOnlinePlayers().stream()
        .filter(p -> !p.getGameMode().equals(GameMode.SPECTATOR))
        .map(Object::toString)
        .collect(Collectors.joining(",")));

Ну можно и так да. Только как оно выведет? Из Player.toString()?

 

[Nordia]

Ну можно и так да.

не можно а нужно

 

[alexandrage]

Фикс.

    public void send(Player sender) {
        sender.sendMessage(Bukkit.getOnlinePlayers().stream()
                .filter(p -> !p.getGameMode().equals(GameMode.SPECTATOR))
                .map(player -> player.getName()).collect(Collectors.joining(",")));
    }

 

[alexandrage]

Лямдодрочерства много не бывает

 

[imDaniX]

в чем проблема стринг билдер в цикле юзать, о каких ты практиках говоришь чел

Я же и предложил использовать билдер в цикле ._.