Как возвратить список игроков из hashMap?

Question

[Cyrus Smith]

Всем привет,
Есть, значит, у меня класс LeagueManager. В нём существует "карта" игроков HashMap. В роли ключа выступает ник игрока(String), в роли значения - сам игрок(Player).
Вопрос написан в топике, но продублирую: Как возвратить список игроков из hashMap? (проблема в методе getAllPlayers())

import java.util.HashMap;
import java.util.Map;

public class LeagueManager implements Manager{
Map players = new HashMap();
public void addPlayer(Player player) {
players.put(player.getNick(), player);
}

public void removePlayer(Player player) {
if (!players.isEmpty()) {
players.remove(player.getNick());
}
}

public Player getPlayer(String name) {
if (!players.isEmpty() && players.containsKey(name)) {
return (Player) players.get(name);
} else {
System.out.println("Error: there is no player with nick " + name);
return null;
}
}

public Player[] getAllPlayers() {
if (!players.isEmpty()) {
return null;
} else {
return null;
}
}

Answer 1

[Cyrus Smith]

На забугорском ресурсе тоже нашлись герои и предложили такой вариант:

public Player[] getAllPlayers() {
Player[] result = new Player[players.size()];
return players.values().toArray(result);
}

Вот он у меня рабочий. Но всем спасибо, что не прошли мимо!
З.Ы. Как я понял, тут избегают изменение типа, тем самым отсекая ошибку, которая вываливалась мне при использовании решения от EugeneP2

Comments

[Эмин]

Ну так спрашивали про возврашение списка, а не массива.

[Cyrus Smith]

Эмин: ответил вам ниже в комментариях.

[bromzh]

Просто вызов метода toArray без параметров вернёт объект типа Object[], а если передать в него переменную, то он будет параметризирован её типом.
Фишка в том, что ты можешь в свой map запихивать любые значения, и компилятор не будет ругаться. А вот уже в рантайме при вызове вот этого метода будут проблемы, если вдруг не все элементы окажутся типа Player.
Поэтому надо использовать дженерики, и писать типобезопасный код. Текущее решение мягко говоря фиговое.

[Cyrus Smith]

bromzh: спасибо, оченнь ценно для меня. Пойду гуглить что такое "типобезопасный код" и как его готовить)

Answer 2

[Evhen]

public Player[] getAllPlayers() {
      return  (Player[])players.values().toArray();
}

Interface Map

И не забываем про дженерики

Comments

[Cyrus Smith]

Спасибо большое

[Cyrus Smith]

Что то рано я обрадовался. Мне компилятор эксепшн выкидывает, мол не может java.lang.Object кастануть в Player

Answer 3

[bromzh]

Дженерики для кого придумали?

Map<String, Player> players = new HashMap<>();
...
public List<Player> getAllPlayers() {
return players.values();
}

Пиши нормальный код. Изучи коллекции и дженерики в яве, это же основа.

Comments

[Николай Павлов]

Тогда уж

public Collection getAllPlayers(){
return players.values();
}