Как сконвертировать Collection(map.values()) в обычный Set?

Question

[devpy14]

Есть Map:
{English=[Antony], Chemistry=[John], Music=[Antony]}
При использовании map.values() возвращает [[Antony], [John], [Antony]], нужно перевести это все дело в Set, избавившись от дубликатов. Я пришел только к след. варианту:

Set<String> uniqueStudents = new HashSet<>();
        for (List<String> listName: map.values()) {
            uniqueStudents.add(listName.get(0));
        }
        System.out.println(uniqueStudents); // [John, Antony]

Может, кто знает решение поизящней?

Comments

[devpy14]

Bavashi, "никогда не отмечает ответы решением" - что конкретно вы имели ввиду? Не совсем понял

[devpy14]

Bavashi, А, извините, понял) Я на сайте сравнительно не долго, не разобрался еще со всем функционалом, не обратил внимания на кнопочку "отметить решением", впредь обязательно буду отмечать

Answer 1

[BorLaze]

Или вопрос странный, или одно из двух... чем простейшее решение-то не устраивает?

Set<String> uniqueStudents = new HashSet<>(map.values());

ЗЫ: не заметил, что значениями map являются массивы строк, а не строки. Тогда, действительно, надо так:

Set<String> set = map.values().stream()
                               .map(a -> a[0])
                               .collect(Collectors.toSet());

Comments

[BorLaze]

Bavashi, хм... отчего же?

ЗЫ: а... черт возьми! не обратил внимание, что значения в мапе - это массив, а не строка.
Да, тогда работать не будет. Верно.

[BorLaze]

Bavashi, вот теперь уже точно интересно :-)

@Test
    public void t() {
        Map<String, String[]> map = newHashMap();

        map.put("English", new String[]{"Antony"});
        map.put("Chemistry", new String[]{"John"});
        map.put("Music", new String[]{"Antony"});

        Set<String> set = map.values().stream()
                .map(a -> a[0])
                .collect(Collectors.toSet());

        log.debug(set.toString());
    }

результат: [John, Antony]

Что не так?

[BorLaze]

Bavashi, ладно, сдаюсь :-)

С другой стороны, отталкиваясь от примера, второй forEach в Вашем решении тоже избыточен - список-то из одного элемента в примере? :-)))

[Денис Загаевский]

BorLaze, flatMap нужно использовать