Помогите Вытащить из списка только онлайн игроков

[Chupaaa]

Вот у меня есть список игроков, из них нужно получить только тех, которые онлайн.
Как получить всех онлайн игроков знаю, а вот перебрать из определенного списка хз. Прошу вашей помощи

 

[sq179s]

for (Player iterablePlayer : collection)
  if (!iterablePlayer.isOnline())
     collection.remove(iterablePlayer);

// Todo: write code.

Что-т вроде такого?

 

[LuckyZeeRo]

for (Player iterablePlayer : collection)
  if (!iterablePlayer.isOnline())
     collection.remove(iterablePlayer);

// Todo: write code.

Что-т вроде такого?

Привет ConcurrentModificationException.
Нельзя удалять элементы из коллекции во время итерации.

 

[AbstractCoder]

players.stream().filter(Player::isOnline).foreEach(...)

 

[LuckyZeeRo]

Вот у меня есть список игроков, из них нужно получить только тех, которые онлайн.
Как получить всех онлайн игроков знаю, а вот перебрать из определенного списка хз. Прошу вашей помощи

List<Player> players = ...;
List<Player> online = players.stream().filter(Player::isOnline).collect(Collectors.toList());

Или

List<Player> players = ...;
List<Player> online = new ArrayList<>();
for (Player player : players)
    if (player.isOnline()) online.add(player);

Или (если удалить оффлайн)

List<Player> players = ...;
players.removeIf(p -> !p.isOnline);

По факту не рекомендуется хранить игроков как Player. Суть в том, что при выходе и входе будет создаваться новый Player, у вас же останется старый и оффлайн.

 

[Chupaaa]

List<Player> players = ...;
List<Player> online = players.stream().filter(Player::isOnline).collect(Collectors.toList());

Или

List<Player> players = ...;
List<Player> online = new ArrayList<>();
for (Player player : players)
    if (player.isOnline()) online.add(player);

Или (если удалить оффлайн)

List<Player> players = ...;
players.removeIf(p -> !p.isOnline);

По факту не рекомендуется хранить игроков как Player. Суть в том, что при выходе и входе будет создаваться новый Player, у вас же останется старый и оффлайн.

Благодарочка 🙂

 

[sq179s]

Привет ConcurrentModificationException.
Нельзя удалять элементы из коллекции во время итерации.

Если это не коллекция с динамичным размером, ну окей, ответ принят.

 

[alexandrage]

Привет ConcurrentModificationException.
Нельзя удалять элементы из коллекции во время итерации.

Гонишь 😁

        for(int i=0; i<list.size();) {
            if(!list.get(i).isOnline()) {
                list.remove(i);
            } else {
                i++;
            }
        }

 

[AbstractCoder]

Гонишь 😁

        for(int i=0; i<list.size(); i++) {
            if(!list.get(i).isOnline()) {
                list.remove(i);
            }
        }

Ну ты собсна даже неправильно написал)

 

[Nikolai_Faint]

Ну ты собсна даже неправильно написал)

С чего бы?

 

[AbstractCoder]

С чего бы?

Глупыш, i++ нужно делать только если не прошла проверка и не было remove.
Ну и бред это, удобней Collection#removeIf

 

[alexandrage]

Ну ты собсна даже неправильно написал)

Фиксанул)

 

[LuckyZeeRo]

Гонишь 😁

        for(int i=0; i<list.size();) {
            if(!list.get(i).isOnline()) {
                list.remove(i);
            } else {
                i++;
            }
        }

Всмысле гонишь? Тип код sq179s не вызовет CME?